Wireshark mailing list archives
Re: How to add ilbc library to wireshark CMake?
From: João Valverde <joao.valverde () tecnico ulisboa pt>
Date: Sun, 29 Dec 2019 20:14:14 +0000
On 29/12/19 13:46, Jiří Novák wrote:
Hi,For Ubuntu there is tools/debian-setup.sh that installs optional packages (as this). I suggest you to test your change at least on this platform since it's the most common.OK. I will try.Did you make your change compilable without that library? You need some preprocessor ifdefs for that.I'm familiar with optional compilation with configure tools, but I'm lost in CMake... On the other hand I tested it and it is possible to compile it without library - CMake detects it is not there and do not compile the code. Therefore it looks I wrote it correctly. BTW I have no idea where to get iLBC library for Windows and Mac. The library is open source therefore we can try to compile it, but I don't know how to incorporate this step/procedure to build of wireshark. Redhat like systems use version 1.1.1 from 2012 but many other platforms use latest 2.0.2 from 2014.If you push your change as WIP you can add me as reviewer: I have a bunch of builders for many platforms. I can help you at least to compile it on as many as possible.OK, thank you for offer. I'm waiting for accepting a few changes it depends on and then I will push it.
I think it would be helpful to push it anyway (your call). It will speed up the review of the pending patches and make it easier for us to help.
Marking it [WIP] or [DONOTCOMMIT] is a good idea in case it doesn't show a merge conflict (so it doesn't get merged ahead of the dependencies).
___________________________________________________________________________ Sent via: Wireshark-dev mailing list <wireshark-dev () wireshark org> Archives: https://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-dev mailto:wireshark-dev-request () wireshark org?subject=unsubscribe
Current thread:
- Re: How to add ilbc library to wireshark CMake? Jiří Novák (Dec 28)
- Re: How to add ilbc library to wireshark CMake? Dario Lombardo (Dec 28)
- Re: How to add ilbc library to wireshark CMake? Jiří Novák (Dec 29)
- Re: How to add ilbc library to wireshark CMake? Pascal Quantin (Dec 29)
- Re: How to add ilbc library to wireshark CMake? João Valverde (Dec 29)
- Re: How to add ilbc library to wireshark CMake? João Valverde (Dec 29)
- Re: How to add ilbc library to wireshark CMake? Roland Knall (Dec 29)
- Re: How to add ilbc library to wireshark CMake? João Valverde (Dec 29)
- Re: How to add ilbc library to wireshark CMake? João Valverde (Dec 29)
- Re: How to add ilbc library to wireshark CMake? Jiří Novák (Dec 29)
- Re: How to add ilbc library to wireshark CMake? Dario Lombardo (Dec 28)