Skip to content

Comments

"[6.38] [cling][test] Fix failing cling tests in master"#21265

Closed
dpiparo wants to merge 4 commits intoroot-project:v6-38-00-patchesfrom
root-project-bot:BP_6.38_pull_21232
Closed

"[6.38] [cling][test] Fix failing cling tests in master"#21265
dpiparo wants to merge 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)
@dpiparo
Copy link
Member Author

dpiparo commented Feb 13, 2026

was a test

@dpiparo dpiparo closed this Feb 13, 2026
@root-project-bot root-project-bot deleted the BP_6.38_pull_21232 branch February 13, 2026 09:27
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.

2 participants