This is the mail archive of the
glibc-cvs@sourceware.org
mailing list for the glibc project.
GNU C Library master sources branch master updated. glibc-2.26.9000-787-g8adfb0e
- From: fw at sourceware dot org
- To: glibc-cvs at sourceware dot org
- Date: 18 Nov 2017 13:12:03 -0000
- Subject: GNU C Library master sources branch master updated. glibc-2.26.9000-787-g8adfb0e
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU C Library master sources".
The branch, master has been updated
via 8adfb0eeffd1888f9b53b8d8677eb1656cd20e47 (commit)
from df0c40ee3a893238ac11f4c0d876a0c3b49d198d (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
http://sourceware.org/git/gitweb.cgi?p=glibc.git;a=commitdiff;h=8adfb0eeffd1888f9b53b8d8677eb1656cd20e47
commit 8adfb0eeffd1888f9b53b8d8677eb1656cd20e47
Author: Florian Weimer <fweimer@redhat.com>
Date: Sat Nov 18 14:11:09 2017 +0100
support_create_temp_directory: Align behavior with create_temp_file
create_temp_file automatically supplies the test directory and the
XXXXXX suffix. support_create_temp_directory required the caller to
specify them, which was confusing.
Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
diff --git a/ChangeLog b/ChangeLog
index 604d571..c58c58c 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2017-11-18 Florian Weimer <fweimer@redhat.com>
+
+ * support/temp_file.c (support_create_temp_directory): Use
+ test_dir and do not rely on the presence of the XXXXXX suffix.
+ * support/temp_file.h (support_create_temp_directory): Update
+ comment.
+ * support/tst-xreadlink.c (do_test): Adjust.
+ * support/support_chroot.c (support_chroot_create): Likewise.
+
2017-11-17 Tulio Magno Quites Machado Filho <tuliom@linux.vnet.ibm.com>
* sysdeps/powerpc/bits/hwcap.h (PPC_FEATURE2_HTM_NO_SUSPEND): New
diff --git a/support/support_chroot.c b/support/support_chroot.c
index f6fad18..693813f 100644
--- a/support/support_chroot.c
+++ b/support/support_chroot.c
@@ -45,12 +45,7 @@ struct support_chroot *
support_chroot_create (struct support_chroot_configuration conf)
{
struct support_chroot *chroot = xmalloc (sizeof (*chroot));
-
- {
- char *template = xasprintf ("%s/tst-resolv-res_init-XXXXXX", test_dir);
- chroot->path_chroot = support_create_temp_directory (template);
- free (template);
- }
+ chroot->path_chroot = support_create_temp_directory ("tst-resolv-res_init-");
/* Create the /etc directory in the chroot environment. */
char *path_etc = xasprintf ("%s/etc", chroot->path_chroot);
diff --git a/support/temp_file.c b/support/temp_file.c
index cbd54e2..547263a 100644
--- a/support/temp_file.c
+++ b/support/temp_file.c
@@ -89,14 +89,14 @@ create_temp_file (const char *base, char **filename)
char *
support_create_temp_directory (const char *base)
{
- char *base_copy = xstrdup (base);
- if (mkdtemp (base_copy) == NULL)
+ char *path = xasprintf ("%s/%sXXXXXX", test_dir, base);
+ if (mkdtemp (path) == NULL)
{
- printf ("error: mkdtemp (\"%s\"): %m", base);
+ printf ("error: mkdtemp (\"%s\"): %m", path);
exit (1);
}
- add_temp_file (base_copy);
- return base_copy;
+ add_temp_file (path);
+ return path;
}
/* Helper functions called by the test skeleton follow. */
diff --git a/support/temp_file.h b/support/temp_file.h
index dfefe58..3b8563e 100644
--- a/support/temp_file.h
+++ b/support/temp_file.h
@@ -32,9 +32,9 @@ void add_temp_file (const char *name);
*FILENAME. */
int create_temp_file (const char *base, char **filename);
-/* Create a temporary directory and schedule it for deletion. BASE
- must end with the six characters "XXXXXX". Return the name of the
- temporary directory. The caller should free the string. */
+/* Create a temporary directory and schedule it for deletion. BASE is
+ used as a prefix for the unique directory name, which the function
+ returns. The caller should free this string. */
char *support_create_temp_directory (const char *base);
__END_DECLS
diff --git a/support/tst-xreadlink.c b/support/tst-xreadlink.c
index cb2c12a..a4a2281 100644
--- a/support/tst-xreadlink.c
+++ b/support/tst-xreadlink.c
@@ -29,7 +29,7 @@
static int
do_test (void)
{
- char *dir = support_create_temp_directory ("/tmp/tst-xreadlink-XXXXXX");
+ char *dir = support_create_temp_directory ("tst-xreadlink-");
char *symlink_name = xasprintf ("%s/symlink", dir);
add_temp_file (symlink_name);
-----------------------------------------------------------------------
Summary of changes:
ChangeLog | 9 +++++++++
support/support_chroot.c | 7 +------
support/temp_file.c | 10 +++++-----
support/temp_file.h | 6 +++---
support/tst-xreadlink.c | 2 +-
5 files changed, 19 insertions(+), 15 deletions(-)
hooks/post-receive
--
GNU C Library master sources