[PATCH] shared libbusybox, option for IMA mode

Bernhard Fischer rep.nop at aon.at
Tue Jan 17 21:50:51 UTC 2006


On Mon, Jan 16, 2006 at 10:40:29AM +0100, Peter S. Mazinger wrote:
>On Mon, 16 Jan 2006, Bernhard Fischer wrote:
>
>> On Sun, Jan 15, 2006 at 07:29:52PM -0600, Rob Landley wrote:
>> >On Sunday 15 January 2006 08:54, Bernhard Fischer wrote:
>> >> >It does. I'll apply it later today.
>> >>
>> >> done.
>> >>
>> >> Please do holler if I broke something.
>> >
>> >You added a warning to every single file in the tree:
>> >
>> >In file included from /home/landley/busybox/busybox/include/unarchive.h:14,
>> >                 from /home/landley/busybox/busybox/archival/ar.c:37:
>> >/home/landley/busybox/busybox/include/libbb.h:89: warning: 
>> >`externally_visible' attribute directive ignored
>> 
>> hmz. There we are again, we need that platform.h, obviously.
>> Thing is, we need this attribute for gcc >= 4.1 to flag bb_applet_name
>> and bb_show_usage global as we're using -fwhole-program there but need
>> these two symbols if busybox itself uses libbusybox.so.
>
>check for gcc, and make all those __attribute__ depedendent on the used 
>version, else noop

Would something like the attached be ok for a start?
I guess no, since we need two attributes for e.g. bb_show_usage().
Needing more than one attribute is rare in busybox, so for the most
part this would do it.

The file also refers to a thread which was started here
http://busybox.net/lists/busybox/2005-October/016627.html
about that _usage.h.

PS: Someone should add ATTRIBUTE_EXTERNALLY_VISIBLE to naked-include
-------------- next part --------------
A non-text attachment was scrubbed...
Name: platform.h
Type: text/x-chdr
Size: 5635 bytes
Desc: not available
Url : http://lists.busybox.net/pipermail/busybox/attachments/20060117/76f166b3/attachment.h 


More information about the busybox mailing list