Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add decoder compatibility argument #20687

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ismayilov-ismayil
Copy link
Contributor

@ismayilov-ismayil ismayilov-ismayil commented Nov 23, 2024

Task name: ArchiveFilesV2

Description: After upgrading 7zip to v2408 the compressed files are not compatible with older versions of 7zip (on linux).
Added new argument to make it compatible with older versions extracting.

Documentation changes required: (Y/N) Y

Added unit tests: (Y/N) N

Attached related issue: (Y/N) Y

Checklist:

  • Task version was bumped - please check instruction how to do it
  • Checked that applied changes work as expected

@ismayilov-ismayil ismayilov-ismayil requested a review from a team as a code owner November 23, 2024 18:06
@ismayilov-ismayil ismayilov-ismayil added enhancement Area: ABTT Akvelon Build Tasks Team area of work labels Nov 23, 2024
@@ -120,6 +120,11 @@ function sevenZipArchive(archive: string, compression: string, files: string[])
sevenZip.arg('-mx=' + mapSevenZipCompressionLevel(sevenZipCompression));
}

const decoderCompatibility = tl.getInput('decoderCompatibility', false);
if (win && decoderCompatibility && archiveType == '7z') {
Copy link
Contributor

@KonstantinTyukalov KonstantinTyukalov Nov 25, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By the PR description, should it be !win os something else as it's only for linux?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

linux uses old version to archive and extract, but windows uses new version decoder for both.
if we archive on windows (new) and extract on linux (old version) it fails.
thus, we provide new param on windows only (new version 7z), old versions (linux) doesn't have this argument

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: ABTT Akvelon Build Tasks Team area of work enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants