Skip to content

Migrate xz-embedded to vcpkg#4734

Open
xmkg wants to merge 5 commits intomainfrom
enhancement/move-xz-embedded-to-vcpkg
Open

Migrate xz-embedded to vcpkg#4734
xmkg wants to merge 5 commits intomainfrom
enhancement/move-xz-embedded-to-vcpkg

Conversation

@xmkg
Copy link
Member

@xmkg xmkg commented Mar 16, 2026

Description

  • What does this PR do?
    Move xz-embedded to vcpkg by adding a minimal portfile for it. The portfile is required bc vcpkg has no official port for it.
  • Why is this change needed?
    It's needed for the "streamlining dependencies" initiative we have.

MULTI-2342

xmkg added 4 commits March 16, 2026 16:47
Signed-off-by: Mustafa Kemal Gilor <mustafa.gilor@canonical.com>
xz-embedded does not have an official vcpkg port yet. This patch adds
a minimal vcpkg port for our use-case.

Signed-off-by: Mustafa Kemal Gilor <mustafa.gilor@canonical.com>
Signed-off-by: Mustafa Kemal Gilor <mustafa.gilor@canonical.com>
Signed-off-by: Mustafa Kemal Gilor <mustafa.gilor@canonical.com>
@xmkg xmkg requested review from a team and jimporter and removed request for a team March 16, 2026 14:27
Signed-off-by: Mustafa Kemal Gilor <mustafa.gilor@canonical.com>
@xmkg xmkg force-pushed the enhancement/move-xz-embedded-to-vcpkg branch from 0e653d3 to 94e321f Compare March 16, 2026 14:40
@codecov
Copy link

codecov bot commented Mar 16, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.65%. Comparing base (dd0b47c) to head (94e321f).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #4734   +/-   ##
=======================================
  Coverage   87.65%   87.65%           
=======================================
  Files         258      258           
  Lines       14124    14124           
=======================================
  Hits        12379    12379           
  Misses       1745     1745           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sharder996
Copy link
Collaborator

Any idea how hard it would be to upstream this? Just out of curiosity 😄

@xmkg
Copy link
Member Author

xmkg commented Mar 17, 2026

Any idea how hard it would be to upstream this? Just out of curiosity 😄

That already crossed my mind, too. It would require a much more elaborate portfile, but we could try.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants