From 2f980e808faa835475ecc5da75fd36b1622902e2 Mon Sep 17 00:00:00 2001 From: Juan Cruz Viotti Date: Fri, 18 Nov 2016 17:28:23 +0000 Subject: [PATCH] chore: add x86 architecture support for darwin/dependencies.sh (#883) We don't really do x64 builds for OS X at the moment, but nice to make the build scripts consistent. Signed-off-by: Juan Cruz Viotti --- scripts/darwin/dependencies.sh | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/scripts/darwin/dependencies.sh b/scripts/darwin/dependencies.sh index 8b8cfc0d..7af56869 100755 --- a/scripts/darwin/dependencies.sh +++ b/scripts/darwin/dependencies.sh @@ -68,9 +68,14 @@ fi # Ensure native addons are compiled with the correct headers # See https://github.com/electron/electron/blob/master/docs/tutorial/using-native-node-modules.md export npm_config_disturl=https://atom.io/download/atom-shell -export npm_config_target=$ARGV_ELECTRON_VERSION -export npm_config_arch=$ARGV_ARCHITECTURE export npm_config_runtime=electron +export npm_config_target=$ARGV_ELECTRON_VERSION + +if [ "$ARGV_ARCHITECTURE" == "x86" ]; then + export npm_config_arch=ia32 +else + export npm_config_arch=$ARGV_ARCHITECTURE +fi rm -rf node_modules bower_components npm install --build-from-source