PR/29031 CVS commit: pkgsrc/mk

[Todd Vierling <tv%netbsd.org@localhost>]

The following reply was made to PR pkg/29031; it has been noted by GNATS.

From: Todd Vierling <tv%netbsd.org@localhost>
To: gnats-bugs%netbsd.org@localhost
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.