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

Adding the write close event #313

Closed
derekschrock opened this issue May 4, 2015 · 8 comments
Closed

Adding the write close event #313

derekschrock opened this issue May 4, 2015 · 8 comments

Comments

@derekschrock
Copy link

Would it be possible to have a write close event?
This would be valuable to monitor file when copy finishes.

@tuxinaut
Copy link

👍

1 similar comment
@aig-
Copy link

aig- commented May 25, 2015

+1

@tamland
Copy link
Collaborator

tamland commented May 25, 2015

Not in a portable way. It would be inotify only (maybe windows too). See #217

@ph-hs
Copy link

ph-hs commented Oct 7, 2016

openssl rand 1000000000 -out biiig.file makes a file big enough that watchdog notifies a couple of times during the file being written.

We have to write our own "quiet time" logic to ensure the the file has finished being written to?

@rrzaripov
Copy link
Contributor

@BoboTiG same issue as in #567

@mortoray
Copy link

mortoray commented Aug 2, 2020

Unfortunately the lack of this feature makes this library very difficult to use. There are numerous software applications that write in a serialized fashion to the file, generating many modified events. It's not safe to respond to any of those events and access the file.

Is there some other way to detect this state, and/or lock it?

@BoboTiG BoboTiG changed the title write close Adding the write close event Aug 7, 2020
@BoboTiG
Copy link
Collaborator

BoboTiG commented Aug 7, 2020

Even if the write-close event is not portable, let's implement it where we can.

We are open to suggestions and PRs :)

@BoboTiG
Copy link
Collaborator

BoboTiG commented Jan 15, 2021

Finally implemented the inotify part with 2fab7c2 (will be part of the 1.0.3 version).
Let's open specific issues for other OSes, if required.

@BoboTiG BoboTiG closed this as completed Jan 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants