Merge pull request #118 from andersk/hash-files

Remove hash-files dependency

package.json

@@ -29,7 +29,6 @@
     "@vusion/webfonts-generator": "^0.6.1",
     "github-sponsors": "^1.0.1",
     "glob": "^7.1.6",
-    "hash-files": "^1.1.1",
     "loader-utils": "^1.4.0"
   },
   "devDependencies": {

utils.js

@@ -1,8 +1,19 @@
-var hashFiles = require('hash-files');
+var crypto = require('crypto');
+var fs = require('fs');
 
 module.exports = {
   hashFiles: function (files, hashLength) {
+    // FIXME: For compatibility with the hash-files package, this
+    // function just sorts by filename and hashes the concatenation of
+    // the file contents. This algorithm will not detect certain
+    // changes where files are renamed or chunks are moved across
+    // file boundaries (https://github.com/mac-/hash-files/issues/4).
+
     hashLength = hashLength && +hashLength >= 8 && +hashLength <= 32 ? +hashLength : 20;
-    return hashFiles.sync({ files: files }).slice(0, hashLength);
+    var hash = crypto.createHash('sha1');
+    Array.from(new Set(files)).sort().forEach(function (file) {
+      hash.update(fs.readFileSync(file));
+    });
+    return hash.digest('hex').slice(0, hashLength);
   }
 };