Skip to content

[6.40] Fix off-by-one error for merging nested directories - #23882

Merged
enirolf merged 3 commits into
root-project:v6-40-00-patchesfrom
root-project-bot:BP_6.40_pull_23878
Oct 29, 2026
Merged

enirolf merged 3 commits into
root-project:v6-40-00-patchesfrom
root-project-bot:BP_6.40_pull_23878

Conversation

@root-project-bot

Copy link
Copy Markdown

Backport of #23878, requested by @enirolf.

enirolf and others added 3 commits October 29, 2026 08:06
Instead of comparing the *file* of the source directory to the target
directory, we should be comparing it to the *file* of the target
directory.

(cherry picked from commit cd13299)
Regression test for root-project#23666

Co-authored-by: Vincenzo Eduardo Padulano <v.e.padulano@gmail.com>
(cherry picked from commit 272aa00)
...directly using `hadd`, and not only the TFileMerger

(cherry picked from commit df488ea)
@github-actions

Copy link
Copy Markdown

Test Results

    23 files      23 suites   3d 19h 39m 43s ⏱️
 3 880 tests  3 880 ✅ 0 💤 0 ❌
81 210 runs  81 210 ✅ 0 💤 0 ❌

Results for commit 3e915aa.

@enirolf
enirolf merged commit 4fa6bf8 into root-project:v6-40-00-patches Oct 29, 2026
43 of 46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug experiment Affects an experiment / reported by its software & computimng experts in:I/O pr:backport

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants