This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
On Tuesday 20 March 2012 12:30:10 Hans-Peter Nilsson wrote: > From: Mike Frysinger <vapier@gentoo.org>: > > On Tuesday 20 March 2012 01:54:27 Hans-Peter Nilsson wrote: > > > nrun.o -MMD -MP -MF .deps/nrun.Tpo /tmp/r/sim/cris/../common/nrun.c > > > /tmp/r/sim/cris/../common/nrun.c: In function 'main': > > > /tmp/r/sim/cris/../common/nrun.c:206: warning: implicit declaration of > > > function 'strsignal' /tmp/r/sim/cris/../common/nrun.c:206: warning: > > > format '%s' expects type 'char *', but argument 4 has type 'int' > > > Don't you see that warning? > > > > the code compiles warning free for me > > > > the file should have a string.h include for the strsignal prototype, so > > i'll update that. > > > > > Instead, to #define _GNU_SOURCE in the Right Way AFAIK, there > > > should be an AC_GNU_SOURCE (missing) in a configure.* used by > > > all src/sim/* (common/configure.ac AFAICT), and an #include > > > > > "cconfig.h" in nrun.c followed later by the mantra: > > not entirely the right way. my guess is that your older glibc does not > > conform to the newer POSIX spec. strsignal() was added in the 2008 spec, > > but old glibc's provided it only when _GNU_SOURCE was defined. > > It's be the right way to maintain interoperability even with old > systems not-fully-conformant-in-every-corner that *used* to > work, that's not what i meant. but it doesn't really matter. patch posted. -mike
Attachment:
signature.asc
Description: This is a digitally signed message part.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |