Skip to content

tests: use relative batch names in source-removal regression - #1158

Closed
psumbera wants to merge 1 commit into
RsyncProject:masterfrom
psumbera:fix1
Closed

psumbera wants to merge 1 commit into
RsyncProject:masterfrom
psumbera:fix1

Conversation

@psumbera

@psumbera psumbera commented Nov 9, 2026

Copy link
Copy Markdown

The batch-file option limits its argument to 256 bytes. Passing the full scratch-directory path makes this test fail in deeply nested build trees before it can exercise source removal or batch replay.

Run both rsync commands from the fixture directory and pass the batch basename. Keep source and destination paths and all existing payload, source-retention, replay and hard-link checks unchanged.

The batch-file option limits its argument to 256 bytes. Passing the full
scratch-directory path makes this test fail in deeply nested build trees
before it can exercise source removal or batch replay.

Run both rsync commands from the fixture directory and pass the batch
basename. Keep source and destination paths and all existing payload,
source-retention, replay and hard-link checks unchanged.
steadytao added a commit that referenced this pull request Nov 10, 2026
Adapt #1158, #1159 and #1160 to the reorganised testsuite.

Co-authored-by: Petr Sumbera <sumbera@volny.cz>
@steadytao

Copy link
Copy Markdown
Member

Perfect timing. Folded into #1122.

@steadytao steadytao closed this Nov 10, 2026
@steadytao

Copy link
Copy Markdown
Member

Thank you for your work !

steadytao added a commit that referenced this pull request Nov 11, 2026
Adapt #1158, #1159 and #1160 to the reorganised testsuite.

Co-authored-by: Petr Sumbera <sumbera@volny.cz>
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