From b87fc217df277c2c7a240bb16bdf5020e3d9b550 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ma=C3=ABl=20Nison?= Date: Mon, 30 Jan 2017 06:15:18 -0800 Subject: [PATCH] Silents errors when running copy-sources Summary: Being a environment test, `npm run is-monolithic` is expected to fail when pulling the package from npm and to succeed when pulling it from github. Unfortunately, when returning false, npm will throw a tantrum by default if not explicitely disabled. It's not a hard failure, the build still complete, but it's better this way. Closes https://github.com/facebook/yoga/pull/362 Reviewed By: gkassabli Differential Revision: D4480608 Pulled By: emilsjolander fbshipit-source-id: 0832a2158647f49b5230e2f658997f94774cec5b --- javascript/package.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/javascript/package.json b/javascript/package.json index 08def8d5..aaad58fb 100644 --- a/javascript/package.json +++ b/javascript/package.json @@ -19,7 +19,7 @@ "emcc-path": "emcc-path", "is-monolithic": "test \"$(basename \"$(pwd)\")\" = javascript", - "copy-sources": "! npm -- run is-monolithic || (rsync -r --checksum --delete ../yoga/ sources/yoga/)", + "copy-sources": "! npm -s -- run is-monolithic || (rsync -r --checksum --delete ../yoga/ sources/yoga/)", "build:node": "npm -- run copy-sources && npm -- run node-gyp configure build", "build:browser": "npm -- run copy-sources && npm -- run node-gyp configure build --asmjs=1",