bug#71029: single-binary build broken on systems with separate libintl

2024-05-21 Thread Pádraig Brady
On 18/05/2024 18:09, Audrey Dutcher wrote: I presume you have the same issue with coreutils 9.4 ? Correct. I presume the problematic flags are propagated through LIBINTL or MBRTOWC_LIB. What are those set to for reference in your Makefile? On plain FreeBSD it is set to

bug#71029: single-binary build broken on systems with separate libintl

2024-05-18 Thread Pádraig Brady
On 17/05/2024 17:11, Audrey Dutcher wrote: On my FreeBSD system, downloading coreutils-9.5.tar.xz, then building with `./configure --enable-single-binary && make` does not succeed, with the error message `don't know how to make -Wl,-rpath. Stop` I believe the root cause of this is

bug#70946: [PATCH] doc: improve the man page for sleep

2024-05-15 Thread Pádraig Brady
On 15/05/2024 03:46, Nikolaos Chatzikonstantinou wrote: On Tue, May 14, 2024 at 4:03 PM Nikolaos Chatzikonstantinou wrote: On Tue, May 14, 2024, 3:59 PM Pádraig Brady wrote: On 14/05/2024 17:36, Nikolaos Chatzikonstantinou wrote: See attachment. Well just above your new mention

bug#70946: [PATCH] doc: improve the man page for sleep

2024-05-14 Thread Pádraig Brady
On 14/05/2024 17:36, Nikolaos Chatzikonstantinou wrote: See attachment. Well just above your new mention of floating-point, we have: "NUMBER need not be an integer". How about I adjust your patch to adjust that text to say: "NUMBER can be an integer or floating-point". cheers, Pádraig.

bug#70887: In coreutils 9.5, "cp" command does not honor "--interactive" option

