Ticket #44 (closed enhancement: fixed)

Opened 16 years ago

Last modified 11 years ago

savannah: More functionally u7z

Reported by: sgh Owned by: slavazanko
Priority: major Milestone: 4.7.0-pre3
Component: mc-vfs Version: master
Keywords: Cc:
Blocked By: Blocking:
Branch state: merged Votes for changeset: committed-master

Description (last modified by ossi) (diff)

Original: http://savannah.gnu.org/bugs/?16303

Submitted by:SGh <sgh>Submitted on:Sat 08 Apr 2006 01:43:59 PM UTC
Category:VFSSeverity:3 - Normal
Status:NonePrivacy:Public
Assigned to:NoneOpen/Closed:Open
Release:current (CVS or snapshot)Operating System:GNU/Linux

Original submission:

I have mc CVS 2005-08-10-19 installed, I found, that I can't delete 
form 7z archives, and can't do some other operations (i.e. mc u7z 
cannot properly handle old 7z archives with ".\" at the beginning 
of filename)... So I wrote my own u7z version with all 
functionality, that provide p7zip archiver. Please include it to 
the mc source code.

Comment 1 by SGh <sgh> at Sat 08 Apr 2006 01:45:08 PM UTC:

P.S. Sorry for my bad english.

Comment 2 by Pavel Tsekov <ptsekov> at Thu 18 May 2006 01:51:52 PM UTC:

It would be better if you tweak the existing u7z script instead of 
providing a new one. I see the exisiting one has support for 
removing files from archives - why doesn't it work for you ? Does 
it print an error message of some kind ?

Comment 3 by SGh <sgh> at Tue 30 May 2006 07:52:33 PM UTC:

What's the difference if I've wrote my own version or changed the 
existent one? I've written it just a bit earlier before I've 
downloaded the CVS. I don't exactly remember but deleting wasn't 
worked in that CVS anyway. Seems like there was just an empty 
delete function.

Comment 4 by Leonard den Ottolander <leonardjo> at Wed 21 Jun 2006 05:35:56 PM UTC:

It's easier to see what changed (and if there are unwanted 
regressions) if you patch the existing version instead of writing a 
new version from scratch.

So if you want to get any improvements considered for inclusion you 
will have to rewrite your patch.

Comment 5 by SGh <sgh> at Wed 21 Jun 2006 07:15:35 PM UTC:

OK, Understand. I will download latest CVS, and see if I have 
something to patch un u7z.

Comment 6 by SGh <sgh> at Fri 23 Jun 2006 11:06:44 PM UTC:

Here is my patch to original u7z, fixed things:
1) Detection if we have 7z or only 7za installed. (With 7z we can 
work with all 7z-supported archives, i.e. CABs, etc... Only have to 
rename this script in ucab.)
2) Workaround for old version p7zip archives with prefix ./ in 
filenames.
3) Fixed directory creation in mcu7zip_mkdir, now it works properly.

Attachments

u7z-9742 (3.7 KB) - added by slavazanko 16 years ago.
added by sgh
u7z-9743 (3.3 KB) - added by slavazanko 16 years ago.
added by sgh
u7z.patch (1.8 KB) - added by slavazanko 16 years ago.
added by sgh

Change History

Changed 16 years ago by slavazanko

added by sgh

Changed 16 years ago by slavazanko

added by sgh

Changed 16 years ago by slavazanko

added by sgh

comment:1 Changed 16 years ago by styx

  • Milestone set to 4.7

comment:2 Changed 16 years ago by winnie

  • Status changed from new to accepted
  • Owner set to winnie
  • severity set to no branch

comment:3 Changed 16 years ago by angel_il

  • Milestone changed from 4.7 to 4.7.0-pre2

comment:4 Changed 15 years ago by slavazanko

  • Milestone changed from 4.7.0-pre2 to 4.7.0-pre3

comment:5 Changed 15 years ago by slavazanko

  • Owner changed from winnie to slavazanko

comment:6 Changed 15 years ago by slavazanko

  • Version set to master
  • severity changed from no branch to on review

As fact, attached patch already applied to master.

Branch 44_more_functionally_u7z created for some little bugfixies.

initial changeset:c057bff00c235a2c439c669979e1aee739065046

review, please.

comment:7 Changed 15 years ago by iNode

  • Votes for changeset set to iNode

comment:8 Changed 15 years ago by angel_il

  • Votes for changeset changed from iNode to iNode angel_il
  • severity changed from on review to approved

comment:9 Changed 15 years ago by slavazanko

  • Status changed from accepted to testing
  • Votes for changeset changed from iNode angel_il to commited-master
  • Resolution set to fixed
  • severity changed from approved to merged

comment:10 Changed 15 years ago by slavazanko

  • Status changed from testing to closed

comment:11 Changed 11 years ago by ossi

  • Description modified (diff)
  • Branch state set to no branch
  • Reporter changed from slavazanko to sgh

comment:12 Changed 11 years ago by andrew_b

  • Branch state changed from no branch to merged
Note: See TracTickets for help on using tickets.