Re: install/51084: 'build.sh release' fails with MAKEVERBOSE >= 3

[dieter roelants <dieter.NetBSD%pandora.be@localhost>]

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

From: dieter roelants <dieter.NetBSD%pandora.be@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: install/51084: 'build.sh release' fails with MAKEVERBOSE >= 3
Date: Fri, 22 Apr 2016 18:56:08 +0200

 On Thu, 21 Apr 2016 22:45:00 +0000 (UTC)
 paul%whooppee.com@localhost wrote:
 
 > 	While almost certainly not a correct fix, the following patch to
 > 	sets.subr demonstrates the source/cause of the problem:
 > 
 > Index: sets.subr
 > ===================================================================
 > RCS file: /cvsroot/src/distrib/sets/sets.subr,v
 > retrieving revision 1.174
 > diff -u -p -r1.174 sets.subr
 > --- sets.subr   21 Feb 2016 04:16:36 -0000      1.174
 > +++ sets.subr   21 Apr 2016 08:21:30 -0000
 > @@ -648,6 +648,11 @@ print_set_lists()
 >  #
 >  arch_to_cpu()
 >  {
 > +       # set +x        # unfortunately, if this is needed, it won't help!
 > +       unset MAKEVERBOSE
 > +       unset MAKEFLAGS
 > +       exec 2> /dev/null       # just in case
 > +
 >  	MACHINE_ARCH=${1} ${MAKE} -B -f- all <<EOMAKE
 >  .include <bsd.own.mk>
 >  all:
 
 Is it incorrect because you unset the vars for the rest of the script
 or because you lose a tiny bit of debug info? If the former, what about:
 	MACHINE_ARCH=${1} ${MAKE} -B -f- all <<EOMAKE | fgrep -v 'echo '
 ? (or `| tail -n 1`, or run the whole function body in a subshell)
 
 kind regards
 dieter