]> git.ipfire.org Git - thirdparty/automake.git/commitdiff
maint: avoid a racy distcheck failure
authorJim Meyering <meyering@meta.com>
Wed, 27 Dec 2023 14:52:50 +0000 (06:52 -0800)
committerJim Meyering <meyering@meta.com>
Wed, 27 Dec 2023 14:52:50 +0000 (06:52 -0800)
* lib/am/distdir.am (am__remove_distdir): Use ";" not "&&" after
the chmod-running find, so that a failing find doesn't cause the
entire rule to fail.  This could happen when a directory like
doc/automake.t2d being processed by find is concurrently deleted
by another rule.

lib/am/distdir.am

index 301239de1e38e130da6c173960df3390a574e185..0dd3a2e3faefdf04eaa106c66865ae81ce46ce5b 100644 (file)
@@ -24,7 +24,7 @@ top_distdir = $(distdir)
 am__remove_distdir = \
   if test -d "$(distdir)"; then \
     find "$(distdir)" -type d ! -perm -700 -exec chmod u+rwx {} ';' \
-      && rm -rf "$(distdir)" \
+      ; rm -rf "$(distdir)" \
 ## On MSYS (1.0.17) it is not possible to remove a directory that is in
 ## use; so, if the first rm fails, we sleep some seconds and retry, to
 ## give pending processes some time to exit and "release" the directory