Review Request 124111: Support overwrite prompt of unrar v5

[Ragnar Thomsen]

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

Review request for KDE Utils, Elvis Angelaccio and Raphael Kubo da Costa.


Repository: ark


Description
-------

The overwrite prompt of unrar v5 changed compared to v3 and v4. This causes Ark to hang when extracting a rar archive with unrar v5 when a destination file exists. In unrar v5 the overwrite prompt looks like:

    Would you like to replace the existing file file1.txt
        32 bytes, modified on 15-05-2015 14:26
    with a new one
        32 bytes, modified on 15-05-2015 14:26
        
and in unrar v3 and v4:

    file1.txt already exists. Overwrite it ?

Ark currently only supports the v3/v4 prompt. This patch adds support for the v5 prompt by defining a QStringList containing regexps for both prompts and iterating through the QStringList in CliInterface::checkForFileExistsMessage().


Diffs
-----

  kerfuffle/cliinterface.cpp 2ce6a06 
  plugins/clirarplugin/cliplugin.cpp 489abd6 

Diff: https://git.reviewboard.kde.org/r/124111/diff/


Testing
-------

Extracting rar archives using both v4 and v5 unrar works as expected, also when destination exists. Extracting zip archives still works.


Thanks,

Ragnar Thomsen

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20150616/28b7f5fa/attachment.html>