Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Time limit options applies to send and receive transfers #179

Merged
merged 1 commit into from
Jan 2, 2022

Conversation

rosorio
Copy link
Contributor

@rosorio rosorio commented Apr 20, 2021

stop-at, stop-after and time-limit options must be transfer
direction agnostic, and must apply to rsync in receiver
or sender mode.

stop-at, stop-after and time-limit options must be transfer
direction agnostic, and must apply to rsync in receiver
or sender mode.
@ppaeps
Copy link

ppaeps commented Apr 21, 2021

This patch appears to work. Previously, rsync 3.2.3 would simply hang, as reported in #177, with this patch, it exits as expected:

stopping at requested limit
rsync error: timeout in data send/receive (code 30) at io.c(802) [receiver=3.2.3]
stopping at requested limit
rsync error: timeout in data send/receive (code 30) at io.c(802) [generator=3.2.3]

For completeness, this is what used to happen with 3.1.3:

run-time limit exceeded
rsync error: timeout in data send/receive (code 30) at io.c(181) [generator=3.1.3]
rsync error: received SIGUSR1 (code 19) at main.c(1441) [receiver=3.1.3]

Thank you very much!

@rosorio
Copy link
Contributor Author

rosorio commented May 7, 2021

Hi @WayneD , do you plan to integrate this fix in a future release ?

freebsd-git pushed a commit to freebsd/freebsd-ports that referenced this pull request May 7, 2021
In latest version, upstream project breaks
time limit options implementation. The fix
allows the use of this option in both receiver
and sender mode. A merger request was submitted
to upstream project[1] but not yet merged.

Bump portrevision.

[1] RsyncProject/rsync#179

Reported by:	philip
Tested by:		philip
@ppaeps
Copy link

ppaeps commented Aug 24, 2021

Gently prodding here. We've been using this patch in production without a problem since April.

@WayneD WayneD merged commit ffbca80 into RsyncProject:master Jan 2, 2022
@WayneD
Copy link
Member

WayneD commented Jan 2, 2022

Thanks! Sorry for the huge delay.

This pull request was closed.
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.

3 participants