coreutils/src, branch v9.2Mirror of https://https.git.savannah.gnu.org/git/coreutils.git/
https://git.shady.money/coreutils/atom?h=v9.22023-03-19T22:25:50Zstty: ensure arbitrary data is not displayed2023-03-19T22:25:50ZPádraig BradyP@draigBrady.com2023-03-19T22:22:18Zurn:sha1:eb745e3b95a79bdad98aaa9c4e753145f995c313
* src/stty.c (main): Use static structures to ensure
they're initialized (to zero), so that random data is
not displayed, or compared resulting in a inaccurate
failure reported to users. This was seen on musl libc
where some parts of the termios c_cc array were
not initialized by tcgetattr().
Reported by Bruno Haible.
doc: uniq: revert previous man page adjustment2023-03-18T14:58:20ZPádraig BradyP@draigBrady.com2023-03-18T14:58:20Zurn:sha1:11f4d591383f4c89d8bf5970db703cbd0e4a174d
* src/uniq.c (usage): -D doesn't take an argument
so should be separated in the description.
doc: uniq: use more standard description of -D long option2023-03-18T11:44:24ZPádraig BradyP@draigBrady.com2023-03-18T11:35:46Zurn:sha1:8a759fdc8855a428b472ed4f921b5e327c915494
* src/uniq.c: Describe -D and --all-repeated together,
as with all other long options.
Fixes https://bugs.gnu.org/62249
maint: avoid discarded-qualifiers warnings with SELinux 3.52023-03-16T23:12:18ZBernhard Voelkermail@bernhard-voelker.de2023-03-16T22:11:47Zurn:sha1:f346fc01445b7192bb129ee12bd505d3ae07d777
Since SELinux version 3.5, the return value of context_str(3) is
declared as const; see:
https://github.com/SELinuxProject/selinux/commit/dd98fa322766
Therefore, GCC complains (here with -Werror):
src/selinux.c: In function 'defaultcon':
src/selinux.c:152:16: error: assignment discards 'const' qualifier \
from pointer target type [-Werror=discarded-qualifiers]
152 | if (!(constr = context_str (tcontext)))
| ^
src/selinux.c: In function 'restorecon_private':
src/selinux.c:252:16: error: assignment discards 'const' qualifier \
from pointer target type [-Werror=discarded-qualifiers]
252 | if (!(constr = context_str (tcontext)))
| ^
* src/selinux.c (defaultcon): Define CONSTR as const.
(restorecon_private): Likewise.
cksum: fix --raw on 64 bit big endian systems2023-03-15T14:05:38ZPádraig BradyP@draigBrady.com2023-03-15T13:57:37Zurn:sha1:a8e6e627f1a1142ec9cb273cfadc3f9aa2aa412a
* src/sum.c (output_bsd): On sparc64 for example,
a crc of 0 was output due to casting an int variable
to uint16_t and thus operating on the wrong end of the variable.
Instead use explicit assignment to the narrower type
to ensure we get the appropriate data.
(output_sysv): Likewise.
Reported by Bruno Haible.
build: avoid -Wsometimes-uninitialized on macOS 122023-03-15T13:15:11ZBruno Haiblebruno@clisp.org2023-03-15T13:15:11Zurn:sha1:f1ff021247304698ae71892060330bcb91683d70
* src/pr.c (integer_overflow): Mark as _Noreturn.
* src/test.c (beyond): Likewise.
tee: avoid undefined behavior after fclose()2023-03-13T21:26:21ZPádraig BradyP@draigBrady.com2023-03-13T21:26:21Zurn:sha1:73d372dcc166afc387b4d130ecc00e5dce1fc89a
* iopoll.c (fclose_wait): Rename from confusing fclose_nonblock name.
Also adjust to do no operations on the stream after fclose()
as this is undefined. Instead use fflush() to determine EAGAIN status.
(fwrite_wait): Renamed from confusing fwrite_nonblock name.
doc: dircolors: better separate the sections in the database2023-03-13T16:20:38ZPádraig BradyP@draigBrady.com2022-09-05T13:31:04Zurn:sha1:f26af5983313f1f4cf04143cecee68551ed6502f
* src/dircolors.hin: Make the separate sections of the self
documenting dircolors database more apparent,
by adding heading comments, and appropriate separation.
ls: --color: honor separate sequences for extension cases2023-03-13T16:20:38ZPádraig BradyP@draigBrady.com2022-09-04T18:59:25Zurn:sha1:47988fad885e8129f9dc36f0ed4d63375de23603
Following on from commit v8.29-45-g24053fbd8 which unconditionally
used case insensitive extension matching, support selective
case sensitive matching when there are separate extension cases
defined with different display sequences.
* src/dircolors.hin: Document how file name suffixes are matched.
Note this is displayed with `dircolors --print-database` which
the texi info recommends to use for details.
* src/ls.c (parse_ls_color): Postprocess the list to
mark entries for case sensitive matching,
and also adjust so that unmatchable entries are more quickly ignored.
(get_color_indicator): Use exact matching rather than
case insensitive matching if so marked.
* tests/ls/color-ext.sh: Add test cases.
* NEWS: Mention the change in behavior.
Addresses https://bugs.gnu.org/33123
tee: support non blocking outputs2023-03-08T18:32:05ZPádraig BradyP@draigBrady.com2023-03-05T15:51:32Zurn:sha1:16000805eb1bcdf25360471b8bbc8ec3f025e035
Non blocking outputs can be seen for example
when piping telnet through tee to a terminal.
In that case telnet sets its input to nonblocking mode,
which results in tee's output being nonblocking,
in which case in may receive an EAGAIN error upon write().
The same issue was seen with mpirun.
The following can be used to reproduce this
locally at a terminal (in most invocations):
$ { dd iflag=nonblock count=0 status=none;
dd bs=10K count=10 if=/dev/zero status=none; } |
tee || echo fail >/dev/tty
* src/iopoll.c (iopoll_internal): A new function refactored from
iopoll(), to also support a mode where we check the output
descriptor is writeable.
(iopoll): Now refactored to just call iopoll_internal().
(fwait_for_nonblocking_write): A new internal function which
uses iopoll_internal() to wait for writeable output
if an EAGAIN or EWOULDBLOCK was received.
(fwrite_nonblock): An fwrite() wrapper which uses
fwait_for_nonblocking_write() to handle EAGAIN.
(fclose_nonblock): Likewise.
src/iopoll.h: Add fclose_nonblock, fwrite_nonblock.
src/tee.c: Call fclose_nonblock() and fwrite_nonblock wrappers,
instead of the standard functions.
* tests/misc/tee.sh: Add a test case.
* NEWS: Mention the improvement.
The idea was suggested by Kamil Dudka in
https://bugzilla.redhat.com/1615467