https://bugs.kde.org/show_bug.cgi?id=440135

--- Comment #11 from Elvis Angelaccio <elvis.angelac...@kde.org> ---
Git commit b5ee900e2caa47e93d8e490500b6859809f4a6bd by Elvis Angelaccio.
Committed on 21/05/2022 at 13:17.
Pushed by elvisangelaccio into branch 'release/22.04'.

cli7z: conditionally add the -l switch

This commit partly reverts commit 35b397c84fdc7c46a98d02aeca13d1cee1facf6e.

The -l switch is actually needed for p7zip, as shown by the failing
addtest/copytest, since `CliInterface::addFiles` creates a symlink and p7zip
doesn't follow symlinks without the -l switch.

Since we need to add the -l switch conditionally, depending on the
actual 7z binary variant available at runtime, we need to actually check
which 7z variant is available before defining the switches for the add
operation.

This fixes all the unit tests with both p7zip and upstream 7-Zip.

M  +5    -1    autotests/plugins/cli7zplugin/cli7ztest.cpp
M  +25   -1    plugins/cli7zplugin/cliplugin.cpp
M  +6    -0    plugins/cli7zplugin/cliplugin.h

https://invent.kde.org/utilities/ark/commit/b5ee900e2caa47e93d8e490500b6859809f4a6bd

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to