feat: support rsync no-option handling #229

Closed
TapTap wants to merge 0 commits from feat/rsync-no-options into dev
Owner

Summary

  • add rsync-compatible --no-OPTION handling for supported boolean options
  • allow negating archive-implied compression, multithreading, and metadata
  • reject unsupported or unsafe negations clearly
  • add unit and integration coverage

Verification

  • strict CMake build
  • unit tests: 25 passed
  • pytest: 40 passed, 10 skipped, 1 xpassed
  • clang-format check
  • cppcheck in fastsync-ci:local
## Summary - add rsync-compatible --no-OPTION handling for supported boolean options - allow negating archive-implied compression, multithreading, and metadata - reject unsupported or unsafe negations clearly - add unit and integration coverage ## Verification - strict CMake build - unit tests: 25 passed - pytest: 40 passed, 10 skipped, 1 xpassed - clang-format check - cppcheck in fastsync-ci:local
TapTap added 1 commit 2026-09-03 16:22:29 +02:00
feat: support rsync no-option handling
CI / lint (pull_request) Successful in 12s
CI / sanitizers (address) (pull_request) Successful in 37s
CI / sanitizers (undefined) (pull_request) Successful in 37s
CI / fuzz-build (pull_request) Successful in 15s
CI / coverage (pull_request) Successful in 31s
CI / build-and-test (pull_request) Successful in 1m15s
CI / valgrind (pull_request) Successful in 33s
2ff2688ef1
TapTap added 1 commit 2026-09-03 21:31:00 +02:00
fix: preserve explicit rsync option negations
CI / lint (pull_request) Successful in 11s
CI / sanitizers (address) (pull_request) Successful in 36s
CI / sanitizers (undefined) (pull_request) Successful in 35s
CI / fuzz-build (pull_request) Successful in 15s
CI / coverage (pull_request) Successful in 31s
CI / build-and-test (pull_request) Successful in 1m15s
CI / valgrind (pull_request) Successful in 33s
302c3cb664
Author
Owner

Merged as part of the Phase 1 rsync-compatibility batch. Content was integrated into dev sequentially via local merges with conflict resolution, then pushed to origin/dev (HEAD 48dfd5d). The incremental-skip regression found during integration (nanosecond mtime comparison at default modify-window=0) was fixed in 48dfd5d. Verification: unit tests 25/25, integration 74 passed. Closing this PR since its changes are already in dev; Gitea's merge endpoint is unavailable (2FA).

Merged as part of the Phase 1 rsync-compatibility batch. Content was integrated into `dev` sequentially via local merges with conflict resolution, then pushed to `origin/dev` (HEAD 48dfd5d). The incremental-skip regression found during integration (nanosecond mtime comparison at default modify-window=0) was fixed in 48dfd5d. Verification: unit tests 25/25, integration 74 passed. Closing this PR since its changes are already in `dev`; Gitea's merge endpoint is unavailable (2FA).
TapTap closed this pull request 2026-09-05 10:45:47 +02:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: TapTap/FastSync#229