First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 194264
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Gentoo's Team for Core System packages <base-system@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Paul Bredbury <brebs@sent.com>
Add CC:
CC:
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
unzip-5.52-r1.diff unzip-5.52-r1.diff patch Paul Bredbury 2007-09-30 10:36 0000 812 bytes Details | Diff
unzip-5.52-r1.diff unzip-5.52-r1.diff patch Paul Bredbury 2007-10-03 09:04 0000 777 bytes Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 194264 depends on: 104315 Show dependency tree
Bug 194264 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


Description:   Opened: 2007-09-30 10:34 0000
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 From Paul Bredbury 2007-09-30 10:36:57 0000 -------
Created an attachment (id=132228) [edit]
unzip-5.52-r1.diff

------- Comment #2 From SpanKY 2007-09-30 15:15:59 0000 -------
(From update of attachment 132228 [edit])
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 From Paul Bredbury 2007-10-03 09:04:08 0000 -------
Created an attachment (id=132437) [edit]
unzip-5.52-r1.diff

Simple addition of CPPFLAGS, to make append-lfs-flags effective.

------- Comment #4 From SpanKY 2007-10-06 14:16:53 0000 -------
looks good, added to the tree ... thanks

First Last Prev Next    No search results available      Search page      Enter new bug