Am Sonntag, 4. März 2007 schrieb Per Inge Mathisen:
> On 3/4/07, Dennis Schridde <[EMAIL PROTECTED]> wrote:
> > Perhaps you could point out the problems the code created on your box...
> >
> > What do you need _GNU_SOURCE for? Maybe we can replace the GNU
> > extension which caused a problem with a POSIX function... Without knowing
> > which function this is, I have no chance to do that.
>
> Without it defined the function definition for strsignal() will not be
> found. You might perhaps use psignal() instead, which is an old BSD
> thing, I believe. But then again, strsignal() might be just as
> portable for all I know. This is where I would normally recommend
> doing an autoconf check...
We could instead use the implementation I had for this in the read_gdmp.c 
file. As we only have a very limited set of signals which we handle, this 
wouldn't be too difficult.

> > Why are the sys/types and sys/stat headers needed? They should be
> > included by the other headers allready... They are guarded by checks on
> > __USE_XOPEN, which is set by_XOPEN_SOURCE. Maybe defining that instead
> > could help. But I have no idea what the problem was, so I don't know how
> > to fix it, either.
>
> At least one of them was needed to find the definitions of the open()
> mode flags. I just looked in "man 2 open" and looked at the headers
> listed there, and copied in the missing ones.
>
> Strange that it did not generate the same warnings on your box. What
> linux distro are you running?
Gentoo
sys-libs/glibc-2.5
sys-devel/gcc-4.1.2

But I think the unwarned usage of GNU extensions might be caused by my 
sdl-config having _GNU_SOURCE in it's CFLAGS, which defines 
_XOPEN_SOURCE=600, which defines __USE_XOPEN...

Attachment: pgpZo7xgKvplL.pgp
Description: PGP signature

_______________________________________________
Warzone-dev mailing list
Warzone-dev@gna.org
https://mail.gna.org/listinfo/warzone-dev

Reply via email to