Merge pull request #446 from edmorley/cache-signature-stack

Add $STACK to the cache signature

CHANGELOG.md

@@ -2,8 +2,10 @@
 
 ## Master
 
+- Add `$STACK` to the cache signature (#445)
 - Log output now differentiates between an empty cache and a new cache signature
 
+
 ## v106 (2017-06-19)
 
 - Default to npm v5 if `package-lock.json` is present (#429)

lib/cache.sh

 source $BP_DIR/lib/binaries.sh
 
 create_signature() {
-  echo "$(node --version); $(npm --version); $(yarn --version 2>/dev/null || true) $PREBUILD"
+  echo "${STACK}; $(node --version); $(npm --version); $(yarn --version 2>/dev/null || true); ${PREBUILD}"
 }
 
 save_signature() {

test/run

@@ -60,6 +60,7 @@ testBuildWithCache() {
 
   compile "stable-node" $cache
   assertNotCaptured "- node_modules (not cached - skipping)"
+  assertFileContains "${STACK}" "${cache}/node/signature"
   assertCapturedSuccess
 
   rm -rf "$cache/node/node_modules"