[PATCH] libkdeinit symbol pruning
David Faure
faure at kde.org
Mon Jul 19 00:16:59 CEST 2004
On Sunday 18 July 2004 16:25, Karl Vogel wrote:
> - appendLines("libkdeinit_${kdeinit}_la_LDFLAGS = -no-undefined -avoid-version \$(all_libraries)\n");
> + appendLines("libkdeinit_${kdeinit}_la_LDFLAGS = -Wl,--version-script=\$(top_srcdir)/admin/kdeinit.map -no-undefined -avoid-version \$(all_libraries)\n");
Looking at kdelibs/kjs/Makefile.am, I think this should read
if include_VERSION_SCRIPT
KDEINIT_VSCRIPT = -Wl,--version-script=$(top_srcdir)/admin/kdeinit.map
endif
libkdeinit_${kdeinit}_la_LDFLAGS = $(KDEINIT_VSCRIPT) -no-undefined -avoid-version \$(all_libraries)
so that it doesn't break on compilers without --version-script support.
Can you make am_edit and unsermake patches for the above?
I'm inclined to commit the whole thing afterwards. Hmm, this will break for
apps currently linking to kdeinit modules (right?), but that's unportable
anyway, and warned as being unportable by at least unsermake, so if we can
make it break even on Linux people won't do that anymore :)
But, hmm, that makes it difficult to commit this while KDE is feature-frozen.
I'll for sure use it locally from now on to see what needs to be fixed.
--
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).
More information about the Kde-optimize
mailing list