Merge branch 'master' into webpack-2

sokra · sokra · commit 7cbd78c1da03 · 2015-09-19T20:11:48.000+02:00
Conflicts:
	.travis.yml
	bin/config-optimist.js
	test/statsCases/define-plugin/expected.txt
diff --git a/.travis.yml b/.travis.yml
@@ -1,6 +1,7 @@
 sudo: false
 language: node_js
 node_js:
+  - "0.12"
   - node
   - iojs-1
   - iojs-2
@@ -10,7 +11,11 @@ env:
   - NO_WATCH_TESTS=1
 
 before_install:
-  - '[ "${TRAVIS_NODE_VERSION}" != "0.10" ] || npm install -g npm'
+  - '[ "${TRAVIS_NODE_VERSION}" != "0.10" ] || npm install -g npm@^2'
+
+before_script:
+  - npm link
+  - npm link webpack
 
 after_success:
   - cat ./coverage/lcov.info | node_modules/.bin/coveralls --verbose
diff --git a/README.md b/README.md
@@ -254,7 +254,7 @@ MIT (http://www.opensource.org/licenses/mit-license.php)
 (In chronological order)
 
 * @google for [Google Web Toolkit (GWT)](https://code.google.com/p/google-web-toolkit), which aims to compile Java to JavaScript. It features a similar [Code Splitting](https://code.google.com/p/google-web-toolkit/wiki/CodeSplitting) as webpack.
-* @medikoo for [modules-webmake](https://github.com/medikoo/modules-webmake), which is a simlar project. webpack was born because I wanted Code Splitting for modules-webpack. Interestingly the [Code Splitting issue is still open](https://github.com/medikoo/modules-webmake/issues/7) (thanks also to @Phoscur for the discussion).
+* @medikoo for [modules-webmake](https://github.com/medikoo/modules-webmake), which is a similar project. webpack was born because I wanted Code Splitting for modules-webpack. Interestingly the [Code Splitting issue is still open](https://github.com/medikoo/modules-webmake/issues/7) (thanks also to @Phoscur for the discussion).
 * @substack for [browserify](http://browserify.org/), which is a similar project and source for many ideas.
 * @jrburke for [require.js](http://requirejs.org/), which is a similar project and source for many ideas.
 * @defunctzombie for the [browser-field spec](https://gist.github.com/defunctzombie/4339901), which makes modules available for node.js, browserify and webpack.
diff --git a/bin/config-optimist.js b/bin/config-optimist.js
@@ -1,7 +1,7 @@
 module.exports = function(optimist) {
 	optimist
 		.boolean("help").alias("help", "h").alias("help", "?").describe("help")
-		.string("config").describe("config")
+		.string("config").alias("config", "c").describe("config")
 		.string("env").describe("env", "Enviroment passed to the config, when it is a function")
 		.string("context").describe("context")
 		.string("entry").describe("entry")
@@ -24,6 +24,7 @@ module.exports = function(optimist) {
 		.string("target").describe("target")
 		.boolean("cache").describe("cache").default("cache", true)
 		.boolean("watch").alias("watch", "w").describe("watch")
+		.boolean("watch-stdin").alias("watch-stdin", "stdin").describe("watch which closes when stdin ends")
 		.describe("watch-aggregate-timeout")
 		.describe("watch-poll")
 		.boolean("hot").alias("hot", "h").describe("hot")
diff --git a/bin/convert-argv.js b/bin/convert-argv.js
@@ -137,6 +137,12 @@ module.exports = function(optimist, argv, convertOptions) {
 			options.watchOptions.poll = true;
 	}
 
+	if(argv["watch-stdin"]) {
+		options.watchOptions = options.watchOptions || {};
+		options.watchOptions.stdin = true;
+		options.watch = true;
+	}
+
 	function processOptions(options) {
 		function ifArg(name, fn, init, finalize) {
 			if(Array.isArray(argv[name])) {
@@ -507,6 +513,7 @@ module.exports = function(optimist, argv, convertOptions) {
 				throw new Error("'output.filename' is required, either in config file or as --output-file");
 			} else {
 				optimist.showHelp();
+				console.error("Output filename not configured.");
 				process.exit(-1);
 			}
 		}
diff --git a/bin/webpack.js b/bin/webpack.js
@@ -172,6 +172,12 @@ function compilerCallback(err, stats) {
 if(options.watch) {
 	var primaryOptions = !Array.isArray(options) ? options : options[0];
 	var watchOptions = primaryOptions.watchOptions || primaryOptions.watch || {};
+	if(watchOptions.stdin) {
+		process.stdin.on('end', function() {
+			process.exit(0)
+		});
+		process.stdin.resume();
+	}
 	compiler.watch(watchOptions, compilerCallback);
 } else
 	compiler.run(compilerCallback);
diff --git a/lib/Compilation.js b/lib/Compilation.js
@@ -59,6 +59,8 @@ function Compilation(compiler) {
 module.exports = Compilation;
 
 Compilation.prototype = Object.create(Tapable.prototype);
+Compilation.prototype.constructor = Compilation;
+
 Compilation.prototype.templatesPlugin = function(name, fn) {
 	this.mainTemplate.plugin(name, fn);
 	this.chunkTemplate.plugin(name, fn);
diff --git a/lib/Compiler.js b/lib/Compiler.js
@@ -166,6 +166,7 @@ function Compiler() {
 module.exports = Compiler;
 
 Compiler.prototype = Object.create(Tapable.prototype);
+Compiler.prototype.constructor = Compiler;
 
 Compiler.Watching = Watching;
 Compiler.prototype.watch = function(watchOptions, handler) {
diff --git a/lib/ContextModuleFactory.js b/lib/ContextModuleFactory.js
@@ -16,6 +16,8 @@ function ContextModuleFactory(resolvers) {
 module.exports = ContextModuleFactory;
 
 ContextModuleFactory.prototype = Object.create(Tapable.prototype);
+ContextModuleFactory.prototype.constructor = ContextModuleFactory;
+
 ContextModuleFactory.prototype.create = function(context, dependency, callback) {
 	this.applyPluginsAsyncWaterfall("before-resolve", {
 		context: context,
diff --git a/lib/DllModuleFactory.js b/lib/DllModuleFactory.js
@@ -11,6 +11,8 @@ function DllModuleFactory() {
 module.exports = DllModuleFactory;
 
 DllModuleFactory.prototype = Object.create(Tapable.prototype);
+DllModuleFactory.prototype.constructor = DllModuleFactory;
+
 DllModuleFactory.prototype.create = function(context, dependency, callback) {
 	callback(null, new DllModule(context, dependency.dependencies, dependency.name, dependency.type));
 };
diff --git a/lib/LibraryTemplatePlugin.js b/lib/LibraryTemplatePlugin.js
@@ -22,9 +22,10 @@ function accessorAccess(base, accessor, joinWith) {
 	}).join(joinWith || "; ");
 }
 
-function LibraryTemplatePlugin(name, target) {
+function LibraryTemplatePlugin(name, target, umdNamedDefine) {
 	this.name = name;
 	this.target = target;
+	this.umdNamedDefine = umdNamedDefine;
 }
 module.exports = LibraryTemplatePlugin;
 LibraryTemplatePlugin.prototype.apply = function(compiler) {
@@ -60,7 +61,10 @@ LibraryTemplatePlugin.prototype.apply = function(compiler) {
 			case "umd":
 			case "umd2":
 				var UmdMainTemplatePlugin = require("./UmdMainTemplatePlugin");
-				compilation.apply(new UmdMainTemplatePlugin(this.name, this.target === "umd2"));
+				compilation.apply(new UmdMainTemplatePlugin(this.name, {
+					optionalAmdExternalAsGlobal: this.target === "umd2",
+					namedDefine: this.umdNamedDefine
+				}));
 				break;
 			case "jsonp":
 				var JsonpExportMainTemplatePlugin = require("./JsonpExportMainTemplatePlugin");
diff --git a/lib/MultiCompiler.js b/lib/MultiCompiler.js
@@ -88,6 +88,7 @@ function MultiCompiler(compilers) {
 module.exports = MultiCompiler;
 
 MultiCompiler.prototype = Object.create(Tapable.prototype);
+MultiCompiler.prototype.constructor = MultiCompiler;
 
 MultiCompiler.prototype.watch = function(watchDelay, handler) {
 	var watchings = this.compilers.map(function(compiler) {
diff --git a/lib/MultiModuleFactory.js b/lib/MultiModuleFactory.js
@@ -11,6 +11,8 @@ function MultiModuleFactory() {
 module.exports = MultiModuleFactory;
 
 MultiModuleFactory.prototype = Object.create(Tapable.prototype);
+MultiModuleFactory.prototype.constructor = MultiModuleFactory;
+
 MultiModuleFactory.prototype.create = function(context, dependency, callback) {
 	callback(null, new MultiModule(context, dependency.dependencies, dependency.name));
 };
diff --git a/lib/NormalModuleFactory.js b/lib/NormalModuleFactory.js
@@ -138,6 +138,8 @@ function NormalModuleFactory(context, resolvers, parser, options) {
 module.exports = NormalModuleFactory;
 
 NormalModuleFactory.prototype = Object.create(Tapable.prototype);
+NormalModuleFactory.prototype.constructor = NormalModuleFactory;
+
 NormalModuleFactory.prototype.create = function(context, dependency, callback) {
 	context = context || this.context;
 	var request = dependency.request;
diff --git a/lib/Parser.js b/lib/Parser.js
@@ -16,6 +16,8 @@ module.exports = Parser;
 // Syntax: https://developer.mozilla.org/en/SpiderMonkey/Parser_API
 
 Parser.prototype = Object.create(Tapable.prototype);
+Parser.prototype.constructor = Parser;
+
 Parser.prototype.initializeEvaluating = function() {
 	function joinRanges(startRange, endRange) {
 		if(!endRange) return startRange;
diff --git a/lib/Template.js b/lib/Template.js
@@ -21,6 +21,7 @@ Template.toIdentifier = function(str) {
 };
 
 Template.prototype = Object.create(Tapable.prototype);
+Template.prototype.constructor = Template;
 
 Template.prototype.indent = function indent(str) {
 	if(Array.isArray(str)) {
diff --git a/lib/UmdMainTemplatePlugin.js b/lib/UmdMainTemplatePlugin.js
@@ -20,9 +20,10 @@ function accessorAccess(base, accessor) {
 	}).join(", ");
 }
 
-function UmdMainTemplatePlugin(name, optionalAmdExternalAsGlobal) {
+function UmdMainTemplatePlugin(name, options) {
 	this.name = name;
-	this.optionalAmdExternalAsGlobal = optionalAmdExternalAsGlobal;
+	this.optionalAmdExternalAsGlobal = options.optionalAmdExternalAsGlobal;
+	this.namedDefine = options.namedDefine;
 }
 module.exports = UmdMainTemplatePlugin;
 UmdMainTemplatePlugin.prototype.apply = function(compilation) {
@@ -87,6 +88,11 @@ UmdMainTemplatePlugin.prototype.apply = function(compilation) {
 				return "__WEBPACK_EXTERNAL_MODULE_" + m.id + "__";
 			}).join(", ");
 		}
+
+		function libraryName(library) {
+			return JSON.stringify(replaceKeys([].concat(library).pop()));
+		}
+
 		if(optionalExternals.length > 0) {
 			var amdFactory = "function webpackLoadOptionalExternalModuleAmd(" + externalsArguments(requiredExternals) + ") {\n" +
 				"			return factory(" + (
@@ -98,18 +104,25 @@ UmdMainTemplatePlugin.prototype.apply = function(compilation) {
 		} else {
 			var amdFactory = "factory";
 		}
+
 		return new ConcatSource(new OriginalSource(
 			"(function webpackUniversalModuleDefinition(root, factory) {\n" +
 			"	if(typeof exports === 'object' && typeof module === 'object')\n" +
 			"		module.exports = factory(" + externalsRequireArray("commonjs2") + ");\n" +
 			"	else if(typeof define === 'function' && define.amd)\n" +
 			(requiredExternals.length > 0 ?
-				"		define(" + externalsDepsArray(requiredExternals) + ", " + amdFactory + ");\n" :
-				"		define(" + amdFactory + ");\n"
+				(this.name && this.namedDefine === true ?
+					"		define(" + libraryName(this.name) + ", " + externalsDepsArray(requiredExternals) + ", " + amdFactory + ");\n" :
+					"		define(" + externalsDepsArray(requiredExternals) + ", " + amdFactory + ");\n"
+				) :
+				(this.name && this.namedDefine === true ?
+					"		define(" + libraryName(this.name) + ", [], " + amdFactory + ");\n" :
+					"		define([], " + amdFactory + ");\n"
+				)
 			) +
 			(this.name ?
 				"	else if(typeof exports === 'object')\n" +
-				"		exports[" + JSON.stringify(replaceKeys([].concat(this.name).pop())) + "] = factory(" + externalsRequireArray("commonjs") + ");\n" +
+				"		exports[" + libraryName(this.name) + "] = factory(" + externalsRequireArray("commonjs") + ");\n" +
 				"	else\n" +
 				"		" + replaceKeys(accessorAccess("root", this.name)) + " = factory(" + externalsRootArray(externals) + ");\n" :
 				"	else {\n" +
diff --git a/lib/WatchIgnorePlugin.js b/lib/WatchIgnorePlugin.js
@@ -15,7 +15,7 @@ function IgnoringWatchFileSystem(wfs, paths) {
 	this.paths = paths;
 }
 
-IgnoringWatchFileSystem.prototype.watch = function(files, dirs, missing, startTime, delay, callback, callbackUndelayed) {
+IgnoringWatchFileSystem.prototype.watch = function(files, dirs, missing, startTime, options, callback, callbackUndelayed) {
 	var ignored = function(path) {
 		return this.paths.some(function(p) {
 			return p instanceof RegExp ? p.test(path) : path.indexOf(p) === 0;
@@ -25,10 +25,11 @@ IgnoringWatchFileSystem.prototype.watch = function(files, dirs, missing, startTi
 	var notIgnored = function(path) {
 		return !ignored(path);
 	};
+
 	var ignoredFiles = files.filter(ignored);
 	var ignoredDirs = dirs.filter(ignored);
 
-	this.wfs.watch(files.filter(notIgnored), dirs.filter(notIgnored), missing, startTime, delay, function(err, filesModified, dirsModified, fileTimestamps, dirTimestamps) {
+	this.wfs.watch(files.filter(notIgnored), dirs.filter(notIgnored), missing, startTime, options, function(err, filesModified, dirsModified, missingModified, fileTimestamps, dirTimestamps) {
 		if(err) return callback(err);
 
 		ignoredFiles.forEach(function(path) {
@@ -39,6 +40,6 @@ IgnoringWatchFileSystem.prototype.watch = function(files, dirs, missing, startTi
 			dirTimestamps[path] = 1;
 		});
 
-		callback(err, filesModified, dirsModified, fileTimestamps, dirTimestamps);
+		callback(err, filesModified, dirsModified, missingModified, fileTimestamps, dirTimestamps);
 	}, callbackUndelayed);
 };
diff --git a/lib/WebpackOptionsApply.js b/lib/WebpackOptionsApply.js
@@ -138,7 +138,8 @@ WebpackOptionsApply.prototype.process = function(options, compiler) {
 						"clipboard",
 						"crash-reporter",
 						"screen",
-						"shell"
+						"shell",
+						"native-image"
 					]),
 					new LoaderTargetPlugin(options.target)
 				);
@@ -153,7 +154,7 @@ WebpackOptionsApply.prototype.process = function(options, compiler) {
 	}
 	if(options.output.library || options.output.libraryTarget !== "var") {
 		var LibraryTemplatePlugin = require("./LibraryTemplatePlugin");
-		compiler.apply(new LibraryTemplatePlugin(options.output.library, options.output.libraryTarget));
+		compiler.apply(new LibraryTemplatePlugin(options.output.library, options.output.libraryTarget, options.output.umdNamedDefine));
 	}
 	if(options.externals) {
 		var ExternalsPlugin = require("./ExternalsPlugin");
diff --git a/lib/dependencies/ConstDependency.js b/lib/dependencies/ConstDependency.js
@@ -23,3 +23,8 @@ ConstDependency.Template.prototype.apply = function(dep, source) {
 	else
 		source.replace(dep.range[0], dep.range[1] - 1, dep.expression);
 };
+
+ConstDependency.prototype.updateHash = function(hash) {
+	hash.update(this.range + "");
+	hash.update(this.expression + "");
+};
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "webpack",
-  "version": "1.11.0",
+  "version": "1.12.2",
   "author": "Tobias Koppers @sokra",
   "description": "Packs CommonJs/AMD modules for the browser. Allows to split your codebase into multiple bundles, which can be loaded on demand. Support loaders to preprocess files, i.e. json, jade, coffee, css, less, ... and your custom stuff.",
   "dependencies": {
@@ -9,13 +9,14 @@
     "clone": "^1.0.2",
     "enhanced-resolve": "~0.9.0",
     "interpret": "^0.6.4",
+    "loader-utils": "^0.2.11",
     "memory-fs": "~0.2.0",
     "mkdirp": "~0.5.0",
     "node-libs-browser": ">= 0.4.0 <=0.6.0",
     "optimist": "~0.6.0",
     "supports-color": "^3.1.0",
     "tapable": "~0.1.8",
-    "uglify-js": "~2.4.13",
+    "uglify-js": "~2.4.24",
     "watchpack": "^0.2.1",
     "webpack-core": "~0.6.0"
   },
@@ -25,6 +26,7 @@
     "bundle-loader": "~0.5.0",
     "codecov.io": "^0.1.2",
     "coffee-loader": "~0.7.1",
+    "coffee-script": "^1.10.0",
     "component-webpack-plugin": "~0.2.0",
     "coveralls": "^2.11.2",
     "css-loader": "~0.15.0",
@@ -37,11 +39,12 @@
     "glob": "^5.0.14",
     "i18n-webpack-plugin": "~0.2.0",
     "istanbul": "^0.3.13",
+    "jade": "^1.11.0",
     "jade-loader": "~0.7.0",
     "js-beautify": "^1.5.10",
     "json-loader": "~0.5.1",
+    "less": "^2.5.1",
     "less-loader": "^2.0.0",
-    "loader-utils": "^0.2.11",
     "mocha": "~2.2.0",
     "mocha-lcov-reporter": "0.0.2",
     "raw-loader": "~0.5.0",
diff --git a/scripts/forEachBeautifiedFile.js b/scripts/forEachBeautifiedFile.js
@@ -29,6 +29,6 @@ module.exports = function forEachBeautifiedFile(fn, callback) {
 			if(err) return callback(err);
 			callback();
 		});
-	})
+	});
 
 };
diff --git a/test/Stats.test.js b/test/Stats.test.js
diff --git a/test/configCases/target/umd-named-define/index.js b/test/configCases/target/umd-named-define/index.js
diff --git a/test/configCases/target/umd-named-define/webpack.config.js b/test/configCases/target/umd-named-define/webpack.config.js
diff --git a/test/statsCases/define-plugin/expected.txt b/test/statsCases/define-plugin/expected.txt
diff --git a/test/statsCases/define-plugin/index.js b/test/statsCases/define-plugin/index.js
diff --git a/test/statsCases/define-plugin/webpack.config.js b/test/statsCases/define-plugin/webpack.config.js

Original file line number	Diff line number	Diff line change
`@@ -137,6 +137,12 @@ module.exports = function(optimist, argv, convertOptions) {`
`137`	`137`	`options.watchOptions.poll = true;`
`138`	`138`	`}`
`139`	`139`
	`140`	`+ if(argv["watch-stdin"]) {`
	`141`	`+ options.watchOptions = options.watchOptions \|\| {};`
	`142`	`+ options.watchOptions.stdin = true;`
	`143`	`+ options.watch = true;`
	`144`	`+ }`
	`145`	`+`
`140`	`146`	`function processOptions(options) {`
`141`	`147`	`function ifArg(name, fn, init, finalize) {`
`142`	`148`	`if(Array.isArray(argv[name])) {`
`@@ -507,6 +513,7 @@ module.exports = function(optimist, argv, convertOptions) {`
`507`	`513`	`throw new Error("'output.filename' is required, either in config file or as --output-file");`
`508`	`514`	`} else {`
`509`	`515`	`optimist.showHelp();`
	`516`	`+ console.error("Output filename not configured.");`
`510`	`517`	`process.exit(-1);`
`511`	`518`	`}`
`512`	`519`	`}`