Skip to content

[core,clingutils] Fix CLANG_RESOURCE_DIR path with builtin_llvm=OFF - #23159

Merged
dpiparo merged 2 commits into
root-project:masterfrom
ferdymercury:patch-23
Aug 31, 2026
Merged

dpiparo merged 2 commits into
root-project:masterfrom
ferdymercury:patch-23

Conversation

@ferdymercury

Copy link
Copy Markdown
Collaborator

This Pull request:

Changes or fixes:

Fixes #23152

Checklist:

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

@ferdymercury
ferdymercury requested a review from bellenot as a code owner August 26, 2026 15:12
@ferdymercury ferdymercury added the clean build Ask CI to do non-incremental build on PR label Aug 26, 2026
@ferdymercury ferdymercury reopened this Aug 26, 2026
@github-actions

Copy link
Copy Markdown

Test Results

    23 files      23 suites   3d 13h 50m 59s ⏱️
 3 856 tests  3 854 ✅ 0 💤 2 ❌
78 596 runs  78 593 ✅ 1 💤 2 ❌

For more details on these failures, see this check.

Results for commit 0014f36.

@dpiparo
dpiparo merged commit a7b1786 into root-project:master Aug 31, 2026
35 of 38 checks passed
@ferdymercury
ferdymercury deleted the patch-23 branch August 31, 2026 09:17
@hahnjo

hahnjo commented Aug 31, 2026

Copy link
Copy Markdown
Member

FWIW I don't think this is correct, if anything the resource directory should be in interpreter/llvm-project/clang/lib/. But first of all, it should be understood what changed, why, and how to correctly fix it instead of updating paths.

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

Labels

clean build Ask CI to do non-incremental build on PR pr:squash on merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[clingutils] build regression between 6.40 and 6.42 with builtin_llvm=OFF

3 participants