Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix(Native AOT): link to static lzma on Linux/MUSL #4326

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jul 8, 2025
Merged

fix(Native AOT): link to static lzma on Linux/MUSL #4326

merged 2 commits into from
Jul 8, 2025

Conversation

jpnurmi
Copy link
Collaborator

@jpnurmi jpnurmi commented Jul 7, 2025 •
edited
Loading

This PR adds the necessary plumbing for using Docker from within the Alpine Linux Docker container by bind-mounting the runner host /var/run/docker.sock. This makes it possible to run the PublishContainer integration test for Alpine Linux, which revealed that static libunwind is not sufficient in minimal/chiseled AOT containers, but needs to be paired with static lzma.

Yet another fix to:

@jpnurmi
Copy link
Collaborator Author

jpnurmi commented Jul 7, 2025 •
edited
Loading

libunwind depends on liblzma, which is not available in chiseled AOT containers:

Building image 'hello-sentry' with tags 'latest' on top of base image 'mcr.microsoft.com/dotnet/nightly/runtime-deps:9.0-alpine-aot'.
  Pushed image 'hello-sentry:latest' to local registry via 'docker'.

Workload updates are available. Run `dotnet workload list` for more information.
Error loading shared library liblzma.so.5: No such file or directory (needed by /app/hello-sentry)
Error relocating /app/hello-sentry: lzma_stream_buffer_decode: symbol not found
Error relocating /app/hello-sentry: lzma_index_uncompressed_size: symbol not found
Error relocating /app/hello-sentry: lzma_index_end: symbol not found
Error relocating /app/hello-sentry: lzma_index_buffer_decode: symbol not found
Error relocating /app/hello-sentry: lzma_index_size: symbol not found
Error relocating /app/hello-sentry: lzma_stream_footer_decode: symbol not found

@devtekve
Copy link

devtekve commented Jul 7, 2025

@jpnurmi ping me if there's anything I can do to help. Thanks for jumping into action so quick <3. I am at work but I can take a look at whatever you need in about ~4 hours

@jpnurmi jpnurmi force-pushed the ci/alpine-docker branch from 2fd53cc to 8c047e2 Compare July 7, 2025 13:33
@jpnurmi jpnurmi changed the title ci: try installing docker on alpine fix(Native AOT): link to static lzma on Linux/MUSL Jul 7, 2025
@jpnurmi jpnurmi marked this pull request as ready for review July 7, 2025 15:12
@jpnurmi jpnurmi mentioned this pull request Jul 7, 2025
Closed
jamescrosswell
jamescrosswell approved these changes Jul 8, 2025
View reviewed changes
Copy link
Collaborator

@jamescrosswell jamescrosswell left a comment

Choose a reason for hiding this comment

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

LGTM

@jpnurmi jpnurmi merged commit baa279b into main Jul 8, 2025
32 checks passed
@jpnurmi jpnurmi deleted the ci/alpine-docker branch July 8, 2025 08:23
@jpnurmi jpnurmi mentioned this pull request Jul 8, 2025
Closed
This was referenced Jul 28, 2025
Closed
Open
Closed
Closed
Closed
Closed
Merged
Open
@dependabot dependabot bot mentioned this pull request Aug 4, 2025
Closed
This was referenced Aug 15, 2025
Closed
Closed
Closed
Merged
Closed
This was referenced Aug 27, 2025
Merged
Closed
@dependabot dependabot bot mentioned this pull request Sep 1, 2025
Merged
This was referenced Sep 8, 2025
Closed
Closed
Closed
Closed
This was referenced Sep 15, 2025
Closed
Open
Open
Open
Open
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jamescrosswell jamescrosswell jamescrosswell approved these changes

@Flash0ver Flash0ver Awaiting requested review from Flash0ver Flash0ver is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jpnurmi @devtekve @jamescrosswell