GNU bug report logs - #13760
am__make_dryrun fails to handle GNU make -I option

Previous Next

Full log

View this message in rfc822 format

From: Stefano Lattarini <stefano.lattarini <at> gmail.com>
To: boris <at> codesynthesis.com
Cc: 13760 <at> debbugs.gnu.org, automake-patches <at> gnu.org
Subject: bug#13760: [PATCH 3/3] dry-run: don't get confused by '-I' option
Date: Tue, 23 Apr 2013 11:12:48 +0200

Fixes automake bug#13760 for non-GNU make implementations that still
support the option '-I'.  So far, the only such make implementation
are FreeBSD (8.x) make and NetBSD (5.x) make.

* lib/am/header-vars.am (am__make_dryrun): If a non-GNU make is being
used, try to handle the '-I' option in $MAKEFLAGS correctly.  For GNU
make, that is already done by the proper use of the $MFLAGS variable.

Signed-off-by: Stefano Lattarini <stefano.lattarini <at> gmail.com>
---
 lib/am/header-vars.am | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/lib/am/header-vars.am b/lib/am/header-vars.am
index 23c627e..d2f0984 100644
--- a/lib/am/header-vars.am
+++ b/lib/am/header-vars.am
@@ -61,9 +61,17 @@ am__make_dryrun = \
            echo 'am--echo: ; @echo "AM"  OK' | $(MAKE) -f - 2>/dev/null \
              | grep '^AM OK$$' >/dev/null || am__dry=yes ;; \
          *) \
+	   am__skip_next=no; \
            for am__flg in $$MAKEFLAGS; do \
+	     if test $$am__skip_next = yes; then \
+	       am__skip_next=no; \
+	       continue; \
+	     fi; \
              case $$am__flg in \
                *=*|--*) ;; \
+## Quite ugly special-casing.  We might need other similar, but let's
+## wait until the need arises.
+	       -I) am__skip_next=yes;; \
                *n*) am__dry=yes; break;; \
              esac; \
            done ;;\
-- 
1.8.2.1.389.gcaa7d79

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.