Re: [PATCH] Basic support for checking NFSv4 ACLs in Linux

2022-12-27 Thread Paul Eggert
Some static checking helped find an off-by-one bug that I introduced to your Gnulib patch. The bug caused file_has_acl to sometimes incorrectly return -1 when given a nontrivial ACL in which a WHOLEN is a multiple of 4. Sorry about that. I installed the attached further patch to fix it.From d65e

Re: [PATCH 0/6] build-aux/bootstrap that doesn't need to replace itself

2022-12-27 Thread Paul Eggert
On 12/27/22 13:09, Bruno Haible wrote: I strongly object against the backwards move regarding the autopull.sh / autogen.sh *concepts*. The concepts are still there; the only issue is the API. Whether the API should be './bootstrap --pull && ./bootstrap --gen' or './autopull.sh && ./autogen.sh

Re: [PATCH 0/6] build-aux/bootstrap that doesn't need to replace itself

2022-12-27 Thread Bruno Haible
Hi Paul, You write in : > I put the latter script into the gzip > repository but did not put the auxiliary files there. I did not notice > the error in my testing, since the three auxiliary files were there in > my working copy. I'm sorry t

[PATCH] fdatasync: fix comment

2022-12-27 Thread Paul Eggert
--- m4/fdatasync.m4 | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/m4/fdatasync.m4 b/m4/fdatasync.m4 index 9ee3294621..083537aa9c 100644 --- a/m4/fdatasync.m4 +++ b/m4/fdatasync.m4 @@ -1,4 +1,4 @@ -# fdatasync.m4 serial 5 +# fdatasync.m4 serial 6 dnl Copyright (C) 2008-2

[PATCH 4/6] Bootstrap with functions, not scripts

2022-12-27 Thread Paul Eggert
* top/bootstrap: Use autopull and autogen functions, not shell scripts. This lets build-aux/bootstrap become a standalone script. It does not change the behavior of top/bootstrap, except for minor rewording of disagnostics. --- ChangeLog | 6 ++ build-aux/bootstrap | 14 ++

[PATCH 2/6] Make autogen a shell function too

2022-12-27 Thread Paul Eggert
This does not change behavior. It is helpful for future changes. * top/autogen.sh: Call autogen to do the actual work. (usage, version_controlled_file, gitignore_entries, insert_if_absent): (insert_vc_ignore, symlink_to_dir): Move to top/bootstrap-funclib.sh. * top/bootstrap-funclib.sh (autogen_us

[PATCH 1/6] Move scriptversion= lines up in scripts

2022-12-27 Thread Paul Eggert
By default they need to be in the first 8 lines to be updated. --- build-aux/bootstrap | 8 top/autogen.sh | 4 ++-- top/autopull.sh | 4 ++-- top/bootstrap| 4 ++-- top/bootstrap-funclib.sh | 4 ++-- 5 files changed, 12 insertions(+), 12 deletions(-)

[PATCH 5/6] Support packages with just 'bootstrap'

2022-12-27 Thread Paul Eggert
* top/bootstrap: With --version, also output library version. Support update of package that has only the bootstrap script, and not the other three files. * top/bootstrap-funclib.sh (scriptlibversion): Rename from scriptversion. All uses changed. This way we can distinguish script from script lib

[PATCH] stdnoreturn: deprecate

2022-12-27 Thread Paul Eggert
C23 says is obsolescent, so deprecate the stdnoreturn module. I don't think it was being used anyway as it had too many problems. * modules/stdnoreturn: Mark as obsolete. --- ChangeLog | 8 +++ NEWS | 3 +++ doc/noreturn.texi

[PATCH 3/6] Make autopull a shell function too

2022-12-27 Thread Paul Eggert
This does not change behavior. It is helpful for future changes. * top/autopull.sh: Call autopull to do the actual work. (usage, download_po_files, update_po_files): Move to top/bootstrap-funclib.sh. * top/bootstrap-funclib.sh (autopull_usage): Rename from ‘usage’. (autopull): New function, contai

[PATCH 6/6] Add --pull, --gen options to build-aux/bootstrap

2022-12-27 Thread Paul Eggert
This supports a single bootstrap script with --pull and --gen options, as an alternative to separate autogen.sh and autopull.sh and bootstrap-funclib.sh auxiliary files. * top/bootstrap: With --version, also output library version. Support update of package that has only the bootstrap script, and n

[PATCH 0/6] build-aux/bootstrap that doesn't need to replace itself

2022-12-27 Thread Paul Eggert
This set of patches addresses a problem I found with gzip . The idea is to add a new feature to the bootstrap process so that a package can put just a single bootstrap script into its repository, a script that does not need to replace itself, while still providing a way

Re: Android environments

2022-12-27 Thread Jeffrey Walton
On Tue, Dec 27, 2022 at 3:05 AM Po Lu wrote: > > Bruno Haible writes: > > > I see. There are two ways to build C programs for Android: > > > > (A) The way it is designed by Google: With the Android NDK, > > that includes a cross-compiler. For the runtime, use an emulator > > (based

Fix compilation errors in C++ mode on Solaris 11 OpenIndiana

2022-12-27 Thread Bruno Haible
The newest Solaris OpenIndiana release (OpenIndiana 2022.10) comes with GCC 11. Therefore the _GL_ATTRIBUTE_DEALLOC and _GL_ATTRIBUTE_DEALLOC_FREE macros start to be effective on this platform. A gnulib testdir shows a couple of compilation errors in C++ files, see attached file 'log2b'. I could r

Re: [PATCH] stdnoreturn: deprecate

2022-12-27 Thread Bruno Haible
Paul Eggert wrote: > + * modules/stdnoreturn: Mark as obsolete. Here too, the classification of a module as "obsolete" is wrong, because an obsolete module is one which has no effect on any reasonable portability target. 2022-12-27 Bruno Haible stdnoreturn: Mark as deprecated, no

Re: Android environments

2022-12-27 Thread Po Lu
Bruno Haible writes: > I see. There are two ways to build C programs for Android: > > (A) The way it is designed by Google: With the Android NDK, > that includes a cross-compiler. For the runtime, use an emulator > (based on qemu) or a physical connection to a real device. > > (B)