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; \ >
