diff options
| author | Ben Noordhuis <info@bnoordhuis.nl> | 2015-02-22 09:50:20 +0300 |
|---|---|---|
| committer | cjihrig <cjihrig@gmail.com> | 2015-02-23 19:46:32 +0300 |
| commit | da730c76e98fb9fd18dac445dafbbec74d79f802 (patch) | |
| tree | 5fc3a24daa5afe5d31b5cc142e9679a5be3c83bb /deps | |
| parent | b41dbc2737e2486fd9271cc7bae096bac1af8a01 (diff) | |
deps: enable node-gyp iojs.lib download checksum
Originally disabled in commit 5de334c ("deps: make node-gyp work again
on windows") due to the then-website lacking the requisite SHASUMS.txt
or SHASUMS256.txt files. The website has a SHASUMS256.txt now so start
checksumming the download again.
PR-URL: https://github.com/iojs/io.js/pull/918
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Diffstat (limited to 'deps')
| -rw-r--r-- | deps/npm/node_modules/node-gyp/lib/install.js | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/deps/npm/node_modules/node-gyp/lib/install.js b/deps/npm/node_modules/node-gyp/lib/install.js index 378fbfae476..f9176b3ab0b 100644 --- a/deps/npm/node_modules/node-gyp/lib/install.js +++ b/deps/npm/node_modules/node-gyp/lib/install.js @@ -295,8 +295,7 @@ function install (gyp, argv, callback) { // check content shasums for (var k in contentShasums) { log.verbose('validating download checksum for ' + k, '(%s == %s)', contentShasums[k], expectShasums[k]) - // TODO(piscisaureus) re-enable checksum verification when the correct files are in place. - if (false || contentShasums[k] !== expectShasums[k]) { + if (contentShasums[k] !== expectShasums[k]) { cb(new Error(k + ' local checksum ' + contentShasums[k] + ' not match remote ' + expectShasums[k])) return } |