[Buildroot] [PATCH 1/8] package/netbsd-compat-headers: provide compatibility headers not in musl

[Khem Raj]

> On Aug 12, 2016, at 2:39 PM, Yann E. MORIN <yann.morin.1998 at free.fr> wrote:
> 
> Khem, All,
> 
> On 2016-08-12 14:30 -0700, Khem Raj spake thusly:
>> 
>>> On Aug 12, 2016, at 1:49 PM, Yann E. MORIN <yann.morin.1998 at free.fr> wrote:
>>> 
>>> musl provides neither sys/queue.h nor sys/cdefs.h. Those two headers are
>>> however quite widely used in a lot of packages (though they should at
>>> least not use cdefs,h which is only full of mostly-legacy macros, and
>>> which is mostly an internal header of glibc and was never really meant to
>>> be exposed to, and used by packages).
>>> 
>>> But we don;t live in an ideal world, so a lot of packages break when
>>> those two headers are missing.
>>> 
>>> We already took care of sys/queue.h with the netbsd-queue package. But
>>> the need for cdefs.h is getting more and more pressing.
>>> 
>>> We rename the netbsd-queue package into musl-compat-headers, and we
>>> make it install sys/queue.h (from NetBSD) and sys/cdefs.h (a minimalist
>>> one we bundle in Buildroot). We can't use the cdefs.h from NetBSD
>>> because it includes machine-dependent headers; instead we bundle a very
>>> minimalistic one, that covers only what we need.
>> 
>> see here http://git.openembedded.org/openembedded-core/tree/meta/recipes-core/bsd-headers/bsd-headers
> 
> Thanks for the pointer! :-)
> But no licensing info in that file, so we can't use it... :-/

Oh, that can be fixed.

> 
> Also, we don't want to add a #warning, to avoid having to patch those
> bad packages that find it smart to add -Werror by default.

yet to encounter this case in OE world yet.

> 
>> you might need some more.
> 
> Right now, all the packages in Buildroot that we were patching because
> of cdefs.h only need those three macros: __BEGIN_DECLS(), __END_DECLS()
> and __P().
> 
> There is no point in adding more than what we strictly require. As I
> said in the cover letter, we can add the missing macros when we see the
> issue.

Fair enough.

> 
> TRhanks for the feedback! :-)
> 
> Regards,
> Yann E. MORIN.
> 
> --
> .-----------------.--------------------.------------------.--------------------.
> |  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
> | +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
> | +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
> | http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
> '------------------------------^-------^------------------^--------------------'

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 204 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20160812/6261bcbf/attachment.asc>