]> sourceware.org Git - glibc.git/commitdiff
Fix Hurd build with read-only source directory.
authorJoseph Myers <joseph@codesourcery.com>
Tue, 27 Nov 2018 22:05:49 +0000 (22:05 +0000)
committerJoseph Myers <joseph@codesourcery.com>
Tue, 27 Nov 2018 22:05:49 +0000 (22:05 +0000)
The logic for generating sysdeps/mach/hurd/bits/errno.h involves a
stamp file and $(move-if-change).

The temporary file (generated unconditionally) is generated in the
source directory.  This means that even if
sysdeps/mach/hurd/bits/errno.h is up to date, and has an up to date
timestamp, the build will fail if the source directory is read-only.
Even with a writable source directory, multiple concurrent builds for
i686-gnu with the same source directory could race to access the
temporary file (which always has the same name).

This patch uses the build directory for the temporary file instead to
avoid those problems.  (In the case where the file is out of date and
the temporary file does need to be moved to the source directory, if
there are multiple concurrent builds for i686-gnu with the same source
directory, and the source and build directories are on different
filesystems, it's possible there might still be races replacing the
file in the source directory, depending on exactly how mv handles such
cross-filesystem moves.  This is certainly no worse than the present
situation, where such a case would have races regardless of whether
the file is out of date or whether different filesystems are in use.)

Tested with a build-many-glibcs.py build for i686-gnu.

* sysdeps/mach/hurd/Makefile ($(common-objpfx)stamp-errnos): Use
$(hurd-objpfx)bits/errno.h-tmp, not $(hurd)/bits/errno.h-tmp.

ChangeLog
sysdeps/mach/hurd/Makefile

index d08c2d84f123962cb949d5cbbf34228baa66239c..0d1ddaad4363fe0c921aa31f5ccf7213e8eb6838 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2018-11-27  Joseph Myers  <joseph@codesourcery.com>
+
+       * sysdeps/mach/hurd/Makefile ($(common-objpfx)stamp-errnos): Use
+       $(hurd-objpfx)bits/errno.h-tmp, not $(hurd)/bits/errno.h-tmp.
+
 2018-11-27  Florian Weimer  <fweimer@redhat.com>
 
        * support/support.h (support_quote_string): Declare.
index e15341ca8d945e8d27cbaba89b047ecc703fba8f..69f12d4efbb98603cf9b074d49f3510c8aaea8f5 100644 (file)
@@ -91,10 +91,11 @@ $(common-objpfx)errnos.d: $(mach-errnos-deps)
 $(hurd)/bits/errno.h: $(common-objpfx)stamp-errnos ;
 $(common-objpfx)stamp-errnos: $(hurd)/errnos.awk $(errno.texinfo) \
                              $(mach-errnos-deps) $(common-objpfx)errnos.d
-       $(AWK) -f $^ > $(hurd)/bits/errno.h-tmp
+       mkdir -p $(hurd-objpfx)bits
+       $(AWK) -f $^ > $(hurd-objpfx)bits/errno.h-tmp
 # Make it unwritable so noone will edit it by mistake.
-       -chmod a-w $(hurd)/bits/errno.h-tmp
-       $(move-if-change) $(hurd)/bits/errno.h-tmp $(hurd)/bits/errno.h
+       -chmod a-w $(hurd-objpfx)bits/errno.h-tmp
+       $(move-if-change) $(hurd-objpfx)bits/errno.h-tmp $(hurd)/bits/errno.h
        touch $@
 
 common-generated += errnos.d stamp-errnos
This page took 0.064155 seconds and 5 git commands to generate.