On 14/10/20 17:29 +0100, Jonathan Wakely wrote:
The FIXME comment saying "Leave out until this is tested a bit more" is
from 1997. I think it's been sufficiently tested.
ChangeLog:
* config-ml.in (multi-do): Add @ to silence recipe. Remove FIXME
comment.
OK for trunk?
This removes 44 lines of irrelevant noise from various build targets,
such as the 'check' target that runs the libstdc++ testsuite.
Actually there are two instances of this FIXME in that file. This
revised patch deals with both.
It looks like this file is shared with binutils-gdb and newlib-cygwin,
I've only tested it for GCC.
commit e257e460d4241bfbb31fd1714ee1c3000b1a378b
Author: Jonathan Wakely <jwak...@redhat.com>
Date: Wed Oct 14 16:15:50 2020
config-ml.in: Suppress output from multi-do recipes
The FIXME comments saying "Leave out until this is tested a bit more"
are from 1997. I think they've been sufficiently tested.
ChangeLog:
* config-ml.in (multi-do, multi-clean): Add @ to silence recipes.
Remove FIXME comments.
diff --git a/config-ml.in b/config-ml.in
index 5720d38d23f..68854a4f16c 100644
--- a/config-ml.in
+++ b/config-ml.in
@@ -499,10 +499,8 @@ cat > Multi.tem <<\EOF
PWD_COMMAND=$${PWDCMD-pwd}
-# FIXME: There should be an @-sign in front of the `if'.
-# Leave out until this is tested a bit more.
multi-do:
- if [ -z "$(MULTIDIRS)" ]; then \
+ @if [ -z "$(MULTIDIRS)" ]; then \
true; \
else \
rootpre=`${PWD_COMMAND}`/; export rootpre; \
@@ -547,10 +545,8 @@ multi-do:
done; \
fi
-# FIXME: There should be an @-sign in front of the `if'.
-# Leave out until this is tested a bit more.
multi-clean:
- if [ -z "$(MULTIDIRS)" ]; then \
+ @if [ -z "$(MULTIDIRS)" ]; then \
true; \
else \
lib=`${PWD_COMMAND} | sed -e 's,^.*/\([^/][^/]*\)$$,\1,'`; \