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

Author: jpnurmi

.github/actions/environment/action.yml

@@ -14,10 +14,16 @@ runs:
     # Android SDK Tools are arm64-incompatible
     # Android SDK Build Tools are musl-incompatible
     - name: Set Environment Variables for Alpine Linux
-      if: ${{ (runner.os == 'Linux' && runner.arch == 'ARM64') || contains(matrix.container, 'alpine') }}
+      if: ${{ (runner.os == 'Linux' && runner.arch == 'ARM64') || contains(matrix.container.image, 'alpine') }}
       shell: bash
       run: echo "NO_MOBILE=true" >> $GITHUB_ENV
 
+    # Allow sudoless Docker in integration tests
+    - name: Prepare Docker on Alpine Linux
+      if: ${{ contains(matrix.container.image, 'alpine') }}
+      shell: bash
+      run: sudo chmod 666 /var/run/docker.sock
+
     # zstd is needed for cross OS actions/cache but missing from windows-11-arm
     # https://github.com/actions/partner-runner-images/issues/99
     - name: Install zstd on Windows ARM64

.github/alpine/Dockerfile

@@ -8,7 +8,7 @@ RUN apk update
 RUN apk add bash build-base cmake curl git icu lsb-release-minimal sudo tar wget
 
 # sentry-native
-RUN apk add curl-dev libunwind-dev libunwind-static linux-headers openssl-dev zlib-dev xz-dev
+RUN apk add curl-dev docker-cli libunwind-dev libunwind-static linux-headers openssl-dev zlib-dev xz-dev xz-static
 
 # sentry-dotnet
 RUN apk add grpc-plugins openjdk11 powershell

.github/workflows/alpine.yml

@@ -3,6 +3,8 @@ name: Build Alpine Linux Docker image
 on:
   workflow_dispatch:
   push:
+    branches:
+      - main
     paths:
       - '.github/alpine/*'
 

.github/workflows/build.yml

@@ -11,9 +11,9 @@ on:
 
 jobs:
   build-sentry-native:
-    name: sentry-native (${{ matrix.container || matrix.os }})
+    name: sentry-native (${{ matrix.container.image || matrix.os }})
     runs-on: ${{ matrix.os }}
-    container: ${{ matrix.container }}
+    container: ${{ matrix.container.image }}
     strategy:
       fail-fast: false
       matrix:
@@ -23,7 +23,8 @@ jobs:
             target: Linux-arm64
           - os: ubuntu-latest
             target: Linux-musl
-            container: ghcr.io/getsentry/sentry-dotnet-alpine:3.21
+            container:
+              image: ghcr.io/getsentry/sentry-dotnet-alpine:3.21
           - os: macos-15 # Pin macos to get the version of Xcode that we need: https://github.com/actions/runner-images/issues/10703
           - os: windows-latest
           - os: windows-11-arm
@@ -59,7 +60,7 @@ jobs:
 
   build:
     needs: build-sentry-native
-    name: .NET (${{ matrix.container || matrix.os }})
+    name: .NET (${{ matrix.container.image || matrix.os }})
     runs-on: ${{ matrix.os }}
     container: ${{ matrix.container }}
 
@@ -73,7 +74,10 @@ jobs:
             target: Linux-arm64
           - os: ubuntu-latest
             target: Linux-musl
-            container: ghcr.io/getsentry/sentry-dotnet-alpine:3.21
+            container:
+              image: ghcr.io/getsentry/sentry-dotnet-alpine:3.21
+              volumes:
+                - /var/run/docker.sock:/var/run/docker.sock
           - os: macos-15 # Pin macos to get the version of Xcode that we need: https://github.com/actions/runner-images/issues/10703
           - os: windows-latest
           - os: windows-11-arm

CHANGELOG.md

@@ -14,6 +14,7 @@
 
 - Custom ISentryEventProcessors are now run for native iOS events ([#4318](https://github.com/getsentry/sentry-dotnet/pull/4318))
 - Crontab validation when capturing checkins ([#4314](https://github.com/getsentry/sentry-dotnet/pull/4314))
+- Native AOT: link to static `lzma` on Linux/MUSL ([#4326](https://github.com/getsentry/sentry-dotnet/pull/4326))
 
 ### Dependencies
 

src/Sentry/Platforms/Native/buildTransitive/Sentry.Native.targets

@@ -47,9 +47,8 @@
 
   <ItemGroup Condition="'$(FrameworkSupportsNative)' == 'true' and '$(RuntimeIdentifier)' == 'linux-musl-x64'">
     <DirectPInvoke Include="sentry-native" />
-    <!-- When musl is detected, static sentry-native links to static libunwind -->
-    <LinkerArg Include="-Wl,-Bstatic -Wl,--whole-archive -lunwind -Wl,--no-whole-archive -Wl,-Bdynamic" />
-    <NativeSystemLibrary Include="lzma" />
+    <!-- When musl is detected, static sentry-native links to static libunwind, which depends on liblzma -->
+    <LinkerArg Include="-Wl,-Bstatic -Wl,--whole-archive -lunwind -Wl,--no-whole-archive -llzma -Wl,-Bdynamic" />
     <NativeLibrary Include="$(MSBuildThisFileDirectory)..\sentry-native\linux-musl-x64\libsentry-native.a" />
   </ItemGroup>