Review Request 122897: Fix 7z zip not listing files after adding on new created archive.

Giuseppe Silletti sillettig at gmail.com
Mon Mar 21 05:15:12 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122897/
-----------------------------------------------------------

(Updated March 21, 2016, 5:15 a.m.)


Status
------

This change has been discarded.


Review request for KDE Utils and Raphael Kubo da Costa.


Repository: ark


Description
-------

Ark doesn't access to a new 7z archive properly. I've made the substituteListVariables function in cliinterface.cpp virtual, and then added it in 7z cliplugin.cpp. In this file i've overridden  substituteListVariables, and then i've appended the .7z extension to the filename() function. So now ark can find the right path of 7z archives.


Diffs
-----

  kerfuffle/cliinterface.h b4d79ed 
  plugins/cli7zplugin/cliplugin.h 9f122e4 
  plugins/cli7zplugin/cliplugin.cpp 500f121 

Diff: https://git.reviewboard.kde.org/r/122897/diff/


Testing
-------

I've added some files in a new archive and it works.


Thanks,

Giuseppe Silletti

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20160321/4b43118f/attachment.html>


More information about the Kde-utils-devel mailing list