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

[Raphael Kubo da Costa]

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


Thanks a lot for the patch and sorry for being absent.
Does this mean Ark is currently unable to add/remove files from 7z archives, or is it that the file list is not updated after that is done? Are you using the new p7zip (9.38.1)?

- Raphael Kubo da Costa


On March 11, 2015, 12:17 a.m., Giuseppe Silletti wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122897/
> -----------------------------------------------------------
> 
> (Updated March 11, 2015, 12:17 a.m.)
> 
> 
> 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/20150317/e82c97f6/attachment.html>