Skip to content

Check FORCE_NO_WATCHMAN env var before invoking watchman#5061

Closed
captbaritone wants to merge 1 commit into
mainfrom
watchman
Closed

Check FORCE_NO_WATCHMAN env var before invoking watchman#5061
captbaritone wants to merge 1 commit into
mainfrom
watchman

Conversation

@captbaritone
Copy link
Copy Markdown
Contributor

@captbaritone captbaritone commented Aug 26, 2025

I had a bug where watchman list-capabilities was hanging and this env variable was not letting me work around that bug. By moving this check earlier it should help more fully skip watchman.

Test Plan

Run ./scripts/compile-tests.sh hard coding it to --output=debug both with and without FORCE_NO_WATCHMAN=1 and see that it runs with Watchman by default and does not use Watchman when FORCE_NO_WATCHMAN=1.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@captbaritone has imported this pull request. If you are a Meta employee, you can view this in D81004271.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@captbaritone merged this pull request in 72e273c.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants