diff options
author | tv <tv> | 2005-03-18 18:16:34 +0000 |
---|---|---|
committer | tv <tv> | 2005-03-18 18:16:34 +0000 |
commit | 749d63986abe33e030f5d6d7e47d2a2edd9ab4ff (patch) | |
tree | 5147a40e92074d16fee67bfab7670e3b69d7b28a /mk/platform/Interix.mk | |
parent | 9d4b14894ea6651e68b622108dda0b831c37289e (diff) | |
download | pkgsrc-749d63986abe33e030f5d6d7e47d2a2edd9ab4ff.tar.gz |
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.
Diffstat (limited to 'mk/platform/Interix.mk')
-rw-r--r-- | mk/platform/Interix.mk | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/mk/platform/Interix.mk b/mk/platform/Interix.mk index dfddf71a4d1..54facc62fb4 100644 --- a/mk/platform/Interix.mk +++ b/mk/platform/Interix.mk @@ -1,4 +1,4 @@ -# $NetBSD: Interix.mk,v 1.24 2005/02/04 09:19:41 jlam Exp $ +# $NetBSD: Interix.mk,v 1.25 2005/03/18 18:16:35 tv Exp $ # # Variable definitions for the Interix operating system. @@ -179,10 +179,8 @@ BUILD_DEPENDS+= ${USE_X11BASE:D${_XPKGWEDGE_DEPENDS}} _OPSYS_WHOLE_ARCHIVE_FLAG= -Wl,--whole-archive _OPSYS_NO_WHOLE_ARCHIVE_FLAG= -Wl,--no-whole-archive -.if (!defined(INSTALL_UNSTRIPPED) || empty(INSTALL_UNSTRIPPED:M[yY][eE][sS])) && !defined(DEBUG_FLAGS) -_STRIPFLAG_CC?= -s # cc(1) option to strip -_STRIPFLAG_INSTALL?= -s # install(1) option to strip -.endif +_STRIPFLAG_CC?= ${_INSTALL_UNSTRIPPED:D:U-s} # cc(1) option to strip +_STRIPFLAG_INSTALL?= ${_INSTALL_UNSTRIPPED:D:U-s} # install(1) option to strip DEFAULT_SERIAL_DEVICE?= /dev/tty00 SERIAL_DEVICES?= /dev/tty00 /dev/tty01 /dev/tty02 /dev/tty03 |