[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#34488: Add sort --limit, or document workarounds for sort|head error
From: |
Pádraig Brady |
Subject: |
bug#34488: Add sort --limit, or document workarounds for sort|head error messages |
Date: |
Sun, 3 Mar 2019 16:07:39 -0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 |
On 03/03/19 15:36, Pádraig Brady wrote:
> On 25/02/19 22:35, Pádraig Brady wrote:
>> Thanks for doing all that.
>> I've attached a few changes:
>>
>> - spelling fixes
>> - usage() clarified/reordered
>> - ensure sigset_t are initialized
>> - Don't setprocmask() unless specified
>> - Simplified SETMASK_SIGNAL_OPTION handling
>> - The test missed `env` as a prerequisite
>> - The test was slow/spun cpu, so used sleep instead of seq
>> - Used $SHELL in case sh didn't support trap
>>
>> I see that the last signal that `kill -l` lists is not included.
>> I think we should be processing SIGNUM_BOUND also?
>
> An additional patch attached to replace --list-signal-actions
> with --list-ignored-signals. This is simpler, and more symmetric
> with the other options. Also the extra output was moot I think
> since handlers are reset upon exec
I've locally adjusted the NEWS/texi for that patch also.
I've a local patch to include SIGNUM_BOUND which I won't spam the list with.
Attached is a patch to remove the -p and --set-mask options,
which I've a light preference for.
I'll think some more about it.
cheers,
Pádraig
env-remove-shortcuts.patch
Description: Text Data