Skip to content

[ci] Add Debian testing (with builtin_llvm=OFF) - #23372

Draft
ferdymercury wants to merge 6 commits into
root-project:masterfrom
ferdymercury:patch-24
Draft

ferdymercury wants to merge 6 commits into
root-project:masterfrom
ferdymercury:patch-24

Conversation

@ferdymercury

@ferdymercury ferdymercury commented Sep 15, 2026 •

Copy link
Copy Markdown
Collaborator

This Pull request:

Changes or fixes:

Fedora CI has rawhide, So Debian CI has testing for analogy. But no need to merge this, it can stay as a draft PR for individual testing of new packages. New package to test will be libmathtext-dev.

It's like Debian13, but tests the new libcivetweb-dev package, so builtin_civetweb can be OFF. fyi @linev confirmed that new Debian civetweb will be compatible.

First results not too bad: only 13 tests failing with builtin_llvm=OFF

Merge first

Checklist:

  • tested changes locally
  • updated the docs (if necessary)

@ferdymercury

ferdymercury commented Sep 15, 2026 •

Copy link
Copy Markdown
Collaborator Author

@alja @osschar maybe you know why debiantesting CI build fails with a heap error? https://github.com/root-project/root/actions/runs/34956369673/job/104339538592?pr=23372

@ferdymercury ferdymercury added the skip code analysis Skip the code analysis CI steps for this PR, including verifying clang-formatting and running Ruff. label Sep 15, 2026
@ferdymercury ferdymercury reopened this Sep 16, 2026
@github-actions

github-actions Bot commented Sep 16, 2026 •

Copy link
Copy Markdown

Test Results

    23 files      23 suites   4d 20h 40m 45s ⏱️
 3 875 tests  3 852 ✅ 0 💤 23 ❌
80 073 runs  80 050 ✅ 0 💤 23 ❌

For more details on these failures, see this check.

Results for commit fae3603.

♻️ This comment has been updated with latest results.

@ferdymercury
ferdymercury force-pushed the patch-24 branch 2 times, most recently from b4684d8 to 9af027a Compare September 16, 2026 07:34
@ferdymercury ferdymercury changed the title [ci] Add Debian testing [ci] Add Debian testing (with builtin_llvm=OFF) Sep 16, 2026
@ferdymercury
ferdymercury marked this pull request as ready for review September 16, 2026 08:49
It's like Debian13, but tests the new libcivetweb-dev package, so builtin_civetweb can be OFF.

[ci] disable pyspark on Debiantesting

CMake Error at cmake/modules/FindPySpark.cmake:34 (message):
    Currently, there are no Spark versions that support Java version greater
    than 21.  Found Java version 25.0.4.1.
  Call Stack (most recent call first):
    cmake/modules/SearchInstalledSoftware.cmake:1259 (find_package)
    CMakeLists.txt:131 (include)
rror: /github/home/ROOT-CI/src/roottest/root/tree/selectorreader/runCollections.C:169:37: error: invalid operands to binary expression ('iterator' (aka '__gnu_cxx::__normal_iterator<std::basic_string<char, std::char_traits<char>, std::allocator<char> > *, vector<std::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::basic_string<char, std::char_traits<char>, std::allocator<char> > > > >') and 'iterator')
     for (std::string const &treeName : trees)
                                      ^
  /github/home/ROOT-CI/build/include/TString.h:56:9: note: candidate function not viable: no known conversion from 'iterator' (aka '__gnu_cxx::__normal_iterator<std::basic_string<char, std::char_traits<char>, std::allocator<char> > *, vector<std::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::basic_string<char, std::char_traits<char>, std::allocator<char> > > > >') to 'const TString' for 1st argument
  Bool_t  operator==(const TString &s1, const char *s2);
          ^          ~~~~~~~~~~~~~~~~~
to figure out crash with libCling
@ferdymercury

Copy link
Copy Markdown
Collaborator Author

@guitargeek in TMVA-SOFIE I see:

/usr/include/c++/16/bits/stl_bvector.h:1158: constexpr std::vector<bool, _Alloc>::reference std::vector<bool, _Alloc>::operator[](size_type) [with _Alloc = std::allocator<bool>; reference = std::vector<bool>::reference; size_type = long unsigned int]: Assertion '__n < this->size()' failed.

This branch has not been deployed

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

Labels

pr:squash on merge skip code analysis Skip the code analysis CI steps for this PR, including verifying clang-formatting and running Ruff.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant