----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124165/ -----------------------------------------------------------
(Updated June 29, 2015, 9:39 a.m.) Review request for KDE Frameworks. Changes ------- Remove unnecessary casts Repository: karchive Description ------- 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) ----- autotests/karchivetest.cpp afc8387 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