Date: Thu, 25 Feb 2016 13:27:54 -0500 From: Rich Felker <dalias@...c.org> To: musl@...ts.openwall.com Subject: Re: Re: [PATCH] Include missing reference to stdio.h for EOF definition On Thu, Feb 25, 2016 at 10:50:01AM +0100, Michael Meeuwisse wrote: > Following up. I know this is a rather silly patch to accept, but I'm trying > to get OSv (http://osv.io) up to date. It currently uses 0.9.12 which did > have this line, but it was dropped (imho erroneously) last June in > following commit; > http://git.musl-libc.org/cgit/musl/commit/?id=1507ebf837334e9e07cfab1ca1c2e88449069a80 > > Since OSv does a few.. unconventional.. things in regards to musl > inclusion, relying on stdio.h being included implicitly here doesn't work, > and working around this omission is a far larger effort than simply > correcting it upstream. Not saying I'm not trying to make the inclusion > more conventional, just that there's bigger fish to fry at the moment. Thanks for the explanation of your motivation. I wasn't opposed to this patch but was curious why it mattered. I'd actually like to go through the source at some point and find places we're making unreasonable assumptions about implicit inclusion, and also get rid of some of the unnecessary implicit inclusions (e.g. stdio.h from libc.h makes no sense and is probably just there because __lockfile is declared in a poorly chosen place). But for now I'm happy to accept this patch as a start. Rich > On Tue, Feb 23, 2016 at 11:49 PM, Michael Meeuwisse <mickeymeeuw@...il.com> > wrote: > > > No binary changes as it was already included implicitly; > > . src/multibyte/internal.h > > .. ./src/internal/locale_impl.h > > ... ./src/internal/libc.h > > .... ./include/stdio.h > > > > However, all evidence suggest this to be a fortunate accident rather than > > an intentional design choice. > > > > Signed-off-by: Michael Meeuwisse <mickeymeeuw+git@...il.com> > > --- > > src/multibyte/wctob.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/src/multibyte/wctob.c b/src/multibyte/wctob.c > > index 4aeda6a..b484a3f 100644 > > --- a/src/multibyte/wctob.c > > +++ b/src/multibyte/wctob.c > > @@ -1,4 +1,5 @@ > > #include <wchar.h> > > +#include <stdio.h> > > #include <stdlib.h> > > #include "internal.h" > > > > -- > > 2.5.0 > > > >
Powered by blists - more mailing lists
Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.