https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7403
--- Comment #17 from Guy Harris <guy@xxxxxxxxxxxx> 2012-09-08 16:36:02 PDT ---
(In reply to comment #15)
> Just to be clear, my first patch, now deprecated, corrected a logic error.
...and isn't now deprecated, it's now marked as "accepted for checkin", and is
still in epan/Makefile.am - the attempt to back it out in 44805 changed the
wrong "HAVE_LIBPY", which broke the build and was undone.
> With https://bugs.wireshark.org/bugzilla/attachment.cgi?id=9098, it needs the
> positive condition to trigger and use awk to append to libwireshark.sym.
That change should be made for the HAVE_PLUGINS optional symbols as well.
> That being said, applying both
> https://bugs.wireshark.org/bugzilla/attachment.cgi?id=8681 and
> https://bugs.wireshark.org/bugzilla/attachment.cgi?id=9098 will be incorrect.
Then 9098 should be replaced with a patch that applies to the current version
of epan/Makefile.am, which has had the change in 8681 made (as that change
fixed a logic error, and did not, in and of itself, break the build, it should
not be reverted), and that also changes the HAVE_PLUGINS optional symbols
stuff.
> The first two patches failed on Solaris and OSX because some of the symbols
> added to libwireshark.def weren't being filtered out. I was about to upload an
> updated patch that had an updated filter, but realized that it would never work
> on Windows, since the .def is being used directly -- and the .def file is a
> native configuration file for Microsoft's LINK.
But that doesn't support libpy support on Windows - it'd be best to have a
mechanism for optionally-exported symbols that works with make, CMake, and
MSVC++ nmake.
--
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.