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

watchman: launchctl unload post-install #9456

Merged
merged 1 commit into from
Feb 10, 2017
Merged

Commits on Jan 30, 2017

  1. watchman: launchctl unload post-install

    This is taking a run at: facebook/watchman#358
    
    Watchman installs and updates its own launchd.plist file on behalf of the user; it is not integrated with `brew services`.
    In the linked issue, watchman would stop working for some homebrew users.  The problem was that the launchd.plist used socket activation to set up the service, and because launchd is maintaining the socket, and the socket connection attempt just hangs, watchman never had a chance to fix up the plist and things stayed broken.
    
    To help solve this, we're forcing an unload in the post-install portion of the recipe.
    wez committed Jan 30, 2017
    Configuration menu
    Copy the full SHA
    199357c View commit details
    Browse the repository at this point in the history