[nix] revert nix watchman to use brew
AbandonedPublic
Actions

Authored by will on Jul 22 2024, 12:30 PM.

Tags

None

Referenced Files

	F3368255: D12848.diff
	Mon, Nov 25, 6:56 PM

	Unknown Object (File)
	Fri, Nov 22, 7:47 PM

	Unknown Object (File)
	Fri, Nov 8, 10:36 PM

	Unknown Object (File)
	Fri, Nov 8, 1:43 PM

	Unknown Object (File)
	Fri, Nov 8, 1:43 PM

	Unknown Object (File)
	Fri, Nov 8, 12:22 PM

	Unknown Object (File)
	Thu, Nov 7, 11:44 PM

	Unknown Object (File)
	Oct 4 2024, 6:15 PM

View All 40 Files

Subscribers

ashoat

tomek

Details

Reviewers

marcin
ashoat
bartek
varun

Group Reviewers

Restricted Owners Package

(Owns No Changed Paths)

Summary

marcin seems to be encountering issues with nix-based watchman. This reverts using watchman with nix https://phab.comm.dev/D12777 in favor of brew

Test Plan

nix develop and brew successfully installs watchman

Diff Detail

Repository

rCOMM Comm

Branch

revert_nix_watchman

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

will created this revision.Jul 22 2024, 12:30 PM

Owners added a reviewer: Restricted Owners Package.Jul 22 2024, 12:31 PM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptJul 22 2024, 12:31 PM

will added reviewers: ashoat, bartek, varun.Jul 22 2024, 12:35 PM

Harbormaster completed remote builds in B30580: Diff 42645.Jul 22 2024, 12:47 PM

will requested review of this revision.Jul 22 2024, 12:47 PM

Before we land this, let's see if we can solve it for @marcin some other way

This revision is now accepted and ready to land.Jul 22 2024, 1:49 PM

will abandoned this revision.Aug 6 2024, 10:31 AM

Diff no longer required. @marcin seems to have resolved his Nix watchman issues on https://linear.app/comm/issue/ENG-8886/watchman-failure-in-cd-native-andand-yarn-dev

Revision Contents
Changeset List

Path

Size

nix/

dev-shell.nix

4 lines

scripts/

install_homebrew_deps.sh

20 lines

Diff 42645

View Options

nix/dev-shell.nix