Skip to content

[builtins] download instead of bundle xxhash and bump from 0.8.0 to 0.8.3#21806

Open
ferdymercury wants to merge 17 commits intoroot-project:masterfrom
ferdymercury:xxh
Open

[builtins] download instead of bundle xxhash and bump from 0.8.0 to 0.8.3#21806
ferdymercury wants to merge 17 commits intoroot-project:masterfrom
ferdymercury:xxh

Conversation

@ferdymercury
Copy link
Copy Markdown
Collaborator

No description provided.

@ferdymercury ferdymercury changed the title [skip-ci,builtins] download instead of bundle xxhash [builtins] download instead of bundle xxhash Apr 7, 2026
@dpiparo dpiparo self-assigned this Apr 7, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 7, 2026

Test Results

    22 files      22 suites   3d 6h 56m 29s ⏱️
 3 833 tests  3 832 ✅  1 💤 0 ❌
76 552 runs  76 534 ✅ 18 💤 0 ❌

Results for commit f94d4a9.

♻️ This comment has been updated with latest results.

add CMakeLists patch

rm bundled copy
@ferdymercury ferdymercury changed the title [builtins] download instead of bundle xxhash [builtins] download instead of bundle xxhash and bump from 0.8.0 to 0.8.3 Apr 7, 2026
@ferdymercury ferdymercury requested a review from hageboeck April 8, 2026 19:28
@ferdymercury ferdymercury requested a review from amadio April 10, 2026 07:00
@dpiparo
Copy link
Copy Markdown
Member

dpiparo commented Apr 10, 2026

thanks. I propose to also add builtin_xxhash to https://github.com/root-project/root/blob/master/.github/workflows/root-ci-config/buildconfig/alma10-clang_ninja.txt in order to test the changes on linux w/o resorting to the system library.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants