On 2025-05-03 at 04:16:51, Collin Funk wrote: > As documented on NetBSD's man page, open with the O_NOFOLLOW flag and a > symlink returns -1 and sets errno to EFTYPE which differs from POSIX. > > This patch fixes the following test failure: > > $ sh t0602-reffiles-fsck.sh --verbose > --- expect 2025-05-02 23:05:23.920890147 +0000 > +++ err 2025-05-02 23:05:23.916794959 +0000 > @@ -1 +1 @@ > -error: packed-refs: badRefFiletype: not a regular file but a symlink > +error: unable to open '.git/packed-refs': Inappropriate file type or format > not ok 12 - the filetype of packed-refs should be checked > > FreeBSD has the same issue for EMLINK instead of EFTYPE. > > This portability issue was introduced in cfea2f2da8 (packed-backend: > check whether the "packed-refs" is regular file, 2025-02-28) Yup, this looks good. Thanks again for the patch. I'll just add one resource for people who might like to look into these kinds of things more. https://man.freebsd.org/cgi/man.cgi is the FreeBSD man page viewer, which lets you view manual pages from the BSDs, Linux, and some proprietary Unix systems. It can be quite helpful for finding and fixing portability issues like this or just seeing what command-line options or arguments a certain Unix supports. -- brian m. carlson (they/them) Toronto, Ontario, CA
Attachment:
signature.asc
Description: PGP signature
