This is the mail archive of the gdb@sources.redhat.com 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]

Re: removes trivial compilation warning


Daniel Jacobowitz wrote:
> Look down a line in your patch.  Is HAVE_STDLIB_H not getting defined?
> 
OK :-)

the configure.in in the gdbserver directory does not check for for stdlib (but use the HAVE_STDLIB_H as seen above).

gdbreplay.c include the local config.h (in the gdbserver directory), so
there is no define HAVE_STDLIB_H 

In the directory above this, the check is there, it this one: 
AC_CHECK_HEADERS(stdlib.h)

As far as I can see the configure in gdb/gdbserver
dont use other configure results (e.g. by including ../config.h),
so I suggest to extend the AC_CHECK_HEADERS to check also                       for stdlib in gdb/gdbserver/configure.in 

Of course my patch was bullshit :-)

    Joerg


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]