[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#22624: [bug-coreutils] coreutils-8.25: big success, but problem on G
From: |
Pádraig Brady |
Subject: |
bug#22624: [bug-coreutils] coreutils-8.25: big success, but problem on GNU/Hurd |
Date: |
Thu, 11 Feb 2016 20:13:27 -0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 11/02/16 09:43, Paul Eggert wrote:
> On 02/11/2016 08:10 AM, Nelson H. F. Beebe wrote:
>> end_offset=9223372036854775807
>>
>
> Thanks, that confirms my suspicions about GNU/Hurd. I'm attaching a
> proposed patch; please give it a try if you have a chance. Turned out to
> be trickier than I thought, but oh well.
Thanks for working on this.
The changes look good, except for this:
$ seq 1000 | split -n4
$ seq 100000 | split -n4
split: -: cannot determine file size: Illegal seek
I.E. it would be better to indicate immediately
if there is an issue determining the file size,
because it's a gotcha that may hit users as data increases,
and -n is complex enough anyway, that it's better to
do as much checking up front as possible.
I'd still disallow this case even for -n1 in case the
number was parameterized to number of CPUs or whatever.
A small point on the tests is that we use `returns_ 1 ... || fail=1`
rather than `... && fail=1` so that we catch seg faults etc. in tests.
thanks!
Pádraig
- bug#22624: [bug-coreutils] coreutils-8.25: big success, but problem on GNU/Hurd, Nelson H. F. Beebe, 2016/02/10
- bug#22624: [bug-coreutils] coreutils-8.25: big success, but problem on GNU/Hurd, Paul Eggert, 2016/02/10
- bug#22624: [bug-coreutils] coreutils-8.25: big success, but problem on GNU/Hurd, Bernhard Voelker, 2016/02/11
- bug#22624: [bug-coreutils] coreutils-8.25: big success, but problem on GNU/Hurd, Nelson H. F. Beebe, 2016/02/11
- bug#22624: [bug-coreutils] coreutils-8.25: big success, but problem on GNU/Hurd, Paul Eggert, 2016/02/11
- bug#22624: [bug-coreutils] coreutils-8.25: big success, but problem on GNU/Hurd,
Pádraig Brady <=
- bug#22624: [bug-coreutils] coreutils-8.25: big success, but problem on GNU/Hurd, Paul Eggert, 2016/02/12
- bug#22624: [bug-coreutils] coreutils-8.25: big success, but problem on GNU/Hurd, Pádraig Brady, 2016/02/13
bug#22624: [bug-coreutils] coreutils-8.25: big success, but problem on GNU/Hurd, Pádraig Brady, 2016/02/10