Balint Reczey wrote:
> On 12/07/2010 04:06 PM, Jeff Morriss wrote:
>> [email protected] wrote:
>>> http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=35145
>>>
>>> User: rbalint
>>> Date: 2010/12/07 05:38 AM
>>>
>>> Log:
>>>   Make libtool export only symbols listed in *.def files
>> [...]
>>> Directory: /trunk/wsutil/
>>>    Changes    Path           Action
>>>    +6 -2      Makefile.am    Modified
>> Hmmm, libwsutil holds the optional targets (e.g., inet_aton() for
>> systems that don't have it).
>>
>> At least on the MacOS 10.5 buildbots, nmedit complains if libtool asks
>> it to export a symbol that isn't present in the library.  I'm not sure
>> of an easy way to fix that (hacking the .def or the .sym file depending
>> on which optional targets are being built doesn't sound fun).  Any
>> ideas?  Or should we export all symbols from libwsutil?
> I revert the change for /trunk/wsutil/ to make us able to build for all 
> platforms.
> 
> I will look into other possibilities of hiding some functions:
> http://www.gnu.org/software/hello/manual/gnulib/Exported-Symbols-of-Shared-
> Libraries.html

Oops, looks like I just beat you to it.

As mentioned in the commit message, it may not be worth it for 
libwsutil: on my system there are only 21 global symbols (including 
things like _init, _end, etc.).  Since this library's purpose is to 
provide utility functions, I suppose we may be able to control what it 
exports fairly easily (there hopefully won't be any cases of module A's 
symbols being needed by module B where that symbol isn't also needed 
outside of the library).
___________________________________________________________________________
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

Reply via email to