From 95511aab9d5202470b90530e83d18a6f4b4aeb6b Mon Sep 17 00:00:00 2001 From: Joseph Myers Date: Fri, 15 Dec 2017 22:37:17 +0000 Subject: [PATCH] Fix testing with read-only source directory. Three tests fail with a read-only source directory because they try to write into the source directory. None of these write into it in a way that should actually be problematic for concurrent builds sharing the same writable source directory, but avoiding any writing into the source directory (from testing, or from building glibc if the source timestamps are properly ordered) is still a good idea, as being able to build with read-only sources helps make sure there isn't anything that could cause problems for concurrent builds. This patch changes the tests in question to use either /tmp or the build directory to write their temporary files (or to test O_TMPFILE, as applicable). Tested for x86_64. * io/Makefile (tst-open-tmpfile-ARGS): New variable. * posix/tst-mmap-offset.c (fname): Use /tmp. * stdlib/tst-setcontext3.sh (tempfile): Use ${objpfx}. --- ChangeLog | 6 ++++++ io/Makefile | 2 ++ posix/tst-mmap-offset.c | 2 +- stdlib/tst-setcontext3.sh | 2 +- 4 files changed, 10 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index e96b2b49e4..852043e521 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2017-12-15 Joseph Myers + + * io/Makefile (tst-open-tmpfile-ARGS): New variable. + * posix/tst-mmap-offset.c (fname): Use /tmp. + * stdlib/tst-setcontext3.sh (tempfile): Use ${objpfx}. + 2017-12-15 Steve Ellcey * nscd/dbg_log.c (dbg_log): Increase msg buffer size. diff --git a/io/Makefile b/io/Makefile index b3ee5ae647..c72519541f 100644 --- a/io/Makefile +++ b/io/Makefile @@ -109,6 +109,8 @@ test-stat2-ARGS = Makefile . $(objpfx)test-stat2 tst-statvfs-ARGS = $(objpfx)tst-statvfs tst-statvfs.c /tmp +tst-open-tmpfile-ARGS = --test-dir=$(objpfx) + ifeq ($(run-built-tests),yes) $(objpfx)ftwtest.out: ftwtest-sh $(objpfx)ftwtest $(SHELL) $< $(common-objpfx) '$(test-program-cmd)' > $@; \ diff --git a/posix/tst-mmap-offset.c b/posix/tst-mmap-offset.c index 5bb88aab10..5e940c8d95 100644 --- a/posix/tst-mmap-offset.c +++ b/posix/tst-mmap-offset.c @@ -29,7 +29,7 @@ static int fd; static long int page_shift; -static char fname[] = "tst-mmap-offset-XXXXXX"; +static char fname[] = "/tmp/tst-mmap-offset-XXXXXX"; static void do_prepare (int argc, char **argv) diff --git a/stdlib/tst-setcontext3.sh b/stdlib/tst-setcontext3.sh index 158d0dc0a8..014e7cfc5b 100644 --- a/stdlib/tst-setcontext3.sh +++ b/stdlib/tst-setcontext3.sh @@ -34,7 +34,7 @@ cleanup() { } trap cleanup 0 -tempfile=$(mktemp "tst-setcontext3.XXXXXXXXXX") +tempfile=$(mktemp "${objpfx}tst-setcontext3.XXXXXXXXXX") # We want to run the test program and see if secontext called # exit() and wrote out the test file we specified. If the -- 2.11.4.GIT