Skip to content

Commit

Permalink
Merge branch 'release/v6.2.1'
Browse files Browse the repository at this point in the history
* release/v6.2.1:
  Bump patch version
  Fix conflicts
  cache: make sure the directory exists, asynchronously
  • Loading branch information
Couto committed Jan 4, 2016
2 parents 8dca831 + 09788f8 commit 2fa087a
Show file tree
Hide file tree
Showing 5 changed files with 27 additions and 23 deletions.
6 changes: 3 additions & 3 deletions index.js
Original file line number Diff line number Diff line change
Expand Up @@ -83,8 +83,8 @@ module.exports = function(source, inputSourceMap) {
if (err) { return callback(err); }
return callback(null, result.code, result.map);
});
} else {
result = transpile(source, options);
this.callback(null, result.code, result.map);
}

result = transpile(source, options);
this.callback(null, result.code, result.map);
};
39 changes: 22 additions & 17 deletions lib/fs-cache.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
* @see https://github.com/babel/babel-loader/pull/41
*/
var crypto = require('crypto');
var mkdirp = require('mkdirp');
var fs = require('fs');
var os = require('os');
var path = require('path');
Expand Down Expand Up @@ -128,23 +129,27 @@ var cache = module.exports = function(params, callback) {
os.tmpdir();
var file = path.join(directory, filename(source, identifier, options));

return read(file, function(err, content) {
var result = {};
// No errors mean that the file was previously cached
// we just need to return it
if (!err) { return callback(null, content); }

// Otherwise just transform the file
// return it to the user asap and write it in cache
try {
result = transform(source, options);
} catch (error) {
return callback(error);
}

return write(file, result, function(err) {
return callback(err, result);
});
// Make sure the directory exists.
return mkdirp(directory, function(err) {
if (err) { return callback(err); }

return read(file, function(err, content) {
var result = {};
// No errors mean that the file was previously cached
// we just need to return it
if (!err) { return callback(null, content); }

// Otherwise just transform the file
// return it to the user asap and write it in cache
try {
result = transform(source, options);
} catch (error) {
return callback(error);
}

return write(file, result, function(err) {
return callback(err, result);
});
});
});
};
1 change: 0 additions & 1 deletion lib/helpers/exists.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ var fs = require('fs');
* exists('.babelrc'); // false
*/
module.exports = function(cache) {

cache = cache || {};

return function(filename) {
Expand Down
1 change: 0 additions & 1 deletion lib/helpers/read.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ var fs = require('fs');
* read('.babelrc'); // file contents...
*/
module.exports = function(cache) {

cache = cache || {};

return function(filename) {
Expand Down
3 changes: 2 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,13 +1,14 @@
{
"name": "babel-loader",
"version": "6.2.0",
"version": "6.2.1",
"description": "babel module loader for webpack",
"files": [
"index.js",
"lib"
],
"dependencies": {
"loader-utils": "^0.2.11",
"mkdirp": "^0.5.1",
"object-assign": "^4.0.1"
},
"peerDependencies": {
Expand Down

0 comments on commit 2fa087a

Please sign in to comment.