[TC 8B4 32] [win XP] file pack with move to zip stopped

Bug reports will be moved here when the described bug has been fixed

Moderators: white, Hacker, petermad, Stefan2

Post Reply
charly_35
Junior Member
Junior Member
Posts: 21
Joined: 2010-03-26, 17:14 UTC
Location: France Rennes 35

[TC 8B4 32] [win XP] file pack with move to zip stopped

Post by *charly_35 »

hello,
When I use Alt-F5 to pack multiple directories with "move option" and "in a separate file each", if one file is read only under one of the directory the process stop without doing the next one.

Is this a bug or limitation ?

Also when I do the same with a single file, it is possible to do it in background but it seems it's not possible with multiple selection.

Cdlt Charles.
Cdlt Charles.
User avatar
petermad
Power Member
Power Member
Posts: 14914
Joined: 2003-02-05, 20:24 UTC
Location: Denmark
Contact:

Post by *petermad »

I can confirm this for TC 8.0 bot 64 and 32 bit.
I get this error message: Could not delete one or more of the files - they might be write protected!"

It is the same in TC 7.56a.

This is for zip files .

Some other file types behaves like this:

7z (packed with Total7zip.wcx) just moves everything also the read-only file.

7z (packed with 7zip.wcx) moves all files except the read-only file which is left in the original dir.

sqx (packed with TotalSQX.wcx) moves all files except the read-only file which is left in the original dir AND gives a message that "One or more directories have not been deleted" befor it continues with the next directory.

rar (packed with DOS rar.exe 3.90) just moves everything.

arj (packed with DOS arj.exe 2.85 ) moves all files except the read-only file, but als leaves all subdirs (without their files) in the original dir!


I would say that the way TotalSQX.wcx handles it is the best approach.
License #524 (1994)
Danish Total Commander Translator
TC 11.03 32+64bit on Win XP 32bit & Win 7, 8.1 & 10 (22H2) 64bit, 'Everything' 1.5.0.1371a
TC 3.50 on Android 6 & 13
Try: TC Extended Menus | TC Languagebar | TC Dark Help | PHSM-Calendar
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 48232
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

The delete error is treated as a fatal error, so the entire operation is aborted.
Author of Total Commander
https://www.ghisler.com
charly_35
Junior Member
Junior Member
Posts: 21
Joined: 2010-03-26, 17:14 UTC
Location: France Rennes 35

Post by *charly_35 »

Could it be changed as a warning ?
Cdlt Charles.
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 48232
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

Currently I'm too busy to rewrite the pack functions, it would require some major changes in the error passing. Maybe in a later release.
Author of Total Commander
https://www.ghisler.com
User avatar
petermad
Power Member
Power Member
Posts: 14914
Joined: 2003-02-05, 20:24 UTC
Location: Denmark
Contact:

Post by *petermad »

Any progress in solving this ?
License #524 (1994)
Danish Total Commander Translator
TC 11.03 32+64bit on Win XP 32bit & Win 7, 8.1 & 10 (22H2) 64bit, 'Everything' 1.5.0.1371a
TC 3.50 on Android 6 & 13
Try: TC Extended Menus | TC Languagebar | TC Dark Help | PHSM-Calendar
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 48232
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

Sorry, no time yet, it would require too many changes to the pack functions.
Author of Total Commander
https://www.ghisler.com
rdss
Junior Member
Junior Member
Posts: 7
Joined: 2011-12-21, 09:15 UTC

Post by *rdss »

I've been packing thousands of directories, with many random read-only files. I can only hope you keep doing the excellent work you are doing with TC and will also have some time to add an option to ignore the read-only error and continue.
BTW: great job converting from Delphi to FPC/Lazarus! :)
rdss
Junior Member
Junior Member
Posts: 7
Joined: 2011-12-21, 09:15 UTC

Post by *rdss »

Woo hoo!!!!
Here is a quote from the history file. Downloaded the beta even before the link was posted on the main page. I will test it out:
"05.01.12 Fixed: Internal ZIP and TAR packer: Do not stop packing when moving files to archives, one archive per file/folder, and at least one file is write protected. Instead, report error after packing all files (32/64)"

Thank you, Christian Ghisler!
User avatar
petermad
Power Member
Power Member
Posts: 14914
Joined: 2003-02-05, 20:24 UTC
Location: Denmark
Contact:

Post by *petermad »

Confirmed fixed in TC 8.0b15 :-)
License #524 (1994)
Danish Total Commander Translator
TC 11.03 32+64bit on Win XP 32bit & Win 7, 8.1 & 10 (22H2) 64bit, 'Everything' 1.5.0.1371a
TC 3.50 on Android 6 & 13
Try: TC Extended Menus | TC Languagebar | TC Dark Help | PHSM-Calendar
User avatar
ghisler(Author)
Site Admin
Site Admin
Posts: 48232
Joined: 2003-02-04, 09:46 UTC
Location: Switzerland
Contact:

Post by *ghisler(Author) »

Thanks!
Author of Total Commander
https://www.ghisler.com
Post Reply