Re: [libvirt] PATCH: configure.in should use 'ld' from $PATH for checking version-script syntax, not hardcode /usr/bin/ld

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
properly.

 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.

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel@xxxxxxxxxxxx  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/

--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]