-----------------------------------------------------------
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
> 
>

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

Reply via email to