On Sat, 10 Sep 2016, Philip Guenther wrote:
> Same diff as binutils, with same open question:
>
> > Maybe we should add ${INSTALL_STRIP} to the INSTALL_PROGRAM assignment
> > here?
Here's the combined diff that does that.
(Note to self: if you set DEBUG in /etc/mk.conf, INSTALL_STRIP isn't set)
ok?
Philip
Index: gnu/usr.bin/binutils-2.17/Makefile.bsd-wrapper
===================================================================
RCS file: /data/src/openbsd/src/gnu/usr.bin/binutils-2.17/Makefile.bsd-wrapper,v
retrieving revision 1.8
diff -u -p -r1.8 Makefile.bsd-wrapper
--- gnu/usr.bin/binutils-2.17/Makefile.bsd-wrapper 5 Jul 2013 21:29:51
-0000 1.8
+++ gnu/usr.bin/binutils-2.17/Makefile.bsd-wrapper 11 Sep 2016 01:54:10
-0000
@@ -102,7 +102,8 @@ install: maninstall
tooldir=${PREFIX} \
BSDSRCDIR=${BSDSRCDIR} \
INSTALL_MODULES='${INSTALL_MODULES}' \
- INSTALL_PROGRAM='install -c -S' \
+ INSTALL_PROGRAM='install -c -S ${INSTALL_STRIP} -o ${BINOWN} -g
${BINGRP} -m ${BINMODE}' \
+ INSTALL_DATA='install -c -o ${BINOWN} -g ${DOCGRP} -m
${NONBINMODE}' \
INSTALL_INFO_HOST_MODULES='${INSTALL_INFO_HOST_MODULES}' \
install install-info
Index: gnu/usr.bin/binutils/Makefile.bsd-wrapper
===================================================================
RCS file: /data/src/openbsd/src/gnu/usr.bin/binutils/Makefile.bsd-wrapper,v
retrieving revision 1.83
diff -u -p -r1.83 Makefile.bsd-wrapper
--- gnu/usr.bin/binutils/Makefile.bsd-wrapper 1 Jun 2015 17:36:19 -0000
1.83
+++ gnu/usr.bin/binutils/Makefile.bsd-wrapper 11 Sep 2016 01:54:15 -0000
@@ -81,7 +81,8 @@ install: maninstall
tooldir=${PREFIX} \
BSDSRCDIR=${BSDSRCDIR} \
INSTALL_MODULES='${INSTALL_MODULES}' \
- INSTALL_PROGRAM='install -c -S' \
+ INSTALL_PROGRAM='install -c -S ${INSTALL_STRIP} -o ${BINOWN} -g
${BINGRP} -m ${BINMODE}' \
+ INSTALL_DATA='install -c -o ${BINOWN} -g ${DOCGRP} -m
${NONBINMODE}' \
INSTALL_INFO_HOST_MODULES='${INSTALL_INFO_HOST_MODULES}' \
install install-info
