Skip to content

[6.38] [cling][test] Fix failing cling tests in master#21273

Merged
devajithvs merged 4 commits intoroot-project:v6-38-00-patchesfrom
root-project-bot:BP_6.38_pull_21232
Feb 16, 2026
Merged

[6.38] [cling][test] Fix failing cling tests in master#21273
devajithvs merged 4 commits intoroot-project:v6-38-00-patchesfrom
root-project-bot:BP_6.38_pull_21232

Conversation

@dpiparo
Copy link
Member

@dpiparo dpiparo commented Feb 13, 2026

Backport of #21232, requested by @foo. For your information @devajithvs

When cling is built with C++17, the implementation in ValuePrinter.cpp
is not compiled. When cling is later used with -std=c++20, we don't have
the implementation to print `source_location`

(cherry picked from commit 2a20a68)
Fix the following assertion failure in cling:

```
[cling]$ #include <filesystem>
Transaction.cpp:180: void cling::Transaction::forceAppend(DelayCallInfo):
Assertion `oldDCI != DCI && "Duplicates?!"' failed.
```

(cherry picked from commit 43610ec)
LLVM relies on libedit for line editing. History and other input
functionalities might not work as expected without this.

(cherry picked from commit effafd1)
@github-actions
Copy link

Test Results

    19 files      19 suites   3d 9h 15m 31s ⏱️
 3 715 tests  3 715 ✅ 0 💤 0 ❌
68 853 runs  68 853 ✅ 0 💤 0 ❌

Results for commit ef1b246.

@devajithvs devajithvs self-requested a review February 16, 2026 09:13
@devajithvs devajithvs merged commit 95c284b into root-project:v6-38-00-patches Feb 16, 2026
29 checks passed
@dpiparo dpiparo deleted the BP_6.38_pull_21232 branch February 16, 2026 09:22
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