-
Notifications
You must be signed in to change notification settings - Fork 42
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
Add compatibility with hinotify-0.3.10 #77
Conversation
Ping ? If this could be merged and released, it would help me out in the hledger project. |
This reverts commit 705513c Waiting on haskell-fswatch/hfsnotify#77
I think this is the very last dependency blocking Stack from being build with GHC 8.4. |
Hey there, Sorry for the silence — I don't really have time to maintain this package and must've unsubscribed from the notifications at some point. I'll merge and release this PR (thanks for writing the patch, Dmitry!), and then will look for a new maintainer and/or home for this project. Does anyone here want to step in as an active maintainer by any chance? |
This is now released as fsnotify-0.2.1.2. |
Thank you! |
Thanks @feuerbach. https://github.com/fpco/stackage/blob/master/build-constraints.yaml can now be updated too, re-enabling all packages marked with "via hinotify" or "via fsnotify". Or if you'd prefer, I can submit the PR. |
close #75
From the ticket discussion:
I looked into the commit kolmodin/hinotify@11c4d31 and implemented the conversions from/to
ByteString
that previously were done byhinotify
. So the logic should've stayed the same