Skip to content

Commit

Permalink
chore: allow running buildup and builddown scripts without lerna glob…
Browse files Browse the repository at this point in the history
…ally installed (aws#2667)

Modift buildup and builddown script to include `$(npm bin)` in `PATH` so that lerna does not need to be installed globally in order to run the scripts.

Closes aws#1338
  • Loading branch information
Naturalclar authored and rix0rrr committed May 29, 2019
1 parent bbf8ba4 commit 436694f
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ fail() {
BUILD_INDICATOR=".BUILD_COMPLETED"
rm -rf $BUILD_INDICATOR

export PATH=node_modules/.bin:$PATH
export PATH=$(npm bin):$PATH
export NODE_OPTIONS="--max-old-space-size=4096 ${NODE_OPTIONS:-}"

# Speed up build by reusing calculated tree hashes
Expand Down
2 changes: 1 addition & 1 deletion install.sh
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ echo "installing repo-global dependencies..."
npm ci --global-style

# Now that we have lerna available...
export PATH=node_modules/.bin:$PATH
export PATH=$(npm bin):$PATH

echo "============================================================================================="
echo "cleanup and start bootstrapping..."
Expand Down
1 change: 1 addition & 0 deletions scripts/builddown
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
#!/bin/bash
set -euo pipefail

export PATH=$(npm bin):$PATH
export NODE_OPTIONS="--max-old-space-size=4096 ${NODE_OPTIONS:-}"

scriptdir=$(cd $(dirname $0) && pwd)
Expand Down
1 change: 1 addition & 0 deletions scripts/buildup
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
#!/bin/bash
set -euo pipefail

export PATH=$(npm bin):$PATH
export NODE_OPTIONS="--max-old-space-size=4096 ${NODE_OPTIONS:-}"

scriptdir=$(cd $(dirname $0) && pwd)
Expand Down

0 comments on commit 436694f

Please sign in to comment.