FS#15177 - [mc] stopped reading ZIP archives (with FIX)

Attached to Project: Arch Linux
Opened by Jaroslaw Swierczynski (swiergot) - Friday, 19 June 2009, 19:53 GMT
Last edited by Dan Griffiths (Ghost1227) - Saturday, 13 February 2010, 00:53 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Aaron Griffin (phrakture)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 4
Private No

Details

After upgrade of unzip mc no longer reads ZIP files. Patch attached.
   uzip.diff (0.7 KiB)
This task depends upon

Closed by  Dan Griffiths (Ghost1227)
Saturday, 13 February 2010, 00:53 GMT
Reason for closing:  Fixed
Additional comments about closing:  fixed in 4.7.0.2-2
Comment by Greg (dolby) - Friday, 19 June 2009, 21:57 GMT
Where is this patch from? Is it coming from upstream? If not is there a report upstream?
Comment by Jaroslaw Swierczynski (swiergot) - Friday, 19 June 2009, 22:38 GMT
It's mine. I'm yet to report it upstream.
Comment by Greg (dolby) - Friday, 19 June 2009, 22:56 GMT
You might wanna check the latest mc version first as its been out for almost 5 months.
And -git too before reporting.
Comment by Jaroslaw Swierczynski (swiergot) - Saturday, 20 June 2009, 08:30 GMT
Checked. The file has indeed changed but does not contain the fix. Attached is an improved version of the patch that works with both unzip versions. The patch is against mc-4.6.1. For patch against mc-4.6.2, please see:

http://www.midnight-commander.org/ticket/423
   uzip2.diff (1 KiB)
Comment by Alessandro Doro (adoroo) - Saturday, 20 June 2009, 12:42 GMT
A cleaner patch.
   uzip.diff (0.4 KiB)
Comment by Jaroslaw Swierczynski (swiergot) - Saturday, 20 June 2009, 12:58 GMT
Yeah, I've seen that patch in Debian's bugtracker. But it doesn't fix the problem, just makes the buggy code bypassed.
Comment by Nagy Gabor (combo) - Saturday, 05 September 2009, 15:38 GMT
my $op_has_zipinfo = 1; method works here.
Comment by Tom Killian (tomk) - Thursday, 21 January 2010, 21:00 GMT
Working in mc 4.7.0.1

Loading...