Milestones
nathany opened this issue · 6 comments
os/fsnotify
It looks like os/fsnotify will take the minimal approach that is appropriate for the os package. It's a good move, especially given the Go 1.3 timeline.
- Merge open pull requests? @howeyc
- Update code.google.com/p/go.exp/fsnotify to match GitHub master. https://codereview.appspot.com/58500043
- Encourage people to move issues to https://code.google.com/p/go/issues/list?q=fsnotify
- Finish API doc http://goo.gl/MrYxyA.
- Modify go.exp/fsnotify to match the proposed os/fsnotify API -- for use in Go 1.2.
- Leave howeyc/fsnotify as-is for existing users.
- Update contributing to get involved with os/fsnotify. #84
I'm not sure what approach @rsc wants to take to get os/fsnotify
itself implemented.
Go 1.3 code freeze is March 1st. Russ Cox wrote a proposal for inclusion of fsnotify and use by go test.
http://golang.org/s/go13fsnotify
http://golang.org/s/go13todo
golang-dev thread on fsnotify
@samjacobson In case you hadn't heard, Go 1.3 is to include os/fsnotify.
There is an active discussion on this CL https://codereview.appspot.com/48310043/ in addition to some threads in the golang-dev list.
Thanks for letting me know. That's great news. Congratulations!
On Tue, Jan 7, 2014 at 9:12 PM, Nathan Youngman notifications@github.comwrote:
@samjacobson https://github.com/samjacobson In case you hadn't heard,
Go 1.3 is to include os/fsnotify.There is an active discussion on this CL
https://codereview.appspot.com/48310043/ in addition to some threads in
the golang-dev list.—
Reply to this email directly or view it on GitHubhttps://github.com//issues/78#issuecomment-31719777
.
@abustany @cespare @henrikedwards @jbowtie @tsg
Have you signed the Go CLA? See http://golang.org/doc/contribute.html
Please indicate that you have signed the CLA here:
https://codereview.appspot.com/58500043
March 1st is code freeze for adding fsnotify to Go 1.3 standard library, and we have lots of work to do beyond copying across what's here. Please sign the CLA as soon as possible. Thanks!
Signed, see https://codereview.appspot.com/58500043/#msg8