Subject: PR/29031 CVS commit: pkgsrc/mk
To: None <pkg-manager@netbsd.org, gnats-admin@netbsd.org,>
From: Todd Vierling <tv@netbsd.org>
List: pkgsrc-bugs
Date: 03/18/2005 18:17:03
The following reply was made to PR pkg/29031; it has been noted by GNATS.

From: Todd Vierling <tv@netbsd.org>
To: gnats-bugs@netbsd.org
Cc: 
Subject: PR/29031 CVS commit: pkgsrc/mk
Date: Fri, 18 Mar 2005 18:16:35 +0000 (UTC)

 Module Name:	pkgsrc
 Committed By:	tv
 Date:		Fri Mar 18 18:16:35 UTC 2005
 
 Modified Files:
 	pkgsrc/mk: bsd.pkg.mk
 	pkgsrc/mk/platform: AIX.mk BSDOS.mk Darwin.mk DragonFly.mk FreeBSD.mk
 	    IRIX.mk Interix.mk Linux.mk NetBSD.mk OSF1.mk OpenBSD.mk SunOS.mk
 	    UnixWare.mk
 
 Log Message:
 Fix a gleaming illustration of why using YES/NO variables everywhere
 possible, just for the sake of doing so, is not a good thing to do:
 
 The platform files define _STRIPFLAG_* to determine whether to strip things.
 But since this is included in bsd.prefs.mk, ".if ..." checks cannot take
 things set in the Makefile into account.  So convert INSTALL_UNSTRIPPED=YES
 to a defined/undefined variable check in bsd.pkg.mk, and use the :D:U idiom
 in the _STRIPFLAG_* variables.
 
 This should fix PR pkg/28772 and PR pkg/29031.
 
 
 To generate a diff of this commit:
 cvs rdiff -r1.1599 -r1.1600 pkgsrc/mk/bsd.pkg.mk
 cvs rdiff -r1.11 -r1.12 pkgsrc/mk/platform/AIX.mk \
     pkgsrc/mk/platform/OpenBSD.mk
 cvs rdiff -r1.7 -r1.8 pkgsrc/mk/platform/BSDOS.mk \
     pkgsrc/mk/platform/Darwin.mk pkgsrc/mk/platform/FreeBSD.mk \
     pkgsrc/mk/platform/NetBSD.mk pkgsrc/mk/platform/UnixWare.mk
 cvs rdiff -r1.9 -r1.10 pkgsrc/mk/platform/DragonFly.mk \
     pkgsrc/mk/platform/IRIX.mk pkgsrc/mk/platform/Linux.mk \
     pkgsrc/mk/platform/SunOS.mk
 cvs rdiff -r1.24 -r1.25 pkgsrc/mk/platform/Interix.mk
 cvs rdiff -r1.2 -r1.3 pkgsrc/mk/platform/OSF1.mk
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.