Re: [PATCH 4/4] Add $pic_flag to archive_cmds and archive_expsym_cmds.

[Ralf Wildenhues]

[ <http://thread.gmane.org/gmane.comp.gnu.libtool.general/10794/focus=9770> ]

* Ralf Wildenhues wrote on Sun, Apr 04, 2010 at 10:57:04AM CEST:
> * libltdl/m4/libtool.m4 (_LT_LINKER_SHLIBS)
> <archive_cmds, archive_expsym_cmds>
> [GNU ld, netbsd]: Add $pic_flag if ELF.
> [GNU ld, solaris]: Likewise.
> [GNU ld, default case]: Likewise.
> [!GNU ld, freebsd, dragonfly]: Add $pic_flag.
> [!GNU ld, GCC, hpux9]: Add $pic_flag.
> [!GNU ld, GCC, hpux10]: Replace -fPIC with $pic_flag
> [!GNU ld, GCC, hpux11, !hppa*64]: Replace -fPIC with $pic_flag.
> [!GNU ld, GCC, irix5, irix6, nonstopux]: Add $pic_flag.
> [!GNU ld, GCC, osf4, osf5]: Likewise.
> [!GNU ld, GCC, solaris]: Likewise.
> (_LT_LANG_CXX_CONFIG) <archive_cmds, archive_expsym_cmds>
> [hpux9]: Add $pic_flag if $GXX.
> [hpux10, hpux11, !hppa*64]: Replace -fPIC with $pic_flag.
> [irix5, irix6]: Add $pic_flag if $GXX.
> [osf4, osf5]: Likewise.
> [solaris]: Add $pic_flag if $GXX and not GCC 2.7.

> This untested patch cleans up a number of hard-coded -fPIC
> entries in archive_cmds and archive_expsym_cmds settings,
> and adds $pic_flag to all that use GCC on ELF systems.
> However, the patch may also change the link command lines
> for some non-GCC compilers as well.  It needs testing.

Since I intend to test all listed systems (hopefully tomorrow), and
regard the patch as pretty safe iff it survives that testing (and
failure fairly obvious if not), I'm pushing it now, after rebasing it
against master (with obvious conflict resolution) and retesting.

That leaves only
<http://thread.gmane.org/gmane.comp.gnu.libtool.patches/9767>
plus its requirement wrt.
<http://thread.gmane.org/gmane.comp.gcc.devel/112719>

for LTO support.

> In _LT_LANG_CXX_CONFIG, hpux10|hpux11, GXX, hppa*64* case,
> the -fPIC there is probably bogus and can be removed, but
> that can be another patch (and I haven't tested that either.)

No changes on this front.

Cheers,
Ralf