Skip to content

[6.36] Fix off-by-one error for merging nested directories - #23883

Merged
enirolf merged 2 commits into
root-project:v6-36-00-patchesfrom
enirolf:v636-hadd
Oct 29, 2026
Merged

enirolf merged 2 commits into
root-project:v6-36-00-patchesfrom
enirolf:v636-hadd

Conversation

@enirolf

@enirolf enirolf commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

Backport of #23878

enirolf and others added 2 commits October 29, 2026 09:25
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 2ebe327)
Regression test for root-project#23666

Co-authored-by: Vincenzo Eduardo Padulano <v.e.padulano@gmail.com>
(cherry picked from commit 43064e0)
@enirolf enirolf self-assigned this Oct 29, 2026
@enirolf
enirolf requested a review from pcanal as a code owner October 29, 2026 08:38
@enirolf enirolf added the experiment Affects an experiment / reported by its software & computimng experts label Oct 29, 2026
@enirolf enirolf changed the title [6.40] Fix off-by-one error for merging nested directories [6.36] Fix off-by-one error for merging nested directories Oct 29, 2026
@github-actions

Copy link
Copy Markdown

Test Results

    15 files      15 suites   2d 5h 42m 55s ⏱️
 2 746 tests  2 746 ✅ 0 💤 0 ❌
39 838 runs  39 838 ✅ 0 💤 0 ❌

Results for commit 958e71f.

@enirolf
enirolf merged commit ab0bf93 into root-project:v6-36-00-patches Oct 29, 2026
19 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.

1 participant