Bug 200422

Summary: archivers/liblz4: r129 breaks stdout pipelines.
Product: Ports & Packages Reporter: eborisch+FreeBSD
Component: Individual Port(s)Assignee: Thomas Zander <riggs>
Status: Closed FIXED    
Severity: Affects Some People CC: riggs
Priority: --- Flags: bugzilla: maintainer-feedback? (adamw)
Version: Latest   
Hardware: Any   
OS: Any   

Description eborisch+FreeBSD 2015-05-24 02:31:32 UTC
r129 of liblz4 enabled sparse file support by default; this breaks [1] lz4c decompression when used in a pipeline. For example,  <some compressed lz4 stream> | lz4c -d | <some other tool> isn't working now, but worked in r128.

I would suggest either reverting back to r128 for now, or applying the changes made to the current dev branch [2] which gracefully recovers rather than errors out. 

[1] https://github.com/Cyan4973/lz4/issues/110
[2] https://github.com/Cyan4973/lz4/commit/58b5aadb1fc5d937e81c8f33e0e8290f2097c6bb
Comment 1 commit-hook freebsd_committer 2015-05-24 18:01:33 UTC
A commit references this bug:

Author: riggs
Date: Sun May 24 18:01:05 UTC 2015
New revision: 387288
URL: https://svnweb.freebsd.org/changeset/ports/387288

Log:
  Fix decompression in a pipeline: import fix from dev branch in
  https://github.com/Cyan4973/lz4/commit/58b5aadb1fc5d937e81c8f33e0e8290f2097c6bb

  PR:		200422
  Submitted by:	eborisch+FreeBSD@gmail.com
  Approved by:	adamw (maintainer, implicit)

Changes:
  head/archivers/liblz4/Makefile
  head/archivers/liblz4/files/
  head/archivers/liblz4/files/patch-programs_Makefile
  head/archivers/liblz4/files/patch-programs_lz4cli.c
  head/archivers/liblz4/files/patch-programs_lz4io.c
Comment 2 Thomas Zander freebsd_committer 2015-05-24 18:05:55 UTC
Tested with the example command line from
https://github.com/Cyan4973/lz4/issues/110
on 10.1 and 9.3.