On Dec 9, 2014, at 1:25 PM, Wireshark code review <[email protected]> wrote:
> URL: > https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=3147087de323c0294b51006f97b641fc408f1b06 > Submitter: Gerald Combs ([email protected]) > Changed: branch: master > Repository: wireshark > > Commits: > > 3147087 by Gerald Combs ([email protected]): > > Fix ranlib "has no symbols" errors in the codecs directory. > > Don't build the G.722 or G.726 codecs for now. Each are wrapped in > "#ifdef HAVE_SPANDSP", which we don't define. Should we be checking for that library: http://www.soft-switch.org as per, for example: https://www.wireshark.org/lists/wireshark-dev/201412/msg00013.html and https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5619 or is spandsp not yet mature or cross-platform (where "platform" includes "CPU" - and "generation of the instruction set of that CPU") enough? They also say: http://www.soft-switch.org/installing-spandsp.html "A number of Linux and other software distributions include SpanDSP, but they usually supply older versions of the library, which lack a lot of the features of the current version." Even if we don't want to use it by default if present, should we have --with-spandsp, defaulting to "no", so that people don't have to do quite as much work to use it if they choose to build a custom version of Wireshark? ___________________________________________________________________________ Sent via: Wireshark-dev mailing list <[email protected]> Archives: http://www.wireshark.org/lists/wireshark-dev Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev mailto:[email protected]?subject=unsubscribe
