From: Bruno Haible <bruno@clisp.org>
To: libc-alpha@sourceware.org,
Adhemerval Zanella <adhemerval.zanella@linaro.org>
Cc: Paul Eggert <eggert@cs.ucla.edu>
Subject: Re: [PATCH v4] posix: Sync tempname with gnulib
Date: Wed, 10 Apr 2024 00:20:37 +0200 [thread overview]
Message-ID: <6820097.n1QcUsQ7Il@nimes> (raw)
In-Reply-To: <20240408184954.2518702-1-adhemerval.zanella@linaro.org>
Hi Adhemerval,
Just a few comments, concentrating on the differences between gnulib
and glibc.
> The 044bf893ac removed __path_search, which is now moved to another
> gnulib shared files (stdio-common/tmpdir.{c,h}) and this patch
> also fixes direxists to use __stat64_time64 instead of __xstat64.
Thanks; I now see that I had suggested this in
<https://lists.gnu.org/r/bug-gnulib/2022-08/msg00077.html>.
1) In gnulib, tmpdir.h and tmpdir.c are under LGPLv3+.
For glibc, the license header needs to be changed to LGPLv2+.
That is
s/version 3/version 2.1/
But before doing this, you need to get permission to do it.
Since the copyright header says "(C) FSF", you need to ask the FSF's
copyright clerk for this permission. Last I tried, this process is
very VERY *VERY* slow.
Alternatively, you can ask the contributors of significant changes
for their permission.
- For tmpdir.h, on its way
gettext/lib/tmpdir.h -> gettext/gettext-tools/lib/tmpdir.h -> gnulib/lib/tmpdir.h
that's only me. (I checked the git history.)
- For tmpdir.c, on its way
glibc/sysdeps/posix/tempname.c -> gettext/lib/tmpdir.c ->
gettext/gettext-tools/lib/tmpdir.c -> gnulib/lib/tmpdir.c
that's only me (2011-08-23) and Paul Eggert (2013-07-15).
(I checked the git history.)
I hereby give the permission to relax the license of tmpdir.[hc]
to LGPLv2+.
Paul, do you do the same?
2) In tmpdir.c: Please conditionalize the
#include "pathmax.h"
with
#ifndef _LIBC
not
#if defined _WIN32 && ! defined __CYGWIN__
so that this change can be merged back to gnulib.
3) In sysdeps/posix/tempname.c:
+#if !_LIBC
*r = mix_random_values (v, clock ());
+#endif
How about adding a comment here?
/* In glibc, clock_gettime is expected to always succeed. */
+#if !_LIBC
*r = mix_random_values (v, clock ());
+#endif
Bruno
next prev parent reply other threads:[~2024-04-09 22:20 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-08 18:49 Adhemerval Zanella
2024-04-09 22:20 ` Bruno Haible [this message]
2024-04-10 1:44 ` Paul Eggert
2024-04-10 11:58 ` Adhemerval Zanella Netto
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=6820097.n1QcUsQ7Il@nimes \
--to=bruno@clisp.org \
--cc=adhemerval.zanella@linaro.org \
--cc=eggert@cs.ucla.edu \
--cc=libc-alpha@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).