Re: new module 'ldd'

[Ralf Wildenhues]

Hi Bruno,

* Bruno Haible wrote on Thu, Jan 12, 2006 at 01:56:07PM CET:
> 
> gnulib is a portability library, and "ldd" is not portable. So I'm adding
> the following module 'ldd'.

This is not portable to packages that use libtool.

> AC_DEFUN([gl_LDD],
> [
>   AC_REQUIRE([AC_CANONICAL_HOST])
>   AC_REQUIRE([AC_PROG_CC])
>   dnl Default values.
>   LDDPROG=':'
>   LDDPOSTPROC=
>   dnl First try objdump, since it works when cross-compiling.
>   AC_CHECK_TOOL([OBJDUMP], [objdump], [:])

This will conflict with libtool-set $OBJDUMP in packages that use
libtool.  Libtool currently uses
  AC_CHECK_TOOL([OBJDUMP], [objdump], [false])

so both this:

>   if test "$OBJDUMP" != ":"; then

and the corresponding test will not behave as expected with given
combinations of gl_LDD and AC_PROG_LIBTOOL.

>     LDDPROG="LC_ALL=C $OBJDUMP -p"

I'd rather put the LC_ALL=C elsewhere.  configure scripts set this by
default already.  Your shell wrapper could trivially set this before
invoking the tool.

Cheers,
Ralf