From dd0a48f50a97ca1185c957b0846e715be7777bcb Mon Sep 17 00:00:00 2001 From: Bruno Haible Date: Thu, 20 Apr 2023 23:21:42 +0200 Subject: getumask: Make it work on native Windows. * lib/getumask.c (getumask): When TMPDIR is unset, try TMP and TEMP. * tests/test-getumask.c (ASSUME_UMASK_CONSTANT): Define to 1 on native Windows. --- ChangeLog | 7 +++++++ lib/getumask.c | 10 ++++++++++ tests/test-getumask.c | 5 +++++ 3 files changed, 22 insertions(+) diff --git a/ChangeLog b/ChangeLog index 7856e350c0..5bdc213a32 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +2023-04-20 Bruno Haible + + getumask: Make it work on native Windows. + * lib/getumask.c (getumask): When TMPDIR is unset, try TMP and TEMP. + * tests/test-getumask.c (ASSUME_UMASK_CONSTANT): Define to 1 on native + Windows. + 2023-04-20 Bruno Haible posixtm tests: Fix link error. diff --git a/lib/getumask.c b/lib/getumask.c index 203d6dc869..e979f9b874 100644 --- a/lib/getumask.c +++ b/lib/getumask.c @@ -106,6 +106,16 @@ getumask (void) { /* Create a temporary file and inspect its access permissions. */ const char *tmpdir = getenv ("TMPDIR"); +# if defined _WIN32 && !defined __CYGWIN__ + if (tmpdir == NULL || *tmpdir == '\0') + { + /* On native Windows, TMPDIR is typically not set, and /tmp does not + exist. $TMP and $TEMP can be used instead. */ + tmpdir = getenv ("TMP"); + if (tmpdir == NULL || *tmpdir == '\0') + tmpdir = getenv ("TEMP"); + } +# endif if (tmpdir == NULL || *tmpdir == '\0') tmpdir = "/tmp"; size_t tmpdir_length = strlen (tmpdir); diff --git a/tests/test-getumask.c b/tests/test-getumask.c index 6ee50836bc..ab1e48ecd9 100644 --- a/tests/test-getumask.c +++ b/tests/test-getumask.c @@ -25,6 +25,11 @@ SIGNATURE_CHECK (getumask, mode_t, (void)); #include "macros.h" +#if defined _WIN32 && !defined __CYGWIN__ +/* On native Windows, getumask() always returns 0111. */ +# define ASSUME_UMASK_CONSTANT 1 +#endif + int main (void) { -- cgit v1.2.1