[Libguestfs] [PATCH] launch: add missing headers on Darwin
Margaret Lewicka
margaret at meaningless.name
Wed Nov 4 12:43:31 UTC 2015
On 4 November 2015 at 12:37, Pino Toscano <ptoscano at redhat.com> wrote:
[...]
> NACK, these OS-specific blocks are hard to maintain, and it makes no
> sense to enclose POSIX headers in them.
>
> Can you please try the attached patch? I see Rich already pushed your
> patch, but IMHO that should be reverted and this one should go...
Happy to. However, these same header files have been removed in
3b17152ec399acb35b23331d2c3b4f424a68824a:
commit 3b17152ec399acb35b23331d2c3b4f424a68824a
Author: Richard W.M. Jones <rjones at redhat.com>
Date: Tue Sep 29 21:58:41 2015 +0100
lib: Remove unused header files.
I used ESR's deheader program to look for unused includes. I then
examined each instance by hand, and also test-compiled (on Linux).
...which makes me suspect it will happen again at some point unless there
is some indication that it's a non-Linux include.
--
Margaret
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libguestfs/attachments/20151104/88e4ff3e/attachment.htm>
More information about the Libguestfs
mailing list