Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 194264 - app-arch/unzip-5.52-r1 fails with >2gb filesize
Summary: app-arch/unzip-5.52-r1 fails with >2gb filesize
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo's Team for Core System packages
URL: http://bugs.debian.org/cgi-bin/bugrep...
Whiteboard:
Keywords:
Depends on: 104315
Blocks:
  Show dependency tree
 
Reported: 2007-09-30 10:34 UTC by Paul Bredbury
Modified: 2007-10-06 14:16 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
unzip-5.52-r1.diff (unzip-5.52-r1.diff,812 bytes, patch)
2007-09-30 10:36 UTC, Paul Bredbury
Details | Diff
unzip-5.52-r1.diff (unzip-5.52-r1.diff,777 bytes, patch)
2007-10-03 09:04 UTC, Paul Bredbury
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Paul Bredbury 2007-09-30 10:34:28 UTC
Hi, unzip-5.52-r1 cannot handle filesizes >2gb, e.g. the 2.73gb rygel-dp-texturepack-ultra.pk3 from
http://www.quakeone.com/forums/showthread.php?t=2778

Ubuntu's diff:
http://archive.ubuntu.com/ubuntu/pool/main/u/unzip/unzip_5.52-10ubuntu1.diff.gz

Includes the following line:
+DEFINES = -DACORN_FTYPE_NFS -DWILD_STOP_AT_DIR -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64

And also contains:
+  * Added appropriate compiler flags for Large File Support (Closes: #192253).
+    This procedure is blessed by upstream in the FAQ, and as a result,
+    some .zip archives may now be uncompressed using Debian unzip.

The bug mentioned is:
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=192253

The FAQ is:
http://www.info-zip.org/FAQ.html#limits
Which specifies:
-DLARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64

"LARGEFILE" doesn't even exist in the source - all that is needed is:
-D_FILE_OFFSET_BITS=64

Enclosed is a patch to add >2gb filesize support. Works with the above file on x86. Properly fixes #104315.
Comment 1 Paul Bredbury 2007-09-30 10:36:57 UTC
Created attachment 132228 [details, diff]
unzip-5.52-r1.diff
Comment 2 SpanKY gentoo-dev 2007-09-30 15:15:59 UTC
Comment on attachment 132228 [details, diff]
unzip-5.52-r1.diff

umm, no ... if you read the ebuild, you'll see we already call append-lfs-flags

the fix is to make sure env CPPFLAGS are respected
Comment 3 Paul Bredbury 2007-10-03 09:04:08 UTC
Created attachment 132437 [details, diff]
unzip-5.52-r1.diff

Simple addition of CPPFLAGS, to make append-lfs-flags effective.
Comment 4 SpanKY gentoo-dev 2007-10-06 14:16:53 UTC
looks good, added to the tree ... thanks