* [PATCH] libstdc++: Reduce output of 'make check'
@ 2023-09-07 13:31 Jonathan Wakely
2023-09-07 17:29 ` François Dumont
` (2 more replies)
0 siblings, 3 replies; 7+ 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] 7+ 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-07 17:29 ` François Dumont
2023-09-08 2:15 ` Eric Gallager
2023-09-08 17:06 ` Jonathan Wakely
2 siblings, 0 replies; 7+ messages in thread
From: François Dumont @ 2023-09-07 17:29 UTC (permalink / raw)
To: Jonathan Wakely, libstdc++
+1 for this cleanup, thanks.
Sorry to troll but I noticed another annoying recent behavior. Maybe the
one who did it will recognize himself and will be willing to find a
solution :-)
When I create a testsuite_files file with 1 test case I have the
following output:
Running
/home/fdumont/dev/gcc/git/libstdc++-v3/testsuite/libstdc++-dg/conformance.exp
...
=== libstdc++ Summary ===
# of expected passes 6
It should be 2. There are 4 additional tests coming from
experimental/simd folder.
I guess they are automatically added for good reasons. But it would be
great to add those only if there is no testsuite_files.
On 07/09/2023 15:31, Jonathan Wakely via Libstdc++ 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] 7+ 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-07 17:29 ` François Dumont
@ 2023-09-08 2:15 ` Eric Gallager
2023-09-08 10:30 ` Jonathan Wakely
2023-09-08 17:06 ` Jonathan Wakely
2 siblings, 1 reply; 7+ 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] 7+ 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; 7+ 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] 7+ 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; 7+ 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] 7+ 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; 7+ 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] 7+ 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-07 17:29 ` François Dumont
2023-09-08 2:15 ` Eric Gallager
@ 2023-09-08 17:06 ` Jonathan Wakely
2 siblings, 0 replies; 7+ 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] 7+ messages in thread
end of thread, other threads:[~2023-09-08 17:06 UTC | newest]
Thread overview: 7+ 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-07 17:29 ` François Dumont
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).