-
Notifications
You must be signed in to change notification settings - Fork 235
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
Graceful stop in repeat mode - part 2 #833
Draft
tleedjarv
wants to merge
6
commits into
bcpierce00:master
Choose a base branch
from
tleedjarv:fix-554
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Commits on Feb 18, 2023
-
Differentiate Lwt_unix platform at run time
Lwt_unix does not implement all functions on Win32. Make it possible to differentiate the Lwt_unix implementation at run time to avoid calling unimplemented functions. Previously this differentiation was done by probing the compiler OS platform (Win32? with Cygwin?) but this is not reliable because it does not indicate the actual Lwt_unix implementation that's linked in.
Configuration menu - View commit details
-
Copy full SHA for 114c696 - Browse repository at this point
Copy the full SHA 114c696View commit details -
Add graceful stop in repeat mode - stdin
Add a new stop request method to terminate repeat mode. See commit e72c1c5. Monitor stdin to read any of the several stop conditions. Using stdin for checking the repeat mode stop request opens up a wide range of possibilities for the user, including manual and automated, local and remote. This method is usable only if stdin is an interactive terminal or is redirected from a readable source that is not a regular file.
Configuration menu - View commit details
-
Copy full SHA for a1bc7be - Browse repository at this point
Copy the full SHA a1bc7beView commit details -
Configuration menu - View commit details
-
Copy full SHA for 1ae3db4 - Browse repository at this point
Copy the full SHA 1ae3db4View commit details -
Add graceful stop in repeat mode - background job
Improve the method from commit a1bc7be to allow Unison run as a background job in an interactive terminal. Reading from stdin while not a foreground process will normally result in the process being stopped (or receiving EIO if ignoring the stop signal). Do not attempt to read from stdin while not in foreground. Guard the process against being stopped when trying to read from stdin at the wrong moment.
Configuration menu - View commit details
-
Copy full SHA for 95bc9f5 - Browse repository at this point
Copy the full SHA 95bc9f5View commit details -
Add graceful stop in repeat mode - regular file
Add a new stop request method to terminate repeat mode. See commit e72c1c5. Enable the method from commit a1bc7be to work with stdin redirected from a regular file. Regular files differ from other sources: - Since a regular file is always ready for reading, limit the reads to once a second. - Reaching EOF in a regular file is not considered a stop request. This enables the process to be started with input redirected from an empty file and then waiting for data to be written into the file. - To prevent reading massive amounts of data, only enable the regular file method if the file is initially empty or at most 32 bytes in size.
Configuration menu - View commit details
-
Copy full SHA for 7a96641 - Browse repository at this point
Copy the full SHA 7a96641View commit details -
Configuration menu - View commit details
-
Copy full SHA for 7ff6fa4 - Browse repository at this point
Copy the full SHA 7ff6fa4View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.