2024-05-12 Thread Pádraig Brady
On 12/05/2024 16:06, Paul Eggert wrote: On 2024-05-12 04:49, Pádraig Brady wrote: @@ -1151,7 +1151,8 @@ main (int argc, char **argv) { /* Default cp operation. */ x.update = false; - x.interactive = I_UNSPECIFIED

bug#70887: In coreutils 9.5, "cp" command does not honor "--interactive" option

2024-05-12 Thread Pádraig Brady
On 12/05/2024 00:03, Robert Hill wrote: After upgrading coreutils from 9.0 to 9.5, the following change occurred: In coreutils 9.0, the command "cp -Tipruvx /src-dir /dst-dir" requested interactive confirmation before replacing an old destination file with a newer source file, as expected. In

bug#70873: join --return-error-if-any-unmatched-lines

2024-05-11 Thread Pádraig Brady
On 11/05/2024 10:14, Dan Jacobson wrote: join should have an option to return an error value in the shell's $? if any lines are not matched. Currently the man page doesn't even mention a return value. So it is not set in stone yet. Currently one must save -v output in a file then use test -s

Re: cksum: add tests to verify the output in case of errors

2024-05-11 Thread Pádraig Brady
On 10/05/2024 22:15, Sylvestre Ledru wrote: Hello The attached patch improves the coverage with --check & --strict, especially in the context of errors/warnings. Pushed with these changes: - Started the summary line with "tests", as not changing cksum - Changed double quotes to single quotes

Re: [PATCH] wc: speed-up by simplifying avx code

2024-05-09 Thread Pádraig Brady
On 31/03/2024 21:23, Pádraig Brady wrote: On 31/03/2024 18:58, Evgeny Nizhibitsky wrote: Yes, it's true that simplifying and speeding-up by the bufsize increase are two different things although the former allowed the latter. I just landed more tests with hyperfine for various configurations

bug#70214: 'install' fails to copy regular file to autofs/cifs, due to ACL or xattr handling

2024-05-09 Thread Pádraig Brady
On 13/04/2024 18:42, Pádraig Brady wrote: On 13/04/2024 17:39, Pádraig Brady wrote: install(1) defaults to mode 600 for new files, and uses set_acl() with that (since 2007 https://github.com/coreutils/coreutils/commit/f634e8844 ) The psuedo code that install(1) uses is: copy_reg() if (x

bug#70814: pwd(1) man page: Unclear "avoid" wording

2024-05-07 Thread Pádraig Brady
On 07/05/2024 05:19, Bruce Jerrick wrote: This wording in the pwd(1) man page is unclear:     -P, --physical     avoid all symlinks "resolve all symlinks" would be much better wording. Pushed that in your name. Marking this as done. thanks, Pádraig

bug#70801: Weird behaviour when standard input is closed

2024-05-06 Thread Pádraig Brady
On 06/05/2024 08:38, Bernard Burette wrote: Hi, If I try: $ cat <&- cat: -: Bad file descriptor cat: closing standard input: Bad file descriptor $ The error on stdin beign closed is displayed twice plus "-" is for a FILE argument to replace standard input, It would make more sense to

Re: [PATCH] maint: factor out the common show_date functionality

2024-05-05 Thread Pádraig Brady
On 05/05/2024 10:06, Nikolay Nechaev via GNU coreutils General Discussion wrote: Note I adjusted your email to be more standard, rather than the above: + fputs (timetostr(pdui->tmax.tv_sec, buf), stdout); `make syntax-check` indicated the missing space in timetostr(), and also a

Re: [PATCH 1/2] maint: factor out common macros of stat and printf

2024-05-05 Thread Pádraig Brady
Pushed both of these. thanks, Pádraig

Re: cksum: add tests to verify the presence of "*" (and question about usability of cksum)

2024-05-04 Thread Pádraig Brady
On 03/05/2024 18:53, Sylvestre Ledru wrote: Hello, The attached patch checks for a code path that wasn't covered before (afaik). Pushed. As a side note, the various combinations of --tag / --untagged and --binary / --text is producing some unexpected results at time :) Maybe that ship

bug#70727: cp doesn't recognize the new --update=none-fail option

2024-05-03 Thread Pádraig Brady
On 03/05/2024 05:12, Attila Fidan via GNU coreutils Bug Reports wrote: Hi, I wanted to use the new cp --update=none-fail option introduced in 9.5, but it said "invalid argument ‘none-fail’ for ‘--update’". It turns out that the commit (49912bac286eb3c0ef7d1567ae790193ad5eb2e8) adding it forgot

bug#70714: realpath no error for unreadable-symlink

2024-05-02 Thread Pádraig Brady
On 02/05/2024 07:16, Nineteendo INC wrote: coreutils version: stable 9.5 (bottled) OS version: macOS 13.6.6 (22G630) `realpath` doesn’t behave correctly for unreadable symlinks: wannes@Stefans-iMac ~ % ln -s . src wannes@Stefans-iMac ~ % grealpath -e src/.. /Users wannes@Stefans-iMac ~ % chmod

bug#70699: Possible bug in /usr/bin/paste

2024-05-01 Thread Pádraig Brady
On 01/05/2024 15:28, Art Shelest via GNU coreutils Bug Reports wrote: Good morning, I am seeing an aberrant behavior from the /usr/bin/paste utility when working with Windows-style CR/LF text files. The repro is for Mint Mate (Virginia). If I change the line endings in the first file to Unix

Re: maint: pacify GCC 14 -Wmissing-variable-declarations

2024-05-01 Thread Pádraig Brady
On 28/04/2024 03:45, Collin Funk wrote: Hi, gnulib-tool.py feels pretty quick in Coreutils. :) I noticed some new warnings from Fedora 40's gcc 14. These patches are pretty much copied from Pauls gnulib commit here:

[PATCH] build: bootstrap with python gnulib-tool by default

2024-04-27 Thread Pádraig Brady
* gnulib: Update to support bootstrapping with python by default. * bootstrap: Sync with gnulib. * cfg.mk: Don't force python implementation with `make world`, rather rely on the auto selection of python if appropriate. --- bootstrap | 210 --

[PATCH] build: provide way to bootstrap with python gnulib-tool

2024-04-23 Thread Pádraig Brady
* cfg.mk: Add a new "world" default target so that one can bootstrap (using the python implementation), configure, and make, by using `make -f cfg.mk`. * gnulib: Update to latest primarily to test the bootstrap python implementation which is now in beta test. * README-hacking: Document the `make

bug#70532: sort: Mention counting fields from the end

2024-04-23 Thread Pádraig Brady
On 23/04/2024 11:14, Dan Jacobson wrote: In (info "(coreutils) sort invocation") be sure to add an example of a way or workaround for counting fields from the end of the line. E.g., we want to sort on the last field, but don't know for sure how many fields a line might contain. E.g., sort by

Re: Modify buffering of standard streams via environment variables (not LD_PRELOAD)?

2024-04-19 Thread Pádraig Brady
On 19/04/2024 12:36, Zachary Santer wrote: On Fri, Apr 19, 2024 at 5:32 AM Pádraig Brady wrote: env variables are what I proposed 18 years ago now: https://sourceware.org/bugzilla/show_bug.cgi?id=2457 And the "resistance to that" from the Red Hat people 24 years ago is listed on

Re: Modify buffering of standard streams via environment variables (not LD_PRELOAD)?

2024-04-19 Thread Pádraig Brady
On 19/04/2024 01:16, Zachary Santer wrote: Was "RFE: enable buffering on null-terminated data" On Wed, Mar 20, 2024 at 4:54 AM Carl Edquist wrote: However, if stdbuf's magic env vars are exported in your shell (either by doing a trick like 'export $(env -i stdbuf -oL env)', or else more

Re: [PATCH] cksum: --quiet option to output only the checksum when reading tty stdin

2024-04-18 Thread Pádraig Brady
On 17/04/2024 18:51, Piotr H. Dabrowski wrote: * src/digest.c: --quiet option when reading tty stdin --- src/digest.c | 15 ++- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/src/digest.c b/src/digest.c index 1a4cfd1fb..6501f5e9d 100644 --- a/src/digest.c +++

bug#70418: ls bug

2024-04-17 Thread Pádraig Brady
On 16/04/2024 23:17, Paul Eggert wrote: On 4/16/24 14:30, Toby Kelsey wrote: The man page doesn't explain this format conflict, while the info page (info '(coreutils) ls invocation' or 'info ls') claims '-f' implies '-1' which is also incorrect: 'ls -1f' gives different output to to 'ls -f'.

Re: maint: Remove unnecessary stdbool.h include

2024-04-17 Thread Pádraig Brady
On 16/04/2024 21:53, Collin Funk wrote: I noticed 'src/temp-stream.c' includes . This should be unnecessary since Gnulib defines in in config.h right? Feel free to apply this patch unless I am missing something. The following returns no other files: $ find . -name '*.[ch]' | xargs grep -E

bug#70411: [bug] install(1) fails to read /dev/stdin on Darwin

2024-04-16 Thread Pádraig Brady
On 16/04/2024 15:47, Alejandro Colomar wrote: Hi Pádraig, On Tue, Apr 16, 2024 at 03:25:22PM +0100, Pádraig Brady wrote: What version of darwin is this? I can't repro on Darwin 21.6.0 (MacOSX 12.6). The issue seems to be that /dev/stdin returns a varying inode which install(1) doesn't like

bug#70411: [bug] install(1) fails to read /dev/stdin on Darwin

2024-04-16 Thread Pádraig Brady
On 16/04/2024 12:33, Pádraig Brady wrote: On 16/04/2024 01:19, Alejandro Colomar wrote: Hi! I don't own a Darwin system, so I can't help much reproduce. However, I've received a bug report to the Linux man-pages, that our build system (GNUmakefile-based), which ends up calling

bug#70411: [bug] install(1) fails to read /dev/stdin on Darwin

2024-04-16 Thread Pádraig Brady
On 16/04/2024 01:19, Alejandro Colomar wrote: Hi! I don't own a Darwin system, so I can't help much reproduce. However, I've received a bug report to the Linux man-pages, that our build system (GNUmakefile-based), which ends up calling ... | install /dev/stdin $@ doesn't work on

bug#70214: 'install' fails to copy regular file to autofs/cifs, due to ACL or xattr handling

2024-04-15 Thread Pádraig Brady
On 15/04/2024 15:37, Andreas Grünbacher wrote: Hello, Am So., 14. Apr. 2024 um 00:43 Uhr schrieb Pádraig Brady : On 13/04/2024 20:29, Bruno Haible wrote: Hi Pádraig, I wrote: 5) The same thing with 'cp -a' succeeds: $ build-sparc64/src/cp -a /var/tmp/foo3941 $HOME/foo3941; echo $? 0

bug#70214: 'install' fails to copy regular file to autofs/cifs, due to ACL or xattr handling

2024-04-13 Thread Pádraig Brady
On 13/04/2024 20:29, Bruno Haible wrote: Hi Pádraig, I wrote: 5) The same thing with 'cp -a' succeeds: $ build-sparc64/src/cp -a /var/tmp/foo3941 $HOME/foo3941; echo $? 0 $ build-sparc64-no-acl/src/cp -a /var/tmp/foo3941 $HOME/foo3941; echo $? 0 You wrote: The psuedo code that install(1)

bug#70214: 'install' fails to copy regular file to autofs/cifs, due to ACL or xattr handling

2024-04-13 Thread Pádraig Brady
On 13/04/2024 17:39, Pádraig Brady wrote: install(1) defaults to mode 600 for new files, and uses set_acl() with that (since 2007 https://github.com/coreutils/coreutils/commit/f634e8844 ) The psuedo code that install(1) uses is: copy_reg() if (x->set_mode) /* install */ set_acl(d

bug#70214: 'install' fails to copy regular file to autofs/cifs, due to ACL or xattr handling

2024-04-13 Thread Pádraig Brady
scount EACESS as a valid errno with FD operations, as CIFS was seen to return that erroneously in some cases. --- ChangeLog | 7 +++ lib/set-permissions.c | 8 +++- 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/ChangeLog b/ChangeLog index c72165e268..fd094d1091 10064

[PATCH] join: fix error message for -a and -v

2024-04-12 Thread Pádraig Brady
* src/join.c (main): s/field/file/ in the error message for -a and -v, introduced in TEXTUTILS-1_13-24-g6f63d53e1. Reported at https://bugs.debian.org/1068864 --- src/join.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/join.c b/src/join.c index fd6fc74ce..fd28febfb

Re: ls flag to show number of hard links

2024-04-10 Thread Pádraig Brady
On 09/04/2024 16:58, Tony Freeman wrote: I seem to recall that some years ago ls -l would show the number of hard links to a file. This may have been on an AIX system. I needed to use this today on a red hat 8 machine but couldn't figure it out. So I used ls -i | sort Am I overlooking a flag

bug#35247: [PATCH] Add alacritty to terminal list supporting colors

2024-04-10 Thread Pádraig Brady
Things have changed a little since the original request. alacritty sets $COLORTERM, and dircolors now auto accepts that since: https://github.com/coreutils/coreutils/commit/75c9fc674 There are some complications with remote shells, but they should boil down to setting up ssh to send/accept

bug#70298: uname -i returns unknown since fedora 38

2024-04-09 Thread Pádraig Brady
On 09/04/2024 10:17, Collin Funk wrote: On 4/9/24 12:57 AM, Paul Eggert wrote: Indeed there is, and I merged your bug report into that old one. It'd be nice if someone could get to the bottom of that bug. I decided to look into this a bit, since I also have an unknown 'uname -i' and 'uname

Re: coreutils-9.5.7-11163 cut/cut test failure

2024-04-07 Thread Pádraig Brady
On 07/04/2024 14:41, Kevin Jerebica wrote: The cut/cut test fails on Arch Linux (kernel: 6.7.0-arch3-1). Any ideas why this might be? The only lines that stick out to me are these three at the top of the log: E 00:00:1712495787.461523  698728 wire_format_lite.cc:603] String field

bug#70231: Performance issue on sort with zero-sized pseudo files

2024-04-07 Thread Pádraig Brady
On 06/04/2024 23:22, Paul Eggert wrote: On 2024-04-06 03:09, Pádraig Brady wrote: I'll apply this. Heh, I beat you to it by looking for similar errors elsewhere and applying the attached patches to fix the issues I found. None of them look like serious bugs. Cool. I thought the sort(1

bug#70219: Bug/Issue with timeout and signals

2024-04-06 Thread Pádraig Brady
tag 70219 notabug close 70219 stop On 06/04/2024 16:50, Branden R. Williams via GNU coreutils Bug Reports wrote: -k, --kill-after=DURATION also send a KILL signal if COMMAND is still running this long after the initial signal was sent If you read the

bug#70231: Performance issue on sort with zero-sized pseudo files

2024-04-06 Thread Pádraig Brady
On 06/04/2024 03:52, Takashi Kusumi wrote: Hi, I have found a performance issue with the sort command when used on pseudo files with zero size. For instance, sorting `/proc/kallsyms`, as demonstrated below, takes significantly longer than executing with `cat`, generating numerous temporary

[PATCH] build: update gnulib submodule to latest

2024-04-04 Thread Pádraig Brady
* gnulib: Update to latest. * NEWS: Document 2 bugs fixed. --- NEWS | 10 ++ gnulib | 2 +- 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index 0d380a8ae..0a5130331 100644 --- a/NEWS +++ b/NEWS @@ -2,6 +2,16 @@ GNU coreutils NEWS

bug#70126: Missing a full stop in a coreutils-9.5-pre2 message

2024-04-01 Thread Pádraig Brady
On 01/04/2024 16:30, Petr Pisar wrote: Hello, while translating coreutils-9.5-pre2 I noticed this message: #: src/chown.c:123 msgid "" " --reference=RFILE use RFILE's ownership rather than specifying values\n" " RFILE is always dereferenced if a symbolic link.\n"

Re: [PATCH] wc: speed-up by simplifying avx code

2024-03-31 Thread Pádraig Brady
On 31/03/2024 18:58, Evgeny Nizhibitsky wrote: Yes, it's true that simplifying and speeding-up by the bufsize increase are two different things although the former allowed the latter. I just landed more tests with hyperfine for various configurations spanning over the current master version

Re: [PATCH] wc: speed-up by simplifying avx code

2024-03-31 Thread Pádraig Brady
On 31/03/2024 13:12, Pádraig Brady wrote: On 31/03/2024 00:18, Evgeny Nizhibitsky wrote: Here is the proposed patch for both simplifying and consistently speeding up the avx version of wc -l by 10% in up to 1 billion rows scenarios on 7800X3D (probably should be tested on different data

Re: [PATCH] wc: speed-up by simplifying avx code

2024-03-31 Thread Pádraig Brady
On 31/03/2024 00:18, Evgeny Nizhibitsky wrote: Here is the proposed patch for both simplifying and consistently speeding up the avx version of wc -l by 10% in up to 1 billion rows scenarios on 7800X3D (probably should be tested on different data samples and CPUs). The patch was mangled, but

bug#70104: "FAIL: test-canonicalize" coreutils v9.5 on musl libc

2024-03-31 Thread Pádraig Brady
On 31/03/2024 10:02, Adept's Lab wrote: test-canonicalize.c:411: assertion 'strcmp (result1, "//") == 0' failed ^ the only error log message I get. Fail was not presented with previous stable versions. This is on musl 1.1.24 as detailed at: https://github.com/coreutils/coreutils/issues/83

Re: wc -l AVX code 10%+10% speedup

2024-03-30 Thread Pádraig Brady
On 30/03/2024 14:52, Evgeny Nizhibitsky wrote: Dear GNU coreutils maintainers, It seems that I found a way to both speed-up (~10%) and simplify (13 insertions, 43 deletions) the wc -l avx code while playing with it, at least on several million to 1 billion row files I tested with my cpu. It

bug#70070: FAIL: test-localtime_r

2024-03-29 Thread Pádraig Brady
On 29/03/2024 12:40, Andreas Schwab wrote: FAIL: test-localtime_r == test-localtime_r.c:58: assertion 'result->tm_hour == 18' failed FAIL test-localtime_r (exit status: 134) FAIL: test-localtime_r-mt = thread2 disturbed by thread1! thread1 disturbed

Re: Feature Request: env -f to read from file

2024-03-29 Thread Pádraig Brady
On 29/03/2024 05:05, Kaz Kylheku wrote: On 2024-03-28 15:29, Balakrishnan Balasubramanian wrote: On 2024-03-28 3:38 p.m., Pádraig Brady wrote: Yes the feature has merit, and was previously discussed at: https://lists.gnu.org/archive/html/coreutils/2021-10/msg1.html There are some

Re: [PATCH 4/4] cut: Optionally treat multiple consecutive delimiters as one

2024-03-28 Thread Pádraig Brady
On 28/03/2024 19:43, Dragan Simic wrote: Hello, Just checking, are there any further thoughts on this topic? It's on the list, but didn't have the priority for this release. I intend to have a look at the new cut proposals for next release. cheers, Pádraig

Re: Date setting examples in manual

2024-03-28 Thread Pádraig Brady
On 24/03/2024 18:32, wrotycz wrote: I noticed there are no examples of how to set date using date, in the manual. Therefore I propose to add these to examples section to make it clear and easy to do so. I used simple formats, more like machine formats to make it even simpler, as no one wants

Re: Feature Request: env -f to read from file

2024-03-28 Thread Pádraig Brady
On 28/03/2024 15:52, Balakrishnan Balasubramanian wrote: When env command is used to pass extra environment variables to a program, it will be useful to read them from a file instead of command line. Instead of     env MYAPP_QA_KEY=X MYAPP_QA_URL=https://qa.myapp.example.com clicommand

[coreutils-announce] coreutils-9.5 released [stable]

2024-03-28 Thread Pádraig Brady
) Petr Malat (1) Bruno Haible (3)Pádraig Brady (75) Christian Göttsche (1) Samuel Tardieu (1) Collin Funk (4) Stephane Chazelas (1) Daan De Meyer (1) Stephen Kitt (1) Greg Wooledge (1) Sylvestre Ledru (3) Grisha Levit

coreutils-9.5 released [stable]

2024-03-28 Thread Pádraig Brady
) Petr Malat (1) Bruno Haible (3)Pádraig Brady (75) Christian Göttsche (1) Samuel Tardieu (1) Collin Funk (4) Stephane Chazelas (1) Daan De Meyer (1) Stephen Kitt (1) Greg Wooledge (1) Sylvestre Ledru (3) Grisha Levit

Re: new snapshot available: coreutils-9.4.185-541b02.tar.xz

2024-03-28 Thread Pádraig Brady
0 test failures on Fedora 39 amd64 (with ASAN enabled) 0 test failures on Centos 7.9 IBM POWER8 8247-22L 0 test failures on AlmaLinux 9.3 IBM POWER10 9043-MRX

Re: coreutils-9.4.185-541b02 od/od-multiple-t test failure

2024-03-27 Thread Pádraig Brady
On 27/03/2024 12:35, Bruno Haible wrote: Pádraig Brady wrote: The original bug report was not specific to floats, so I'll just remove them from the test. I think it is enough to remove the 'long double' part from the test (option letter 'L'). 'float' and 'double' are standardized by IEEE 754

Re: coreutils-9.4.185-541b02 od/od-multiple-t test failure

2024-03-27 Thread Pádraig Brady
On 27/03/2024 12:03, Bruno Haible wrote: The tests/od/od-multiple-t.sh test fails on - Linux/ia64 (Gentoo, recent glibc, real hardware) - Linux/m68k (Debian 12, QEMU emulated) The first 'od' invocation that causes fail=1 to be set is: $ seq 19 > input $ ./od -An -taz -tfLz input 1

Re: [platform-testers] new snapshot available: coreutils-9.4.184-606f5.tar.xz

2024-03-26 Thread Pádraig Brady
On 26/03/2024 21:19, Bruno Haible wrote: Pádraig Brady wrote: We plan to release coreutils-9.5 in the next few days I would suggest to update to the current gnulib, in order to get these two fixes: https://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=commit;h

new snapshot available: coreutils-9.4.185-541b02.tar.xz

2024-03-26 Thread Pádraig Brady
We plan to release coreutils-9.5 in the next few days so any testing you can do on various different systems between now and then would be most welcome. This release comes about 30 weeks after the 9.4 release. Changes since the last pre-release

Re: new snapshot available: coreutils-9.4.184-606f5.tar.xz

2024-03-26 Thread Pádraig Brady
0 test failures on Fedora 39 amd64 (with ASAN enabled) 0 test failures on macOS 12.6 arm64 (m1) FAIL: tests/chown/preserve-root === Bruno already reported. Not a new issue. Due to "chgrp: invalid group: '4294967295'" 1 gnulib test failure on Centos 7.9.2009 ppc64

new snapshot available: coreutils-9.4.184-606f5.tar.xz

2024-03-26 Thread Pádraig Brady
We plan to release coreutils-9.5 in the next few days so any testing you can do on various different systems between now and then would be most welcome. This release comes about 30 weeks after the 9.4 release. Changes since the last pre-release

Re: coreutils-9.4.170-7b206 misc/numfmt test failure

2024-03-26 Thread Pádraig Brady
On 26/03/2024 18:54, Bruno Haible wrote: Pádraig Brady wrote: The misc/numfmt test fails on - Alpine Linux 3.18, - macOS 12.5, - FreeBSD 14.0, - Solaris 11 OpenIndiana. Find attached the test-suite.log files. If these test failures are due to platform-specific *printf bugs

Re: uniq for unsorted input

2024-03-26 Thread Pádraig Brady
On 26/03/2024 16:40, Bruno Haible wrote: The documentation of 'uniq' [1] says: "The input need not be sorted, but repeated input lines are detected only if they are adjacent. If you want to discard non-adjacent duplicate lines, perhaps you want to use 'sort -u'." When I wrote

Re: coreutils-9.4.170-7b206 ls/removed-directory test failure

2024-03-26 Thread Pádraig Brady
On 26/03/2024 18:25, Bruno Haible wrote: Pádraig Brady wrote: The ls/removed-directory test fails on FreeBSD 14.0. tests-suite.log from FreeBSD 14.0: FAIL: tests/ls/removed-directory diff -u /dev/null err --- /dev/null 1970-01-01 +++ err 1970-01-01 +ls

Re: coreutils-9.4.170-7b206 ls/removed-directory test failure

2024-03-26 Thread Pádraig Brady
On 26/03/2024 17:08, Paul Eggert wrote: On 3/26/24 10:50, Pádraig Brady wrote: It seems that readdir() on FreeBSD 14 is _not_ eating the ENOENT from getdirentries(). Attached is an extra check for that, that avoids the test in that case. An alternative would be for ls to ignore the ENOENT from

Re: coreutils-9.4.170-7b206 ls/removed-directory test failure

2024-03-26 Thread Pádraig Brady
On 24/03/2024 16:30, Bruno Haible wrote: The ls/removed-directory test fails on FreeBSD 14.0. tests-suite.log from FreeBSD 14.0: FAIL: tests/ls/removed-directory diff -u /dev/null err --- /dev/null 1970-01-01 +++ err 1970-01-01 +ls: reading directory '.':

Re: Feature request: numfmt --from=K or k

2024-03-26 Thread Pádraig Brady
On 26/03/2024 13:13, Tim Baverstock wrote: Hi. I sometimes end up with a list of files whose sizes represent K instead of bytes. I recognise I can filter them to add the 'K' before passing them through numfmt, but if I could tell numfmt --from=K it would be more convenient. Thanks for your

Re: coreutils-9.4.170-7b206 uniq/uniq test failure

2024-03-26 Thread Pádraig Brady
On 25/03/2024 22:28, Bruno Haible wrote: Pádraig Brady wrote: while uniq (c32isblank) now determines it is not blank (which seems more correct). I agree that U+00A0 NO-BREAK SPACE should better be considered to be non-blank (and Gnulib's c32isblank does so). The only solaris 11 system I

Re: coreutils-9.4.170-7b206 cp/preserve-mode test failure

2024-03-25 Thread Pádraig Brady
On 24/03/2024 16:42, Bruno Haible wrote: The cp/preserve-mode test fails on Alpine Linux 3.18. test-suite.log from Alpine Linux 3.18: FAIL: tests/cp/preserve-mode + mkdir d1 d2 + chmod 705 d2 + cp '--no-preserve=mode' -r d2 d3 + get_mode d1 + stat '-c%f' d1 +

Re: coreutils-9.4.170-7b206 uniq/uniq test failure

2024-03-25 Thread Pádraig Brady
On 24/03/2024 16:15, Bruno Haible wrote: The uniq/uniq test fails on Solaris 11 OpenIndiana. test-suite.log on Solaris 11 OpenIndiana: FAIL: tests/uniq/uniq = schar... uniq: test schar: stdout mismatch, comparing schar.2 (expected) and schar.O (actual) *** schar.2

Re: coreutils-9.4.170-7b206 misc/numfmt test failure

2024-03-25 Thread Pádraig Brady
On 24/03/2024 17:14, Bruno Haible wrote: The misc/numfmt test fails on - Alpine Linux 3.18, - macOS 12.5, - FreeBSD 14.0, - Solaris 11 OpenIndiana. Find attached the test-suite.log files. If these test failures are due to platform-specific *printf bugs, please provide a test case

bug#69995: Untranslatable string

2024-03-25 Thread Pádraig Brady
On 25/03/2024 14:02, Göran Uddeborg wrote: While translating the new version of coreutils to Swedish, I came across this code from the end of chown-core.h: printf (_("\ --from=CURRENT_OWNER:CURRENT_GROUP\n\ change the %sgroup of each file only if\n\

Re: coreutils-9.4.170-7b206 ls/dired test failure

2024-03-25 Thread Pádraig Brady
On 24/03/2024 16:37, Bruno Haible wrote: The ls/dired test fails on: - FreeBSD 14.0, - Solaris 11 OpenIndiana. https://github.com/coreutils/coreutils/commit/a51af4329 thanks, Pádraig

Re: coreutils-9.4.170-7b206 chmod/symlinks test failure

2024-03-25 Thread Pádraig Brady
On 24/03/2024 16:55, Bruno Haible wrote: The chmod/symlinks test fails on: - Alpine Linux 3.18, That was not Alpine specific, rather due to setgid dirs: https://github.com/coreutils/coreutils/commit/f77a6352f - macOS 12.5, - FreeBSD 14.0, - NetBSD 9.3 and 10.0-rc4, - OpenBSD

Re: Correction to pl po translation

2024-03-24 Thread Pádraig Brady
On 24/03/2024 18:43, wrotycz wrote: I noticed some minor mistakes or inconsistences in translation so I send correction. Regards Wrotycz Please direct this to the translation project https://translationproject.org/team/ lists this as: translation-team...@lists.sourceforge.net thanks,

bug#69985: Untranslatable message in src/chown-core.h:95

2024-03-24 Thread Pádraig Brady
On 24/03/2024 16:57, Frédéric Marchal wrote: Hi, In src/chown-core.h:95 (coreutils-9.5-pre1), the following message is impossible to translate as it is created by concatenating string fragments: static inline void emit_from_option_description (bool user) { printf (_("\

Re: some string issues in coreutils-9.4.170-7b206

2024-03-24 Thread Pádraig Brady
On 24/03/2024 16:35, Wenbin Lv wrote: Hi, I just finished updating the zh_CN translation for 9.4.170-7b206. Below are some string related issues I have noticed: src/chown-core.h:101: "owner and/or "should be translatable; Fixed in https://bugs.gnu.org/69985 src/chown.c:84:

Re: Test failure when building in setgid directory

2024-03-24 Thread Pádraig Brady
On 24/03/2024 16:29, Sven Joachim wrote: When building coreutils 9.4.170-7b206 on x86_64 GNU/Linux, I got a test failure in tests/chmod/symlinks, apparently because I was building in a setgid directory. The log file of the failed test is attached. , | $ LANG=C ls -ld

Re: coreutils-9.4.170-7b206 mv/mv-exchange test failure

2024-03-24 Thread Pádraig Brady
On 24/03/2024 16:26, Bruno Haible wrote: The mv/mv-exchange test fails on: - NetBSD 9.3 - OpenBSD 7.4 - AIX 7.1 - AIX 7.3.1 The attached should fix the issue with this new test. thanks for all the testing! Pádraig From c45fcb2ed2d80df5e7dea3c52eba24a9676d9eae Mon Sep 17 00:00:00

Re: env: follow up on argv0 setting feature

2024-03-24 Thread Pádraig Brady
On 23/03/2024 20:24, Matheus Afonso Martins Moreira wrote: Thank you!! Really happy to have this feature in upstream coreutils! Would you mind giving me feedback on the patch I sent? What can I do to make future contributions acceptable? I'm not very familiar with the GNU development processes.

bug#69979: [PATCH-v2][doc] ls -l doc and size -> maj, min for device files

2024-03-24 Thread Pádraig Brady
On 24/03/2024 12:40, Stephane Chazelas wrote: Tags: patch My bad, the patch was incorrect, it should have said "replaced by the corresponding device major and minor numbers as two decimal numbers separated by a comma and at least one space.", as there's not always only once space between the

Re: new snapshot available: coreutils-9.4.170-7b206.tar.xz

2024-03-23 Thread Pádraig Brady
On 23/03/2024 17:47, Sam James wrote: Pádraig Brady writes: We plan to release coreutils-9.5 in the coming week so any testing you can do on various different systems between now and then would be most welcome. This release comes about 30 weeks after the 9.4 release. I hit this build

new snapshot available: coreutils-9.4.170-7b206.tar.xz

2024-03-23 Thread Pádraig Brady
ln,mv: improve dir vs nondir diagnostics mv: treat --exchange more like non-exchange Petr Malat (1): mv: add --swap (-x) option to atomically swap 2 paths Pádraig Brady (59): maint: post-release administrivia maint: avoid syntax check failure doc: add subsections for ck

bug#69951: coreutils: printf formatting bug for nb_NO and nn_NO locales

2024-03-23 Thread Pádraig Brady
tag 69951 notabug close 69951 stop On 22/03/2024 20:22, Thomas Dreibholz wrote: Hi, I just discovered a printf bug for at least the nb_NO and nn_NO locales when printing numbers with thousands separator. To reproduce: #!/bin/bash for l in de_DE nb_NO ; do    echo "LC_NUMERIC=$l.UTF-8"   

Re: sc_strftime_check: adjust to newer info quoting style

2024-03-23 Thread Pádraig Brady
On 23/03/2024 10:18, Bernhard Voelker wrote: On my openSUSE:Tumbleweed system, the info pages seem to have changed the quoting style for the strftime for the conversion specifiers from the good old '%a' or `%a' styles to ‘%a’. $ info libc date calendar format | grep '^[^a-z%]*%.[^a-z%]*$'

bug#69939: test utility bug: "test -a -a -a" and "test -o -o -o" fail

2024-03-22 Thread Pádraig Brady
On 22/03/2024 11:20, Vincent Lefevre wrote: With GNU Coreutils 9.4, both "test -a -a -a" and "test -o -o -o" fail: $ export POSIXLY_CORRECT=1 $ /usr/bin/test -a -a -a ; echo $? /usr/bin/test: ‘-a’: unary operator expected 2 $ /usr/bin/test -o -o -o ; echo $? /usr/bin/test: ‘-o’: unary operator

bug#69807: questioning automatic -i in multicolumn pr

2024-03-21 Thread Pádraig Brady
for the inconvenient tabs, e.g. post-processing with pr -t -e. Good call on the documentation. I'll add this now: commit 91e69cd2d02f015fc296e02388e0b18a293faa56 (HEAD -> master) Author: Pádraig Brady Date: Thu Mar 21 15:26:48 2024 + doc: pr: give solution to expanding T

Re: env: follow up on argv0 setting feature

2024-03-21 Thread Pádraig Brady
On 20/03/2024 23:19, Pádraig Brady wrote: On 13/03/2024 10:19, Matheus Afonso Martins Moreira wrote: About a year ago, I posted an env feature request on this list: the ability to set the value of argv[0]. https://lists.gnu.org/archive/html/coreutils/2023-03/msg2.html I also sent a patch

Re: env: follow up on argv0 setting feature

2024-03-20 Thread Pádraig Brady
On 13/03/2024 10:19, Matheus Afonso Martins Moreira wrote: About a year ago, I posted an env feature request on this list: the ability to set the value of argv[0]. https://lists.gnu.org/archive/html/coreutils/2023-03/msg2.html I also sent a patch:

bug#10311: RFE: Give chmod a "-h" option as well

2024-03-20 Thread Pádraig Brady
On 16/12/2011 16:29, Jan Engelhardt wrote: Hi, chown(1) has a -h option by which it affects symlinks directly rather than the pointed-to file. The bonus side effect is that the pointed-to files don't get changed in any way, which is kinda welcome if you attempt to "fix" permissions/ownership in

bug#11108: [PATCH] chmod: fix symlink race condition

2024-03-20 Thread Pádraig Brady
On 28/03/2012 21:28, Paul Eggert wrote: On 03/28/2012 01:13 PM, Jim Meyering wrote: $ ./chmod u+w f ./chmod: changing permissions of 'f': Operation not supported Yeouch. I undid the change for now. Hmm, why did "make check" work for me? I'll have to investigate later, alas. Patch

Re: Symlink flag for chmod

2024-03-19 Thread Pádraig Brady
On 18/03/2024 18:31, Pádraig Brady wrote: On 25/01/2024 14:13, Pádraig Brady wrote: On 25/01/2024 12:30, Johannes Segitz wrote: Hello, chown has a flag that prevents symlink following. chown/chmod is sometimes used in %post/%pre sections of rpm packages to fix up permissions. When

[PATCH] maint: basenc: consistently check buffer bounds when encoding

2024-03-19 Thread Pádraig Brady
* src/basenc.c (base16_encode, base2msbf_encode, base2lsbf_encode): Ensure we don't overflow the output buffer, whose length is passed in the OUTLEN parameter. This issue was flagged by clang with -Wunused-but-set-parameter. --- src/basenc.c | 10 +++--- 1 file changed, 7 insertions(+), 3

Re: Bug#628815: coreutils: pinky makes crazy DNS queries

2024-03-19 Thread Pádraig Brady
On 19/03/2024 10:54, deb...@perkelt.hu wrote: I looked into this and found that pinky tries to canonicalize the information in the "Where" column automaticaly, and it has no option to disable this behaviour. see line 285 pinky.c: if (*ut_host) /* See if we can

[PATCH] doc: fix stale --dereference info for chown/chgrp

2024-03-18 Thread Pádraig Brady
Following v5.2.1-679-g7e29ef8b8 symlinks specified on the command line no longer induce an error if lchown() is not supported on the system. * doc/coreutils.texi (chown invocation, chgrp invocation): Adjust accordingly, and also use a macro to avoid duplication. * src/chown-core.c: Use our more

Re: Symlink flag for chmod

2024-03-18 Thread Pádraig Brady
On 25/01/2024 14:13, Pádraig Brady wrote: On 25/01/2024 12:30, Johannes Segitz wrote: Hello, chown has a flag that prevents symlink following. chown/chmod is sometimes used in %post/%pre sections of rpm packages to fix up permissions. When this is done in user owned directories (somewhere

bug#69532: mv's new -x option should be made orthogonal to -t/-T/default

2024-03-17 Thread Pádraig Brady
On 17/03/2024 11:32, Pádraig Brady wrote: On 17/03/2024 06:10, Paul Eggert wrote: On 2024-03-05 06:16, Pádraig Brady wrote: I think I'll remove the as yet unreleased mv --swap from coreutils, given that util-linux is as widely available as coreutils on GNU/Linux platforms. Although removing

bug#69532: mv's new -x option should be made orthogonal to -t/-T/default

2024-03-17 Thread Pádraig Brady
On 17/03/2024 06:10, Paul Eggert wrote: On 2024-03-05 06:16, Pádraig Brady wrote: I think I'll remove the as yet unreleased mv --swap from coreutils, given that util-linux is as widely available as coreutils on GNU/Linux platforms. Although removing that "mv --swap" implementation

  1   2   3   4   5   6   7   8   9   10   >