Subject: CVS commit: [netbsd-1-6] pkgsrc
To: None <pkgsrc-changes@netbsd.org>
From: Thomas Klausner <wiz@netbsd.org>
List: pkgsrc-changes
Date: 10/24/2002 19:59:57
Module Name:	pkgsrc
Committed By:	wiz
Date:		Thu Oct 24 16:59:57 UTC 2002

Modified Files:
	pkgsrc/audio/arts [netbsd-1-6]: Makefile
	pkgsrc/audio/kdemultimedia2 [netbsd-1-6]: Makefile
	pkgsrc/audio/kdemultimedia3 [netbsd-1-6]: Makefile
	pkgsrc/graphics/dx [netbsd-1-6]: Makefile
	pkgsrc/graphics/gimp-base [netbsd-1-6]: Makefile
	pkgsrc/graphics/kdegraphics2 [netbsd-1-6]: Makefile
	pkgsrc/graphics/kdegraphics3 [netbsd-1-6]: Makefile
	pkgsrc/misc/kdeutils2 [netbsd-1-6]: Makefile
	pkgsrc/misc/koffice [netbsd-1-6]: Makefile.common
	pkgsrc/mk [netbsd-1-6]: defs.Darwin.mk defs.Linux.mk defs.NetBSD.mk
	    defs.SunOS.mk
	pkgsrc/www/ap-php4 [netbsd-1-6]: Makefile
	pkgsrc/www/galeon [netbsd-1-6]: Makefile
	pkgsrc/www/php3 [netbsd-1-6]: Makefile.module
	pkgsrc/www/php4 [netbsd-1-6]: Makefile Makefile.module distinfo
	pkgsrc/www/php4/patches [netbsd-1-6]: patch-aa

Log Message:
Pull the following up to the 1.6 branch (requested by agc):

Apply patches from Lubomir Sedlacik <salo@Xtrmntr.org> in PR 18115 to
generalise the linker flags used to export symbols by setting them on
a per-OS basis.

> many packages force -Wl,-export-dynamic which is not portable outside GNU ld
> and cause problems e.g. on Solaris.  some of these packages use if
> conditionals either only for NetBSD or except SunOS, but the state is not
> coherent and it may complicate later when support for new OS is added to
> pkgsrc (e.g. ongoing work on HP-UX support).
>
> jlam proposed the following framework in discussion on tech-pkg:
>
>   http://mail-index.netbsd.org/tech-pkg/2002/06/21/0009.html
>
> now, ${EXPORT_SYMBOLS_LDFLAGS} is used instead of directly defining
> -Wl,-export-dynamic which is set in appropriate defs.*.mk to reasonable
> values.  packages should be converted to this framework by:
>
> 1) replacing LDFLAGS+= -Wl,-export-dynamic and LIBS+= -export-dynamic with:
>
>    LDFLAGS+= ${EXPORT_SYMBOLS_LDFLAGS}
>
> 2) for use in patchfiles, add this variable to MAKE_ENV if needed:
>
>    MAKE_ENV+= EXPORT_SYMBOLS_LDFLAGS=${EXPORT_SYMBOLS_LDFLAGS}
>
> 3) replace occurances of -Wl,-export-dynamic and -export-dynamic in patch
>    files with:
>
>    $(EXPORT_SYMBOLS_LDFLAGS)


To generate a diff of this commit:
cvs rdiff -r1.6 -r1.6.4.1 pkgsrc/audio/arts/Makefile
cvs rdiff -r1.12 -r1.12.4.1 pkgsrc/audio/kdemultimedia2/Makefile
cvs rdiff -r1.3 -r1.3.4.1 pkgsrc/audio/kdemultimedia3/Makefile
cvs rdiff -r1.15 -r1.15.8.1 pkgsrc/graphics/dx/Makefile
cvs rdiff -r1.3 -r1.3.8.1 pkgsrc/graphics/gimp-base/Makefile
cvs rdiff -r1.8 -r1.8.8.1 pkgsrc/graphics/kdegraphics2/Makefile
cvs rdiff -r1.2 -r1.2.6.1 pkgsrc/graphics/kdegraphics3/Makefile
cvs rdiff -r1.10 -r1.10.8.1 pkgsrc/misc/kdeutils2/Makefile
cvs rdiff -r1.1 -r1.1.8.1 pkgsrc/misc/koffice/Makefile.common
cvs rdiff -r1.26 -r1.26.6.1 pkgsrc/mk/defs.Darwin.mk pkgsrc/mk/defs.NetBSD.mk \
    pkgsrc/mk/defs.SunOS.mk
cvs rdiff -r1.28 -r1.28.6.1 pkgsrc/mk/defs.Linux.mk
cvs rdiff -r1.21 -r1.21.8.1 pkgsrc/www/ap-php4/Makefile
cvs rdiff -r1.33 -r1.33.6.1 pkgsrc/www/galeon/Makefile
cvs rdiff -r1.3 -r1.3.4.1 pkgsrc/www/php3/Makefile.module
cvs rdiff -r1.23 -r1.23.8.1 pkgsrc/www/php4/Makefile
cvs rdiff -r1.15 -r1.15.8.1 pkgsrc/www/php4/Makefile.module
cvs rdiff -r1.8 -r1.8.8.1 pkgsrc/www/php4/distinfo
cvs rdiff -r1.7 -r1.7.8.1 pkgsrc/www/php4/patches/patch-aa

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.