Re: [PATCH 0/3] Use gnulib c-ctype module, not <ctype.h>

[Tom Tromey <tromey@adacore.com>]

Guinevere> This patch series causes compilation to fail when using clang to
Guinevere> compile GDB in fedora rawhide.

Guinevere> The errors look like this:

Guinevere> /usr/bin/ld: gdbsupport/common-utils-ipa.o: in function
Guinevere> `strtoulst(char const*, char const**, int)':
Guinevere> /home/buildbot/buildbot/binutils-gdb-clang-fedrawhide-x86_64/build/gdbserver/../gdbsupport/common-utils.cc:256:(.text+0xa3d):
Guinevere> undefined reference to `c_isalnum'
Guinevere> /usr/bin/ld:
Guinevere> /home/buildbot/buildbot/binutils-gdb-clang-fedrawhide-x86_64/build/gdbserver/../gdbsupport/common-utils.cc:256:(.text+0xb0a):
Guinevere> undefined reference to `c_isalnum'
Guinevere> clang++: error: linker command failed with exit code 1 (use -v to see
Guinevere> invocation)

Guinevere> I have no idea why the newest clang has this problem and no other
Guinevere> compiler does, might be a bug, but regardless, it is causing the clang
Guinevere> buildbot to stop working

gnulib + libinproctrace.so + clang ... lol

I was able to reproduce this.  I am not completely sure what to do here.
Maybe for the IPA case we can use <ctype.h> conditionally.  I'll
probably take this route.

Another option would be to write our own thing akin to "c-ctype" but
that uses the libiberty code instead.  That would work since
libinproctrace.so seems to link against libiberty.

I am more and more tempted to remove gnulib entirely.

I also wonder - is anyone using libinproctrace.so?

Tom