[PATCH] memmem removed from gdbserver's AC_CHECK_DECLS.
Pedro Alves
palves@redhat.com
Thu Apr 19 12:21:00 GMT 2012
On 04/19/2012 11:24 AM, Pedro Alves wrote:
> 2012-04-19 Pedro Alves <palves@redhat.com>
>
> * configure.ac: Remove AC_CHECK_DECLS check for memmem.
> * server.h (memmem): Remove declaration.
> * config.in: Regenerate.
> * configure: Regenerate.
This actually breaks the build on Windows ...
../../../src/gdb/gdbserver/server.c: In function 'handle_search_memory_1':
../../../src/gdb/gdbserver/server.c:757:7: error: implicit declaration of function 'memmem' [-Werror=implicit-function-declaration]
../../../src/gdb/gdbserver/server.c:757:17: error: assignment makes pointer from integer without a cast [-Werror]
because nothing is pointing the compiler at gnulib's build directory, which
contains the generated replacement headers (string.h in this case), which contain
the memmem declaration for systems that need it, such as mingw32.
Missed it before because I had originally written that patch against the
new wrapper gnulib, which didn't have this bug.
This fixes it. Applied.
Note GDB does the exact same thing (discounting the source path difference):
INCGNU = -I$(srcdir)/gnulib -Ignulib
2012-04-19 Pedro Alves <palves@redhat.com>
* Makefile.in (INCGNU): Add -Ignulib.
---
gdb/gdbserver/Makefile.in | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/gdb/gdbserver/Makefile.in b/gdb/gdbserver/Makefile.in
index 94b9cae..1e78f98 100644
--- a/gdb/gdbserver/Makefile.in
+++ b/gdb/gdbserver/Makefile.in
@@ -77,7 +77,7 @@ ustinc = @ustinc@
# gnulib
LIBGNU = gnulib/libgnu.a
-INCGNU = -I$(srcdir)/../gnulib
+INCGNU = -I$(srcdir)/../gnulib -Ignulib
# We build gnulib directly under the gdbserver build directory, but
# its sources don't live directly under gdbserver's source directory.
More information about the Gdb-patches
mailing list