Avoid another use of `chmod -R'.

* Makefile.am (path-check): To be safe, do not use `chmod -R' on
$(distdir) before removing it (as Solaris `chmod -R' touches
symlink targets).  Instead, use the cleanup strategy used in
distdir.am (which is also employed in tests/defs.in).

diff --git a/ChangeLog b/ChangeLog
index 74b2970660823f2ab25c6bf56a394a9d6845cfae..99e0f5ff092a79c88cbc70c80af25ddac27b7e6d 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2010-03-30  Stefano Lattarini  <stefano.lattarini@gmail.com>
+
+       Avoid another use of `chmod -R'.
+       * Makefile.am (path-check): To be safe, do not use `chmod -R' on
+       $(distdir) before removing it (as Solaris `chmod -R' touches
+       symlink targets).  Instead, use the cleanup strategy used in
+       distdir.am.
+
 2010-03-28  Ralf Wildenhues  <Ralf.Wildenhues@gmx.de>
 
        Remove uses of @acronym and @sc.

diff --git a/Makefile.am b/Makefile.am
index d176d7b6a41ffb9ab3b91e01fe081d2e9ecff7c7..82e1aef5530f7142ea6004ddb6e0ebb721670db8 100644 (file)
--- a/Makefile.am
+++ b/Makefile.am
@@ -483,7 +483,8 @@ path-check: distdir
 ## of files on stdin, at least.
          find . -print | xargs pathchk -p); \
          status=$$?; \
-         chmod -R a+w $(distdir) > /dev/null 2>&1; rm -rf $(distdir); \
+         find $(distdir) -type d '!' -perm -200 -exec chmod u+w {} ';'; \
+         rm -rf $(distdir); \
          exit $$status
 
 ## Program to use to fetch files.

diff --git a/Makefile.in b/Makefile.in
index 9088a0b630a1c37fc0bfe55b582eb37bd77f48d8..dd81522b3701e04f7e0393753aef0e86723316df 100644 (file)
--- a/Makefile.in
+++ b/Makefile.in
@@ -1153,7 +1153,8 @@ path-check: distdir
        (cd $(distdir) && \
          find . -print | xargs pathchk -p); \
          status=$$?; \
-         chmod -R a+w $(distdir) > /dev/null 2>&1; rm -rf $(distdir); \
+         find $(distdir) -type d '!' -perm -200 -exec chmod u+w {} ';'; \
+         rm -rf $(distdir); \
          exit $$status
 
 fetch: