Fix watchman missing error spawning logs
Summary: "Watchman not in PATH" shows up in our error monitoring, because the underlying implementation first logs an error, and then throw it (which we properly catch). So patch-package once again to the rescue to make sure we don't error log an error we actually handle. Reviewed By: antonk52 Differential Revision: D33982047 fbshipit-source-id: 64384b034b1991e30b2651b3ebf66cd26d519edc
This commit is contained in:
committed by
Facebook GitHub Bot
parent
d0d0b68e0b
commit
704e14a91a
13
desktop/patches/fb-watchman+2.0.1.patch
Normal file
13
desktop/patches/fb-watchman+2.0.1.patch
Normal file
@@ -0,0 +1,13 @@
|
|||||||
|
diff --git a/node_modules/fb-watchman/index.js b/node_modules/fb-watchman/index.js
|
||||||
|
index 9f46e16..b7625cf 100644
|
||||||
|
--- a/node_modules/fb-watchman/index.js
|
||||||
|
+++ b/node_modules/fb-watchman/index.js
|
||||||
|
@@ -166,7 +166,7 @@ Client.prototype.connect = function() {
|
||||||
|
'https://facebook.github.io/watchman/docs/install.html ' +
|
||||||
|
'for installation instructions';
|
||||||
|
}
|
||||||
|
- console.error('Watchman: ', error.message);
|
||||||
|
+ console.warn('Watchman: ', error.message);
|
||||||
|
self.emit('error', error);
|
||||||
|
}
|
||||||
|
|
||||||
Reference in New Issue
Block a user