FS#56676 - Add support for LZ4 compression

Attached to Project: Pacman
Opened by Pascal Ernster (hardfalcon) - Monday, 11 December 2017, 12:30 GMT
Last edited by Eli Schwartz (eschwartz) - Friday, 11 October 2019, 19:39 GMT
Task Type Feature Request
Category makepkg
Status Closed
Assigned To No-one
Architecture All
Severity Low
Priority Normal
Reported Version 5.0.1
Due in Version 5.2.0
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Please add support for LZ4 compression. libarchive/bsdtar support LZ4 since 3.1.900a / 3.2.0:

https://github.com/libarchive/libarchive/blob/master/NEWS

This would also be a nice (and faster) alternative to LZO.
This task depends upon

Closed by  Eli Schwartz (eschwartz)
Friday, 11 October 2019, 19:39 GMT
Reason for closing:  Implemented
Additional comments about closing:  https://git.archlinux.org/pacman.git/com mit/?id=0cbb128818726e4d41964ea8df8c42ee 37697ef7
Comment by Eli Schwartz (eschwartz) - Wednesday, 13 December 2017, 01:20 GMT
Hmm, we could do this as a followup to https://lists.archlinux.org/pipermail/pacman-dev/2017-October/022187.html and the associated zstd patches.

The issues is that this is already getting quite unwieldy and it only gets more so every time we add a new one...
Comment by Pascal Ernster (hardfalcon) - Wednesday, 13 December 2017, 06:43 GMT
Nice patch, but that doesn't cover repo-add yet (although it probably could be easily extended to do so).
Comment by Pascal Ernster (hardfalcon) - Wednesday, 13 December 2017, 06:50 GMT
For the record: Already now, repo-add doesn't work for LZO-compressed packages due to the "bsdtar -tqf" line throwing an error. I haven't filed a bug report about that because I suspect an upstream bug in libarchive, but haven't had the time yet to debug that issue properly, let alone to write a patch that fixes it.

Anyhow, the approach your patch takes seems like a much cleaner (and more flexible) solution when compared to the code currently used.

Loading...