Skip to content

🐛 (patch) Fix test_package on Conan 2.33 and build packages with LLVM 23 - #18

Merged
kammce merged 1 commit into
mainfrom
fix-test-package-run
Oct 14, 2026
Merged

kammce merged 1 commit into
mainfrom
fix-test-package-run

Conversation

@kammce

@kammce kammce commented Oct 14, 2026 •

Copy link
Copy Markdown
Member

Changes

  • test_package: pass the binary path to self.run() as a string. Conan 2.33 rejects a pathlib.Path, so the test step failed.
  • CI and release package builds use LLVM 23 (llvm_profile: hal/tc/llvm-23, Conan 2.33.0), the same as libhal v5.

Why LLVM 23

Once the test step stopped failing early, test_package actually ran, and it hung on the macos-26 runner. It's built with ASan, and LLVM 20's ASan runtime deadlocks during initialization on macOS 26+. LLVM 23's ASan works there, so ASan stays on in test_package.

Testing

conan create . -pr:a hal/tc/llvm-23 on macOS 27 with Conan 2.33.0: the test_package builds and runs. CI on this PR covers macOS 26.

Conan 2.33 rejects a pathlib.Path as the command for self.run(), so the
test_package step failed. Convert the path with str().

With that fixed, test_package runs, and it hung on the macOS 26 runner:
it is built with ASan, and LLVM 20's ASan runtime deadlocks during
initialization on macOS 26+. Build the CI and release packages with
LLVM 23 (hal/tc/llvm-23, which needs Conan 2.33.0), whose ASan works
there, the same as libhal v5.
@kammce kammce changed the title 🐛 Pass test_package binary path to self.run() as a string 🐛 Fix test_package on Conan 2.33 and build packages with LLVM 23 Oct 14, 2026
@kammce
kammce force-pushed the fix-test-package-run branch from 9c83079 to 33db060 Compare October 14, 2026 20:24
@kammce kammce changed the title 🐛 Fix test_package on Conan 2.33 and build packages with LLVM 23 🐛 (patch) Fix test_package on Conan 2.33 and build packages with LLVM 23 Oct 14, 2026
@kammce
kammce merged commit f152e69 into main Oct 14, 2026
9 checks passed
@kammce
kammce deleted the fix-test-package-run branch October 14, 2026 20:35
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.

1 participant