[libvirt] PATCH: configure.in should use 'ld' from $PATH for checking version-script syntax, not hardcode /usr/bin/ld
Daniel P. Berrange
berrange at redhat.com
Mon Nov 16 13:38:12 UTC 2009
On Mon, Nov 16, 2009 at 02:31:39PM +0100, Daniel Veillard wrote:
> On Sun, Nov 15, 2009 at 06:42:34PM -0800, Steve Yarmie wrote:
> > Hi,
> > I had a few minor patches which I had sent by using Bugzilla.
> > I believe I should have sent them to this list instead...
> > Here's the first (reference 531496
> > https://bugzilla.redhat.com/show_bug.cgi?id=531496)
> > configure.in should use 'ld' from $PATH for checking version-script
> > syntax, not hard-coded in /usr/bin/ld
> > -Steve
> > --- libvirt/configure.in 2009-11-08 20:57:08.194911693 -0800
> > +++ libvirt-ld/configure.in 2009-11-08 22:12:57.524840188 -0800
> > @@ -66,7 +66,7 @@
> > AM_PROG_CC_C_O
> > VERSION_SCRIPT_FLAGS=-Wl,--version-script=
> > -`/usr/bin/ld --help 2>&1 | grep -- --version-script >/dev/null` || \
> > +`ld --help 2>&1 | grep -- --version-script >/dev/null` || \
> > VERSION_SCRIPT_FLAGS="-Wl,-M -Wl,"
> > AC_SUBST(VERSION_SCRIPT_FLAGS)
> I'm not so sure about this honnestly
> The problem is that you don't know which linker gcc (or whatever CC is
> being used) will call. That can be hardcoded inside the gcc spec file
> for example if using a cross compiler.
> I have hit the issue with libxml2 in different ways because I reused
> that code in configure but I still don't know how to really address that
> Using ld from $PATH might be a bit better in some case but it's really
> not garanteed to work in all cases, taht's painful.
I think this is incrementally better, because it means that if someone
does PATH=/opt/other/gcc/install:$PATH ./configure, then the ld
check will probably work correctly, whereas current behaviour is
|: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :|
|: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|
More information about the libvir-list