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

(Updated June 25, 2015, 12:44 p.m.)


Review request for KDE Frameworks.


Changes
-------

Changing patch as requested by dfaure


Summary (updated)
-----------------

Make KArchive preserve executable permissions from the files inside the archive


Repository: karchive


Description (updated)
-------

The KArchiveFile::copyTo() doesn't set the permissions of the files it copies 
-- it just copies the data. So if an archive contains an executable, it's 
extracted from the archive as a non-executable. This patch makes it apply the 
executable permissions found in the KArchiveFile to the QFile being copied.


Diffs (updated)
-----

  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

_______________________________________________
Kde-frameworks-devel mailing list
Kde-frameworks-devel@kde.org
https://mail.kde.org/mailman/listinfo/kde-frameworks-devel

Reply via email to