Review Request 124165: Make KArchive preserve permissions from the files inside the archive

[David Faure]

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


Preserving the executable flag sounds like a good idea indeed, but I wouldn't want extracted files to be readonly just because the person who made the archive decided it to be so. Just like we don't preserve ownership either, it wouldn't make sense, across systems.

I think this patch should be reduced to "making executable if it was executable".

- David Faure


On June 24, 2015, 5 p.m., Romário Rios wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124165/
> -----------------------------------------------------------
> 
> (Updated June 24, 2015, 5 p.m.)
> 
> 
> Review request for KDE Frameworks.
> 
> 
> Repository: karchive
> 
> 
> Description
> -------
> 
> The KArchiveFile::copyTo() wasn't setting the permissions of the files it copied -- it just copied the data. So, for instance, if an archive contained an executable, it was extracted from the archive as a non-executable. This patch makes it apply the permission information found in the KArchiveFile to the QFile being copied.
> 
> 
> Diffs
> -----
> 
>   src/karchive.cpp 344cba8 
> 
> Diff: https://git.reviewboard.kde.org/r/124165/diff/
> 
> 
> Testing
> -------
> 
> Seems to work for all files (tested with xz and zip files containing executables, directories, text, etc.).
> 
> 
> Thanks,
> 
> Romário Rios
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150625/b0c57f2a/attachment.html>