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

File watching fails on MacOS Big Sur beta #151

Closed
rgm opened this issue Jul 8, 2020 · 3 comments
Closed

File watching fails on MacOS Big Sur beta #151

rgm opened this issue Jul 8, 2020 · 3 comments
Labels
question Further information is requested

Comments

@rgm
Copy link

rgm commented Jul 8, 2020

There's an issue in JNA that's biting Figwheel too for users on macOS v10.16 / v11 / Big Sur or whatever we're calling it (ie. the new version for Sept 2020 being beta tested right now). The annual increase in security-related lockdowns has introduced a crasher into JNA, which kills barbary, which kills hawk, which kills bin/kaocha --watch.

bhauman/figwheel-main#253

The low-friction temp fix we figured out for Figwheel while this is being sorted out is to tell hawk to start with a :polling watcher instead of a :barbary watcher. It kills battery life since it's no longer using native filesystem change events, but at least it keeps things working while it gets sorted out at the JNA level.

Is there a similar way to allow passing hawk options into kaocha, or, if not, a PR for the same being welcome? Might be a useful add beyond this one temp fix. I think it's in Figwheel to help cope with some Docker-related issues.

@alysbrooks
Copy link
Member

@rgm Looks like this has been fixed? thheller/shadow-cljs#767. Adding the ability to pass hawk options isn't a bad idea, so I may keep this issue open for that change.

@alysbrooks alysbrooks added the question Further information is requested label Jan 27, 2021
@alysbrooks
Copy link
Member

Oh, I figured out you can already pass options into Hawk: https://cljdoc.org/d/lambdaisland/kaocha/CURRENT/doc/7-watch-mode#configuring-the-watcher

@oxalorg
Copy link
Member

oxalorg commented Feb 4, 2022

Fixed in #268 and released in v1.62.993

@oxalorg oxalorg closed this as completed Feb 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants