On 20/02/2023 19:35, George Valkov wrote:
On 2023-02-20, at 7:49 PM, Pádraig Brady <p...@draigbrady.com> wrote:
On 20/02/2023 15:02, George Valkov wrote:
Hi Paul, the following tests fail after your patch:
FAIL: tests/split/filter.sh
FAIL: tests/split/b-chunk.sh
FAIL: tests/split/l-chunk.sh
These look unrelated and may be due to some other change in your environment.
Specifically lseek() is failing on /dev/null and returning:
split: /dev/null: cannot determine file size
I deleted the lines that were introduced by the patch in unistd.in.h, then make
clean; make -j 16 and ran all tests: back to 5 failed. Then I restored the
deleted lines, rebuild and I got 9 failed tests again.
Oh right sorry.
I think I see what's happening.
Since https://github.com/coreutils/gnulib/commit/4db8db34
gnulib has been unconditionally replacing lseek() on macos.
Now with SEEK_DATA undefined that replaced lseek()
will run the code intended for BeOS.
So either the lseek.m4 or lseek.c needs adjusting accordingly.
cheers,
Pádraig
