* [PATCH] libstdc++: Reduce output of 'make check' @ 2023-09-07 13:31 Jonathan Wakely 2023-09-08 2:15 ` Eric Gallager 2023-09-08 17:06 ` Jonathan Wakely 0 siblings, 2 replies; 6+ messages in thread From: Jonathan Wakely @ 2023-09-07 13:31 UTC (permalink / raw) To: libstdc++, gcc-patches Any objections to this change? -- >8 -- This removes the 39 lines of shell commands that get echoed when starting the testsuite. The fact that near the end of that output it prints `echo "WARNING: could not find \`runtest'" 1>&2; :;` makes it look like that warning is actually being shown the the user. Suppress echoing the recipe, so that users only see the actual output from the testsuite, not the makefile recipe as well. libstdc++-v3/ChangeLog: * testsuite/Makefile.am (check-DEJAGNU): Use @ in recipe. * testsuite/Makefile.in: Regenerate. --- libstdc++-v3/testsuite/Makefile.am | 2 +- libstdc++-v3/testsuite/Makefile.in | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libstdc++-v3/testsuite/Makefile.am b/libstdc++-v3/testsuite/Makefile.am index 7adc5318192..4cee585fd8e 100644 --- a/libstdc++-v3/testsuite/Makefile.am +++ b/libstdc++-v3/testsuite/Makefile.am @@ -117,7 +117,7 @@ $(check_DEJAGNU_normal_targets): check-DEJAGNUnormal%: normal%/site.exp # Run the testsuite in normal mode. check-DEJAGNU $(check_DEJAGNU_normal_targets): check-DEJAGNU%: site.exp - $(if $*,@)AR="$(AR)"; export AR; \ + @$(if $*,@)AR="$(AR)"; export AR; \ RANLIB="$(RANLIB)"; export RANLIB; \ if [ -z "$*" ] && [ -n "$(filter -j%, $(MFLAGS))" ]; then \ rm -rf normal-parallel || true; \ ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] libstdc++: Reduce output of 'make check' 2023-09-07 13:31 [PATCH] libstdc++: Reduce output of 'make check' Jonathan Wakely @ 2023-09-08 2:15 ` Eric Gallager 2023-09-08 10:30 ` Jonathan Wakely 2023-09-08 17:06 ` Jonathan Wakely 1 sibling, 1 reply; 6+ messages in thread From: Eric Gallager @ 2023-09-08 2:15 UTC (permalink / raw) To: Jonathan Wakely; +Cc: libstdc++, gcc-patches Maybe use $(AM_V_at) instead? That would allow it to be controlled by the --enable-silent-rules flag to configure, as well as make V=1 vs. make V=0 too. On Thu, Sep 7, 2023 at 9:32 AM Jonathan Wakely via Gcc-patches <gcc-patches@gcc.gnu.org> wrote: > > Any objections to this change? > > -- >8 -- > > This removes the 39 lines of shell commands that get echoed when > starting the testsuite. The fact that near the end of that output it > prints `echo "WARNING: could not find \`runtest'" 1>&2; :;` makes it > look like that warning is actually being shown the the user. > > Suppress echoing the recipe, so that users only see the actual output > from the testsuite, not the makefile recipe as well. > > libstdc++-v3/ChangeLog: > > * testsuite/Makefile.am (check-DEJAGNU): Use @ in recipe. > * testsuite/Makefile.in: Regenerate. > --- > libstdc++-v3/testsuite/Makefile.am | 2 +- > libstdc++-v3/testsuite/Makefile.in | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libstdc++-v3/testsuite/Makefile.am b/libstdc++-v3/testsuite/Makefile.am > index 7adc5318192..4cee585fd8e 100644 > --- a/libstdc++-v3/testsuite/Makefile.am > +++ b/libstdc++-v3/testsuite/Makefile.am > @@ -117,7 +117,7 @@ $(check_DEJAGNU_normal_targets): check-DEJAGNUnormal%: normal%/site.exp > > # Run the testsuite in normal mode. > check-DEJAGNU $(check_DEJAGNU_normal_targets): check-DEJAGNU%: site.exp > - $(if $*,@)AR="$(AR)"; export AR; \ > + @$(if $*,@)AR="$(AR)"; export AR; \ > RANLIB="$(RANLIB)"; export RANLIB; \ > if [ -z "$*" ] && [ -n "$(filter -j%, $(MFLAGS))" ]; then \ > rm -rf normal-parallel || true; \ > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] libstdc++: Reduce output of 'make check' 2023-09-08 2:15 ` Eric Gallager @ 2023-09-08 10:30 ` Jonathan Wakely 2023-09-08 12:14 ` Eric Gallager 0 siblings, 1 reply; 6+ messages in thread From: Jonathan Wakely @ 2023-09-08 10:30 UTC (permalink / raw) To: Eric Gallager; +Cc: libstdc++, gcc-patches On Fri, 8 Sept 2023 at 03:16, Eric Gallager <egall@gwmail.gwu.edu> wrote: > > Maybe use $(AM_V_at) instead? That would allow it to be controlled by > the --enable-silent-rules flag to configure, as well as make V=1 vs. > make V=0 too. I see two problems with that. Firstly, nobody uses that configure option, so the net result would be that these 39 lines are still printed for everybody. Secondly, why would anybody ever want this output? Having it configurable (and default to noisy) just doesn't seem helpful. This is the output I'm talking about: make: Entering directory '/home/jwakely/src/gcc/build/x86_64-pc-linux-gnu/libstdc++-v3/testsuite' make check-DEJAGNU make[1]: Entering directory '/home/jwakely/src/gcc/build/x86_64-pc-linux-gnu/libstdc++-v3/testsuite' Making a new site.exp file... AR="ar --plugin /home/jwakely/src/gcc/build/./gcc/liblto_plugin.so"; export AR; \ RANLIB="ranlib --plugin /home/jwakely/src/gcc/build/./gcc/liblto_plugin.so"; export RANLIB; \ if [ -z "" ] && [ -n "" ]; then \ rm -rf normal-parallel || true; \ mkdir normal-parallel; \ make check-DEJAGNUnormal1 check-DEJAGNUnormal2 check-DEJAGNUnormal3 check-DEJAGNUnormal4 check-DEJAGNUnormal5 check-DEJAGNUnormal6 check-DEJAGNUnormal7 c heck-DEJAGNUnormal8 check-DEJAGNUnormal9 check-DEJAGNUnormal10 check-DEJAGNUnormal11 check-DEJAGNUnormal12 check-DEJAGNUnormal13 check-DEJAGNUnormal14 check -DEJAGNUnormal15 check-DEJAGNUnormal16 check-DEJAGNUnormal17 check-DEJAGNUnormal18 check-DEJAGNUnormal19 check-DEJAGNUnormal20 check-DEJAGNUnormal21 check-D EJAGNUnormal22 check-DEJAGNUnormal23 check-DEJAGNUnormal24 check-DEJAGNUnormal25 check-DEJAGNUnormal26 check-DEJAGNUnormal27 check-DEJAGNUnormal28 check-DEJ AGNUnormal29 check-DEJAGNUnormal30 check-DEJAGNUnormal31 check-DEJAGNUnormal32 check-DEJAGNUnormal33 check-DEJAGNUnormal34 check-DEJAGNUnormal35 check-DEJAG NUnormal36 check-DEJAGNUnormal37 check-DEJAGNUnormal38 check-DEJAGNUnormal39 check-DEJAGNUnormal40 check-DEJAGNUnormal41 check-DEJAGNUnormal42 check-DEJAGNU normal43 check-DEJAGNUnormal44 check-DEJAGNUnormal45 check-DEJAGNUnormal46 check-DEJAGNUnormal47 check-DEJAGNUnormal48 check-DEJAGNUnormal49 check-DEJAGNUno rmal50 check-DEJAGNUnormal51 check-DEJAGNUnormal52 check-DEJAGNUnormal53 check-DEJAGNUnormal54 check-DEJAGNUnormal55 check-DEJAGNUnormal56 check-DEJAGNUnorm al57 check-DEJAGNUnormal58 check-DEJAGNUnormal59 check-DEJAGNUnormal60 check-DEJAGNUnormal61 check-DEJAGNUnormal62 check-DEJAGNUnormal63 check-DEJAGNUnormal 64 check-DEJAGNUnormal65 check-DEJAGNUnormal66 check-DEJAGNUnormal67 check-DEJAGNUnormal68 check-DEJAGNUnormal69 check-DEJAGNUnormal70 check-DEJAGNUnormal71 check-DEJAGNUnormal72 check-DEJAGNUnormal73 check-DEJAGNUnormal74 check-DEJAGNUnormal75 check-DEJAGNUnormal76 check-DEJAGNUnormal77 check-DEJAGNUnormal78 c heck-DEJAGNUnormal79 check-DEJAGNUnormal80 check-DEJAGNUnormal81 check-DEJAGNUnormal82 check-DEJAGNUnormal83 check-DEJAGNUnormal84 check-DEJAGNUnormal85 che ck-DEJAGNUnormal86 check-DEJAGNUnormal87 check-DEJAGNUnormal88 check-DEJAGNUnormal89 check-DEJAGNUnormal90 check-DEJAGNUnormal91 check-DEJAGNUnormal92 check -DEJAGNUnormal93 check-DEJAGNUnormal94 check-DEJAGNUnormal95 check-DEJAGNUnormal96 check-DEJAGNUnormal97 check-DEJAGNUnormal98 check-DEJAGNUnormal99 check-D EJAGNUnormal100 check-DEJAGNUnormal101 check-DEJAGNUnormal102 check-DEJAGNUnormal103 check-DEJAGNUnormal104 check-DEJAGNUnormal105 check-DEJAGNUnormal106 ch eck-DEJAGNUnormal107 check-DEJAGNUnormal108 check-DEJAGNUnormal109 check-DEJAGNUnormal110 check-DEJAGNUnormal111 check-DEJAGNUnormal112 check-DEJAGNUnormal1 13 check-DEJAGNUnormal114 check-DEJAGNUnormal115 check-DEJAGNUnormal116 check-DEJAGNUnormal117 check-DEJAGNUnormal118 check-DEJAGNUnormal119 check-DEJAGNUno rmal120 check-DEJAGNUnormal121 check-DEJAGNUnormal122 check-DEJAGNUnormal123 check-DEJAGNUnormal124 check-DEJAGNUnormal125 check-DEJAGNUnormal126 check-DEJA GNUnormal127 check-DEJAGNUnormal128; \ rm -rf normal-parallel || true; \ for idx in 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128; do \ if [ -d normal$idx ]; then \ mv -f normal$idx/libstdc++.sum normal$idx/libstdc++.sum.sep; \ mv -f normal$idx/libstdc++.log normal$idx/libstdc++.log.sep; \ fi; \ done; \ /bin/sh /home/jwakely/src/gcc/gcc/libstdc++-v3/testsuite/../../contrib/dg-extract-results.sh \ normal[0-9]*/libstdc++.sum.sep > libstdc++.sum; \ /bin/sh /home/jwakely/src/gcc/gcc/libstdc++-v3/testsuite/../../contrib/dg-extract-results.sh -L \ normal[0-9]*/libstdc++.log.sep > libstdc++.log; \ exit 0; \ fi; \ srcdir=`CDPATH="${ZSH_VERSION+.}:" && cd /home/jwakely/src/gcc/gcc/libstdc++-v3/testsuite && pwd`; export srcdir; \ EXPECT=expect; export EXPECT; \ runtest=; \ if [ -z "$runtest" ]; then runtest=runtest; fi; \ tool=libstdc++; \ if [ -n "" ]; then \ if [ -f normal-parallel/finished ]; then rm -rf ""; exit 0; fi; \ GCC_RUNTEST_PARALLELIZE_DIR=`${PWDCMD-pwd}`/normal-parallel; \ export GCC_RUNTEST_PARALLELIZE_DIR; \ cd ""; \ fi; \ if /bin/sh -c "$runtest --version" > /dev/null 2>&1; then \ $runtest --tool $tool --srcdir $srcdir \ ; \ if [ -n "" ]; then \ touch $GCC_RUNTEST_PARALLELIZE_DIR/finished; \ fi; \ else \ echo "WARNING: could not find \`runtest'" 1>&2; :;\ fi WARNING: Couldn't find the global config file. dejagnu - timeout default set to 90s Using /home/jwakely/src/gcc/gcc/libstdc++-v3/testsuite/lib/libstdc++.exp as tool init file. Test run by jwakely on Fri Sep 8 11:27:44 2023 Native configuration is x86_64-pc-linux-gnu === libstdc++ tests === Everything from the AR= line to the echo "WARNING" line and the "fi" after it would be suppressed with this patch. I don't think making it configurable (and default to still being printed) is useful. > > On Thu, Sep 7, 2023 at 9:32 AM Jonathan Wakely via Gcc-patches > <gcc-patches@gcc.gnu.org> wrote: > > > > Any objections to this change? > > > > -- >8 -- > > > > This removes the 39 lines of shell commands that get echoed when > > starting the testsuite. The fact that near the end of that output it > > prints `echo "WARNING: could not find \`runtest'" 1>&2; :;` makes it > > look like that warning is actually being shown the the user. > > > > Suppress echoing the recipe, so that users only see the actual output > > from the testsuite, not the makefile recipe as well. > > > > libstdc++-v3/ChangeLog: > > > > * testsuite/Makefile.am (check-DEJAGNU): Use @ in recipe. > > * testsuite/Makefile.in: Regenerate. > > --- > > libstdc++-v3/testsuite/Makefile.am | 2 +- > > libstdc++-v3/testsuite/Makefile.in | 2 +- > > 2 files changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/libstdc++-v3/testsuite/Makefile.am b/libstdc++-v3/testsuite/Makefile.am > > index 7adc5318192..4cee585fd8e 100644 > > --- a/libstdc++-v3/testsuite/Makefile.am > > +++ b/libstdc++-v3/testsuite/Makefile.am > > @@ -117,7 +117,7 @@ $(check_DEJAGNU_normal_targets): check-DEJAGNUnormal%: normal%/site.exp > > > > # Run the testsuite in normal mode. > > check-DEJAGNU $(check_DEJAGNU_normal_targets): check-DEJAGNU%: site.exp > > - $(if $*,@)AR="$(AR)"; export AR; \ > > + @$(if $*,@)AR="$(AR)"; export AR; \ > > RANLIB="$(RANLIB)"; export RANLIB; \ > > if [ -z "$*" ] && [ -n "$(filter -j%, $(MFLAGS))" ]; then \ > > rm -rf normal-parallel || true; \ > > > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] libstdc++: Reduce output of 'make check' 2023-09-08 10:30 ` Jonathan Wakely @ 2023-09-08 12:14 ` Eric Gallager 2023-09-08 12:18 ` Jonathan Wakely 0 siblings, 1 reply; 6+ messages in thread From: Eric Gallager @ 2023-09-08 12:14 UTC (permalink / raw) To: Jonathan Wakely; +Cc: libstdc++, gcc-patches On Fri, Sep 8, 2023 at 6:32 AM Jonathan Wakely <jwakely@redhat.com> wrote: > > On Fri, 8 Sept 2023 at 03:16, Eric Gallager <egall@gwmail.gwu.edu> wrote: > > > > Maybe use $(AM_V_at) instead? That would allow it to be controlled by > > the --enable-silent-rules flag to configure, as well as make V=1 vs. > > make V=0 too. > > I see two problems with that. Firstly, nobody uses that configure > option, so the net result would be that these 39 lines are still > printed for everybody. > > Secondly, why would anybody ever want this output? Having it > configurable (and default to noisy) just doesn't seem helpful. > > This is the output I'm talking about: > > make: Entering directory > '/home/jwakely/src/gcc/build/x86_64-pc-linux-gnu/libstdc++-v3/testsuite' > make check-DEJAGNU > make[1]: Entering directory > '/home/jwakely/src/gcc/build/x86_64-pc-linux-gnu/libstdc++-v3/testsuite' > Making a new site.exp file... > AR="ar --plugin /home/jwakely/src/gcc/build/./gcc/liblto_plugin.so"; > export AR; \ > RANLIB="ranlib --plugin > /home/jwakely/src/gcc/build/./gcc/liblto_plugin.so"; export RANLIB; \ > if [ -z "" ] && [ -n "" ]; then \ > rm -rf normal-parallel || true; \ > mkdir normal-parallel; \ > make check-DEJAGNUnormal1 check-DEJAGNUnormal2 check-DEJAGNUnormal3 > check-DEJAGNUnormal4 check-DEJAGNUnormal5 check-DEJAGNUnormal6 > check-DEJAGNUnormal7 c > heck-DEJAGNUnormal8 check-DEJAGNUnormal9 check-DEJAGNUnormal10 > check-DEJAGNUnormal11 check-DEJAGNUnormal12 check-DEJAGNUnormal13 > check-DEJAGNUnormal14 check > -DEJAGNUnormal15 check-DEJAGNUnormal16 check-DEJAGNUnormal17 > check-DEJAGNUnormal18 check-DEJAGNUnormal19 check-DEJAGNUnormal20 > check-DEJAGNUnormal21 check-D > EJAGNUnormal22 check-DEJAGNUnormal23 check-DEJAGNUnormal24 > check-DEJAGNUnormal25 check-DEJAGNUnormal26 check-DEJAGNUnormal27 > check-DEJAGNUnormal28 check-DEJ > AGNUnormal29 check-DEJAGNUnormal30 check-DEJAGNUnormal31 > check-DEJAGNUnormal32 check-DEJAGNUnormal33 check-DEJAGNUnormal34 > check-DEJAGNUnormal35 check-DEJAG > NUnormal36 check-DEJAGNUnormal37 check-DEJAGNUnormal38 > check-DEJAGNUnormal39 check-DEJAGNUnormal40 check-DEJAGNUnormal41 > check-DEJAGNUnormal42 check-DEJAGNU > normal43 check-DEJAGNUnormal44 check-DEJAGNUnormal45 > check-DEJAGNUnormal46 check-DEJAGNUnormal47 check-DEJAGNUnormal48 > check-DEJAGNUnormal49 check-DEJAGNUno > rmal50 check-DEJAGNUnormal51 check-DEJAGNUnormal52 > check-DEJAGNUnormal53 check-DEJAGNUnormal54 check-DEJAGNUnormal55 > check-DEJAGNUnormal56 check-DEJAGNUnorm > al57 check-DEJAGNUnormal58 check-DEJAGNUnormal59 check-DEJAGNUnormal60 > check-DEJAGNUnormal61 check-DEJAGNUnormal62 check-DEJAGNUnormal63 > check-DEJAGNUnormal > 64 check-DEJAGNUnormal65 check-DEJAGNUnormal66 check-DEJAGNUnormal67 > check-DEJAGNUnormal68 check-DEJAGNUnormal69 check-DEJAGNUnormal70 > check-DEJAGNUnormal71 > check-DEJAGNUnormal72 check-DEJAGNUnormal73 check-DEJAGNUnormal74 > check-DEJAGNUnormal75 check-DEJAGNUnormal76 check-DEJAGNUnormal77 > check-DEJAGNUnormal78 c > heck-DEJAGNUnormal79 check-DEJAGNUnormal80 check-DEJAGNUnormal81 > check-DEJAGNUnormal82 check-DEJAGNUnormal83 check-DEJAGNUnormal84 > check-DEJAGNUnormal85 che > ck-DEJAGNUnormal86 check-DEJAGNUnormal87 check-DEJAGNUnormal88 > check-DEJAGNUnormal89 check-DEJAGNUnormal90 check-DEJAGNUnormal91 > check-DEJAGNUnormal92 check > -DEJAGNUnormal93 check-DEJAGNUnormal94 check-DEJAGNUnormal95 > check-DEJAGNUnormal96 check-DEJAGNUnormal97 check-DEJAGNUnormal98 > check-DEJAGNUnormal99 check-D > EJAGNUnormal100 check-DEJAGNUnormal101 check-DEJAGNUnormal102 > check-DEJAGNUnormal103 check-DEJAGNUnormal104 check-DEJAGNUnormal105 > check-DEJAGNUnormal106 ch > eck-DEJAGNUnormal107 check-DEJAGNUnormal108 check-DEJAGNUnormal109 > check-DEJAGNUnormal110 check-DEJAGNUnormal111 check-DEJAGNUnormal112 > check-DEJAGNUnormal1 > 13 check-DEJAGNUnormal114 check-DEJAGNUnormal115 > check-DEJAGNUnormal116 check-DEJAGNUnormal117 check-DEJAGNUnormal118 > check-DEJAGNUnormal119 check-DEJAGNUno > rmal120 check-DEJAGNUnormal121 check-DEJAGNUnormal122 > check-DEJAGNUnormal123 check-DEJAGNUnormal124 check-DEJAGNUnormal125 > check-DEJAGNUnormal126 check-DEJA > GNUnormal127 check-DEJAGNUnormal128; \ > rm -rf normal-parallel || true; \ > for idx in 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 > 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 > 46 47 48 49 50 51 > 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 > 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 > 98 99 100 101 102 > 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 > 120 121 122 123 124 125 126 127 128; do \ > if [ -d normal$idx ]; then \ > mv -f normal$idx/libstdc++.sum normal$idx/libstdc++.sum.sep; \ > mv -f normal$idx/libstdc++.log normal$idx/libstdc++.log.sep; \ > fi; \ > done; \ > /bin/sh /home/jwakely/src/gcc/gcc/libstdc++-v3/testsuite/../../contrib/dg-extract-results.sh > \ > normal[0-9]*/libstdc++.sum.sep > libstdc++.sum; \ > /bin/sh /home/jwakely/src/gcc/gcc/libstdc++-v3/testsuite/../../contrib/dg-extract-results.sh > -L \ > normal[0-9]*/libstdc++.log.sep > libstdc++.log; \ > exit 0; \ > fi; \ > srcdir=`CDPATH="${ZSH_VERSION+.}:" && cd > /home/jwakely/src/gcc/gcc/libstdc++-v3/testsuite && pwd`; export > srcdir; \ > EXPECT=expect; export EXPECT; \ > runtest=; \ > if [ -z "$runtest" ]; then runtest=runtest; fi; \ > tool=libstdc++; \ > if [ -n "" ]; then \ > if [ -f normal-parallel/finished ]; then rm -rf ""; exit 0; fi; \ > GCC_RUNTEST_PARALLELIZE_DIR=`${PWDCMD-pwd}`/normal-parallel; \ > export GCC_RUNTEST_PARALLELIZE_DIR; \ > cd ""; \ > fi; \ > if /bin/sh -c "$runtest --version" > /dev/null 2>&1; then \ > $runtest --tool $tool --srcdir $srcdir \ > ; \ > if [ -n "" ]; then \ > touch $GCC_RUNTEST_PARALLELIZE_DIR/finished; \ > fi; \ > else \ > echo "WARNING: could not find \`runtest'" 1>&2; :;\ > fi > WARNING: Couldn't find the global config file. > dejagnu - timeout default set to 90s > Using /home/jwakely/src/gcc/gcc/libstdc++-v3/testsuite/lib/libstdc++.exp > as tool init file. > Test run by jwakely on Fri Sep 8 11:27:44 2023 > Native configuration is x86_64-pc-linux-gnu > > === libstdc++ tests === > > > Everything from the AR= line to the echo "WARNING" line and the "fi" > after it would be suppressed with this patch. I don't think making it > configurable (and default to still being printed) is useful. > You can change the default by sticking AM_SILENT_RULES([yes]) in configure.ac; see the "Silencing `make`" portion of the automake documentation: https://www.gnu.org/software/automake/manual/html_node/Automake-Silent-Rules.html > > > > > > On Thu, Sep 7, 2023 at 9:32 AM Jonathan Wakely via Gcc-patches > > <gcc-patches@gcc.gnu.org> wrote: > > > > > > Any objections to this change? > > > > > > -- >8 -- > > > > > > This removes the 39 lines of shell commands that get echoed when > > > starting the testsuite. The fact that near the end of that output it > > > prints `echo "WARNING: could not find \`runtest'" 1>&2; :;` makes it > > > look like that warning is actually being shown the the user. > > > > > > Suppress echoing the recipe, so that users only see the actual output > > > from the testsuite, not the makefile recipe as well. > > > > > > libstdc++-v3/ChangeLog: > > > > > > * testsuite/Makefile.am (check-DEJAGNU): Use @ in recipe. > > > * testsuite/Makefile.in: Regenerate. > > > --- > > > libstdc++-v3/testsuite/Makefile.am | 2 +- > > > libstdc++-v3/testsuite/Makefile.in | 2 +- > > > 2 files changed, 2 insertions(+), 2 deletions(-) > > > > > > diff --git a/libstdc++-v3/testsuite/Makefile.am b/libstdc++-v3/testsuite/Makefile.am > > > index 7adc5318192..4cee585fd8e 100644 > > > --- a/libstdc++-v3/testsuite/Makefile.am > > > +++ b/libstdc++-v3/testsuite/Makefile.am > > > @@ -117,7 +117,7 @@ $(check_DEJAGNU_normal_targets): check-DEJAGNUnormal%: normal%/site.exp > > > > > > # Run the testsuite in normal mode. > > > check-DEJAGNU $(check_DEJAGNU_normal_targets): check-DEJAGNU%: site.exp > > > - $(if $*,@)AR="$(AR)"; export AR; \ > > > + @$(if $*,@)AR="$(AR)"; export AR; \ > > > RANLIB="$(RANLIB)"; export RANLIB; \ > > > if [ -z "$*" ] && [ -n "$(filter -j%, $(MFLAGS))" ]; then \ > > > rm -rf normal-parallel || true; \ > > > > > > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] libstdc++: Reduce output of 'make check' 2023-09-08 12:14 ` Eric Gallager @ 2023-09-08 12:18 ` Jonathan Wakely 0 siblings, 0 replies; 6+ messages in thread From: Jonathan Wakely @ 2023-09-08 12:18 UTC (permalink / raw) To: Eric Gallager; +Cc: libstdc++, gcc-patches On Fri, 8 Sept 2023 at 13:15, Eric Gallager <egall@gwmail.gwu.edu> wrote: > > On Fri, Sep 8, 2023 at 6:32 AM Jonathan Wakely <jwakely@redhat.com> wrote: > > > > On Fri, 8 Sept 2023 at 03:16, Eric Gallager <egall@gwmail.gwu.edu> wrote: > > > > > > Maybe use $(AM_V_at) instead? That would allow it to be controlled by > > > the --enable-silent-rules flag to configure, as well as make V=1 vs. > > > make V=0 too. > > > > I see two problems with that. Firstly, nobody uses that configure > > option, so the net result would be that these 39 lines are still > > printed for everybody. > > > > Secondly, why would anybody ever want this output? Having it > > configurable (and default to noisy) just doesn't seem helpful. > > > > This is the output I'm talking about: > > > > make: Entering directory > > '/home/jwakely/src/gcc/build/x86_64-pc-linux-gnu/libstdc++-v3/testsuite' > > make check-DEJAGNU > > make[1]: Entering directory > > '/home/jwakely/src/gcc/build/x86_64-pc-linux-gnu/libstdc++-v3/testsuite' > > Making a new site.exp file... > > AR="ar --plugin /home/jwakely/src/gcc/build/./gcc/liblto_plugin.so"; > > export AR; \ > > RANLIB="ranlib --plugin > > /home/jwakely/src/gcc/build/./gcc/liblto_plugin.so"; export RANLIB; \ > > if [ -z "" ] && [ -n "" ]; then \ > > rm -rf normal-parallel || true; \ > > mkdir normal-parallel; \ > > make check-DEJAGNUnormal1 check-DEJAGNUnormal2 check-DEJAGNUnormal3 > > check-DEJAGNUnormal4 check-DEJAGNUnormal5 check-DEJAGNUnormal6 > > check-DEJAGNUnormal7 c > > heck-DEJAGNUnormal8 check-DEJAGNUnormal9 check-DEJAGNUnormal10 > > check-DEJAGNUnormal11 check-DEJAGNUnormal12 check-DEJAGNUnormal13 > > check-DEJAGNUnormal14 check > > -DEJAGNUnormal15 check-DEJAGNUnormal16 check-DEJAGNUnormal17 > > check-DEJAGNUnormal18 check-DEJAGNUnormal19 check-DEJAGNUnormal20 > > check-DEJAGNUnormal21 check-D > > EJAGNUnormal22 check-DEJAGNUnormal23 check-DEJAGNUnormal24 > > check-DEJAGNUnormal25 check-DEJAGNUnormal26 check-DEJAGNUnormal27 > > check-DEJAGNUnormal28 check-DEJ > > AGNUnormal29 check-DEJAGNUnormal30 check-DEJAGNUnormal31 > > check-DEJAGNUnormal32 check-DEJAGNUnormal33 check-DEJAGNUnormal34 > > check-DEJAGNUnormal35 check-DEJAG > > NUnormal36 check-DEJAGNUnormal37 check-DEJAGNUnormal38 > > check-DEJAGNUnormal39 check-DEJAGNUnormal40 check-DEJAGNUnormal41 > > check-DEJAGNUnormal42 check-DEJAGNU > > normal43 check-DEJAGNUnormal44 check-DEJAGNUnormal45 > > check-DEJAGNUnormal46 check-DEJAGNUnormal47 check-DEJAGNUnormal48 > > check-DEJAGNUnormal49 check-DEJAGNUno > > rmal50 check-DEJAGNUnormal51 check-DEJAGNUnormal52 > > check-DEJAGNUnormal53 check-DEJAGNUnormal54 check-DEJAGNUnormal55 > > check-DEJAGNUnormal56 check-DEJAGNUnorm > > al57 check-DEJAGNUnormal58 check-DEJAGNUnormal59 check-DEJAGNUnormal60 > > check-DEJAGNUnormal61 check-DEJAGNUnormal62 check-DEJAGNUnormal63 > > check-DEJAGNUnormal > > 64 check-DEJAGNUnormal65 check-DEJAGNUnormal66 check-DEJAGNUnormal67 > > check-DEJAGNUnormal68 check-DEJAGNUnormal69 check-DEJAGNUnormal70 > > check-DEJAGNUnormal71 > > check-DEJAGNUnormal72 check-DEJAGNUnormal73 check-DEJAGNUnormal74 > > check-DEJAGNUnormal75 check-DEJAGNUnormal76 check-DEJAGNUnormal77 > > check-DEJAGNUnormal78 c > > heck-DEJAGNUnormal79 check-DEJAGNUnormal80 check-DEJAGNUnormal81 > > check-DEJAGNUnormal82 check-DEJAGNUnormal83 check-DEJAGNUnormal84 > > check-DEJAGNUnormal85 che > > ck-DEJAGNUnormal86 check-DEJAGNUnormal87 check-DEJAGNUnormal88 > > check-DEJAGNUnormal89 check-DEJAGNUnormal90 check-DEJAGNUnormal91 > > check-DEJAGNUnormal92 check > > -DEJAGNUnormal93 check-DEJAGNUnormal94 check-DEJAGNUnormal95 > > check-DEJAGNUnormal96 check-DEJAGNUnormal97 check-DEJAGNUnormal98 > > check-DEJAGNUnormal99 check-D > > EJAGNUnormal100 check-DEJAGNUnormal101 check-DEJAGNUnormal102 > > check-DEJAGNUnormal103 check-DEJAGNUnormal104 check-DEJAGNUnormal105 > > check-DEJAGNUnormal106 ch > > eck-DEJAGNUnormal107 check-DEJAGNUnormal108 check-DEJAGNUnormal109 > > check-DEJAGNUnormal110 check-DEJAGNUnormal111 check-DEJAGNUnormal112 > > check-DEJAGNUnormal1 > > 13 check-DEJAGNUnormal114 check-DEJAGNUnormal115 > > check-DEJAGNUnormal116 check-DEJAGNUnormal117 check-DEJAGNUnormal118 > > check-DEJAGNUnormal119 check-DEJAGNUno > > rmal120 check-DEJAGNUnormal121 check-DEJAGNUnormal122 > > check-DEJAGNUnormal123 check-DEJAGNUnormal124 check-DEJAGNUnormal125 > > check-DEJAGNUnormal126 check-DEJA > > GNUnormal127 check-DEJAGNUnormal128; \ > > rm -rf normal-parallel || true; \ > > for idx in 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 > > 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 > > 46 47 48 49 50 51 > > 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 > > 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 > > 98 99 100 101 102 > > 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 > > 120 121 122 123 124 125 126 127 128; do \ > > if [ -d normal$idx ]; then \ > > mv -f normal$idx/libstdc++.sum normal$idx/libstdc++.sum.sep; \ > > mv -f normal$idx/libstdc++.log normal$idx/libstdc++.log.sep; \ > > fi; \ > > done; \ > > /bin/sh /home/jwakely/src/gcc/gcc/libstdc++-v3/testsuite/../../contrib/dg-extract-results.sh > > \ > > normal[0-9]*/libstdc++.sum.sep > libstdc++.sum; \ > > /bin/sh /home/jwakely/src/gcc/gcc/libstdc++-v3/testsuite/../../contrib/dg-extract-results.sh > > -L \ > > normal[0-9]*/libstdc++.log.sep > libstdc++.log; \ > > exit 0; \ > > fi; \ > > srcdir=`CDPATH="${ZSH_VERSION+.}:" && cd > > /home/jwakely/src/gcc/gcc/libstdc++-v3/testsuite && pwd`; export > > srcdir; \ > > EXPECT=expect; export EXPECT; \ > > runtest=; \ > > if [ -z "$runtest" ]; then runtest=runtest; fi; \ > > tool=libstdc++; \ > > if [ -n "" ]; then \ > > if [ -f normal-parallel/finished ]; then rm -rf ""; exit 0; fi; \ > > GCC_RUNTEST_PARALLELIZE_DIR=`${PWDCMD-pwd}`/normal-parallel; \ > > export GCC_RUNTEST_PARALLELIZE_DIR; \ > > cd ""; \ > > fi; \ > > if /bin/sh -c "$runtest --version" > /dev/null 2>&1; then \ > > $runtest --tool $tool --srcdir $srcdir \ > > ; \ > > if [ -n "" ]; then \ > > touch $GCC_RUNTEST_PARALLELIZE_DIR/finished; \ > > fi; \ > > else \ > > echo "WARNING: could not find \`runtest'" 1>&2; :;\ > > fi > > WARNING: Couldn't find the global config file. > > dejagnu - timeout default set to 90s > > Using /home/jwakely/src/gcc/gcc/libstdc++-v3/testsuite/lib/libstdc++.exp > > as tool init file. > > Test run by jwakely on Fri Sep 8 11:27:44 2023 > > Native configuration is x86_64-pc-linux-gnu > > > > === libstdc++ tests === > > > > > > Everything from the AR= line to the echo "WARNING" line and the "fi" > > after it would be suppressed with this patch. I don't think making it > > configurable (and default to still being printed) is useful. > > > > You can change the default by sticking AM_SILENT_RULES([yes]) in > configure.ac; see the "Silencing `make`" portion of the automake > documentation: > https://www.gnu.org/software/automake/manual/html_node/Automake-Silent-Rules.html That doesn't address the fact that the output is never useful. I don't want to suppress *all* build output for libstdc++, just those lines above. And if you try it you'll see --enable-silent-rules doesn't really help much for libstdc++, as only half the rules are silenced. All the libtool compile and libtool link lines are still printed in full, even with V=0. So more work would be needed, and it still wouldn't make those lines from the testsuite useful to anybody, ever. > > > > > > > > > > > On Thu, Sep 7, 2023 at 9:32 AM Jonathan Wakely via Gcc-patches > > > <gcc-patches@gcc.gnu.org> wrote: > > > > > > > > Any objections to this change? > > > > > > > > -- >8 -- > > > > > > > > This removes the 39 lines of shell commands that get echoed when > > > > starting the testsuite. The fact that near the end of that output it > > > > prints `echo "WARNING: could not find \`runtest'" 1>&2; :;` makes it > > > > look like that warning is actually being shown the the user. > > > > > > > > Suppress echoing the recipe, so that users only see the actual output > > > > from the testsuite, not the makefile recipe as well. > > > > > > > > libstdc++-v3/ChangeLog: > > > > > > > > * testsuite/Makefile.am (check-DEJAGNU): Use @ in recipe. > > > > * testsuite/Makefile.in: Regenerate. > > > > --- > > > > libstdc++-v3/testsuite/Makefile.am | 2 +- > > > > libstdc++-v3/testsuite/Makefile.in | 2 +- > > > > 2 files changed, 2 insertions(+), 2 deletions(-) > > > > > > > > diff --git a/libstdc++-v3/testsuite/Makefile.am b/libstdc++-v3/testsuite/Makefile.am > > > > index 7adc5318192..4cee585fd8e 100644 > > > > --- a/libstdc++-v3/testsuite/Makefile.am > > > > +++ b/libstdc++-v3/testsuite/Makefile.am > > > > @@ -117,7 +117,7 @@ $(check_DEJAGNU_normal_targets): check-DEJAGNUnormal%: normal%/site.exp > > > > > > > > # Run the testsuite in normal mode. > > > > check-DEJAGNU $(check_DEJAGNU_normal_targets): check-DEJAGNU%: site.exp > > > > - $(if $*,@)AR="$(AR)"; export AR; \ > > > > + @$(if $*,@)AR="$(AR)"; export AR; \ > > > > RANLIB="$(RANLIB)"; export RANLIB; \ > > > > if [ -z "$*" ] && [ -n "$(filter -j%, $(MFLAGS))" ]; then \ > > > > rm -rf normal-parallel || true; \ > > > > > > > > > > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] libstdc++: Reduce output of 'make check' 2023-09-07 13:31 [PATCH] libstdc++: Reduce output of 'make check' Jonathan Wakely 2023-09-08 2:15 ` Eric Gallager @ 2023-09-08 17:06 ` Jonathan Wakely 1 sibling, 0 replies; 6+ messages in thread From: Jonathan Wakely @ 2023-09-08 17:06 UTC (permalink / raw) To: Jonathan Wakely; +Cc: libstdc++, gcc-patches On Thu, 7 Sept 2023 at 14:32, Jonathan Wakely via Libstdc++ <libstdc++@gcc.gnu.org> wrote: > > Any objections to this change? Pushed to trunk now. > > -- >8 -- > > This removes the 39 lines of shell commands that get echoed when > starting the testsuite. The fact that near the end of that output it > prints `echo "WARNING: could not find \`runtest'" 1>&2; :;` makes it > look like that warning is actually being shown the the user. > > Suppress echoing the recipe, so that users only see the actual output > from the testsuite, not the makefile recipe as well. > > libstdc++-v3/ChangeLog: > > * testsuite/Makefile.am (check-DEJAGNU): Use @ in recipe. > * testsuite/Makefile.in: Regenerate. > --- > libstdc++-v3/testsuite/Makefile.am | 2 +- > libstdc++-v3/testsuite/Makefile.in | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libstdc++-v3/testsuite/Makefile.am b/libstdc++-v3/testsuite/Makefile.am > index 7adc5318192..4cee585fd8e 100644 > --- a/libstdc++-v3/testsuite/Makefile.am > +++ b/libstdc++-v3/testsuite/Makefile.am > @@ -117,7 +117,7 @@ $(check_DEJAGNU_normal_targets): check-DEJAGNUnormal%: normal%/site.exp > > # Run the testsuite in normal mode. > check-DEJAGNU $(check_DEJAGNU_normal_targets): check-DEJAGNU%: site.exp > - $(if $*,@)AR="$(AR)"; export AR; \ > + @$(if $*,@)AR="$(AR)"; export AR; \ > RANLIB="$(RANLIB)"; export RANLIB; \ > if [ -z "$*" ] && [ -n "$(filter -j%, $(MFLAGS))" ]; then \ > rm -rf normal-parallel || true; \ > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-09-08 17:06 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2023-09-07 13:31 [PATCH] libstdc++: Reduce output of 'make check' Jonathan Wakely 2023-09-08 2:15 ` Eric Gallager 2023-09-08 10:30 ` Jonathan Wakely 2023-09-08 12:14 ` Eric Gallager 2023-09-08 12:18 ` Jonathan Wakely 2023-09-08 17:06 ` Jonathan Wakely
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).