Junio C Hamano <gitster@xxxxxxxxx> wrote:
> Eric Wong <e@xxxxxxxxx> writes:
> > Junio C Hamano <gitster@xxxxxxxxx> wrote:
> >> +test_lazy_prereq UNZIP_AUTOTEXT '
> >> + (
> >> + mkdir unzip-autotext &&
> >> + cd unzip-autotext
> >> + "$GIT_UNZIP" -a "$TEST_DIRECTORY"/t5003/infozip-text.zip &&
> >> + test -f text
> >> + )
> >
> > /usr/bin/unzip actually takes -a on FreeBSD, just not in the
> > same way the Info-ZIP version does, so I suspect "test -f"
> > here is not enough.
>
> Hmph. So it only and always does "CRLF -> LF", while Info-ZIP
> version does something like autocrlf?
No, it does CRLF -> LF based on the absence of non-ASCII
characters and ignoring metadata set in the zipfile.
The unzip manpage states:
Normally, the -a option should only affect files which are
marked as text files in the zipfile's central directory. Since
the archive(3) library reads zipfiles sequentially, and does
not use the central directory, that information is not available
to the unzip utility. Instead, the unzip utility will assume
that a file is a text file if no non-ASCII characters are
present within the first block of data decompressed for that
file. If non-ASCII characters appear in subsequent blocks of
data, a warning will be issued.
https://www.freebsd.org/cgi/man.cgi?query=unzip&sektion=1&manpath=FreeBSD+10.3-RELEASE
> Heh. It was created like so:
>
> $ printf 'text\r\n' >text && zip -ll infozip-text.zip text
> $ zipinfo infozip-text.zip text
> -rw-r----- 3.0 unx 5 tx stor 16-Jul-18 13:12 text
>
Thanks, but I think the test file is too small. I tried
setting up a test to store the text file as binary in the
zip to check for inadvertant CRLF conversions:
printf 'text\r\n' >binary && zip -B infozip-binary.zip binary
But zip -B/--binary only works on VM/CMS and MVS...
So I'm inclined to go with Dscho's patch.
(apologies for messing up René's name in my previous email;
on my new FreeBSD setup: mutt displays it fine, as does vim when
taking it from .mailmap, but something gets lost when mutt
populates the file for vim. Perhaps some Debian patch didn't
make it upstream...)
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html