[haiku-bugs] Re: [Haiku] #12617: Merge all instances of BSD headers into one place (WIP/Help requested)

  • From: "TwoFx" <trac@xxxxxxxxxxxx>
  • Date: Tue, 26 Jan 2016 08:46:52 -0000

#12617: Merge all instances of BSD headers into one place (WIP/Help requested)
----------------------------+----------------------------
   Reporter:  TwoFx         |      Owner:  korli
       Type:  bug           |     Status:  assigned
   Priority:  normal        |  Milestone:  R1
  Component:  Build System  |    Version:  R1/Development
 Resolution:                |   Keywords:
 Blocked By:                |   Blocking:
Has a Patch:  0             |   Platform:  All
----------------------------+----------------------------

Comment (by TwoFx):

 Replying to [comment:4 pulkomandy]:

However, your patch moves a lot more things to the public headers, which
 shouldn't be there. All the "bus" and other low-level things are for
 kernel-side of FreeBSD, not public APIs, and we don't want to expose those
 as public headers.

So, I suggest moving only what you really need (queue.h, maybe ar.h) to
 the public headers.

 Agreed, I will scrap the last commit of the patch and redo it in a more
 selective way.

I have a doubt about ar.h: are we using the same format as FreeBSD for
 ar archives? Or are we using some gnu-extended one?

 I am no expert on the AR format, but here are my observations: Right now,
 Haiku does not supply ar.h at all. The ar.h which is installed on my
 Debian machine and is part of a reasonably modern version of the GNU C
 library does not define anything which the FreeBSD version which I
 included in the patch does not also define. To me, this suggests that GNU
 does not extend the AR format in a way which would concern this header.
 If you want to be safe, pulling the header locally into the
 libelftoolchain patches rather than making it available publicly might be
 an option.

--
Ticket URL: <https://dev.haiku-os.org/ticket/12617#comment:5>
Haiku <https://dev.haiku-os.org>
Haiku - the operating system.

Other related posts: