After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 583499 - Decompression of cbz files with p7zip
Decompression of cbz files with p7zip
Status: RESOLVED FIXED
Product: evince
Classification: Core
Component: backends
git master
Other All
: Normal enhancement
: ---
Assigned To: Evince Maintainers
Evince Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-05-21 22:08 UTC by Juanjo Marín
Modified: 2011-07-01 19:33 UTC
See Also:
GNOME target: ---
GNOME version: Unversioned Enhancement


Attachments
Decompression of cbz files with p7zip. Fixes #583499 (2.08 KB, patch)
2009-05-21 22:36 UTC, Juanjo Marín
none Details | Review
fallback mode for cbz files using 7za and 7z (1.25 KB, patch)
2011-06-30 14:55 UTC, Juanjo Marín
none Details | Review

Description Juanjo Marín 2009-05-21 22:08:57 UTC
Until now evince decompress cbz files with unzip. It'd be nice add support for decompressing cbz files with p7zip commands.
Comment 1 Juanjo Marín 2009-05-21 22:36:43 UTC
Created attachment 135142 [details] [review]
Decompression of cbz files with p7zip. Fixes #583499

Also "fixes" some indentation problems of the adjacent lines
Comment 2 Nickolay V. Shmyrev 2009-05-22 03:02:18 UTC
Why exactly it would be nice?
Comment 3 Juanjo Marín 2009-05-22 06:40:03 UTC
Sorry, I think I wrote a stupid bug and not very "nice" report :)

What I mean is you can decompress a cbz file without unzip, just using p7zip. Right now, p7zip is used only for decompressing cb7 files; but you can decompress cbz files too if you don't have unzip installed. This way, you give more options to the users and less problems.
Comment 4 Juanjo Marín 2009-05-22 09:24:36 UTC
s/bug and not very "nice" report/and not very "nice" bug report
Comment 5 Nickolay V. Shmyrev 2009-05-23 23:22:29 UTC
I can hardly imagine the environment without unzip. I think we can leave is as is  without complicating the code with more execution branches.
Comment 6 Carlos Garcia Campos 2011-06-19 09:39:02 UTC
Review of attachment 135142 [details] [review]:

So, would this work if you have unzip installed but not 7z?
Comment 7 Juanjo Marín 2011-06-19 11:48:59 UTC
The idea was for using p7zip for opening zip files (cbz) in case you don't have unzip as a second option. If you think we should add this option, I can check the patch in master.
Comment 8 Carlos Garcia Campos 2011-06-19 14:33:58 UTC
if it's used as a fallback when unzip is not installed, it's ok to me.
Comment 9 Juanjo Marín 2011-06-30 14:52:31 UTC
Carlos,

Here you are the git formatted patch
Comment 10 Juanjo Marín 2011-06-30 14:55:33 UTC
Created attachment 191037 [details] [review]
fallback mode for cbz files using 7za and 7z
Comment 11 Juanjo Marín 2011-07-01 19:33:03 UTC
I pushed the patch to master