Merge pull request #2569 from mattermost/run-fullmap

Adding make target for building with a full source map

Makefile

@@ -185,8 +185,15 @@ run-client:
 
 	cd $(BUILD_WEBAPP_DIR) && $(MAKE) run
 
+run-client-fullmap:
+	@echo Running mattermost client for development with FULL SOURCE MAP
+
+	cd $(BUILD_WEBAPP_DIR) && $(MAKE) run-fullmap
+
 run: run-server run-client
 
+run-fullmap: run-server run-client-fullmap
+
 stop-server:
 	@echo Stopping mattermost
 

webapp/Makefile

@@ -22,6 +22,11 @@ run: .npminstall
 
 	npm run run &
 
+run-fullmap: .npminstall
+	@echo FULL SOURCE MAP Running mattermost Webapp for development FULL SOURCE MAP
+
+	npm run run-fullmap &
+
 stop:
 	@echo Stopping changes watching
 

webapp/package.json

@@ -56,6 +56,7 @@
   "scripts": {
     "check": "eslint --ext \".jsx\" --ignore-pattern node_modules --quiet .",
     "build": "webpack",
-    "run": "webpack --progress --watch"
+    "run": "webpack --progress --watch",
+    "run-fullmap": "webpack --progress --watch"
   }
 }

webapp/webpack.config.js

@@ -8,8 +8,12 @@ const htmlExtract = new ExtractTextPlugin('html', 'root.html');
 const NPM_TARGET = process.env.npm_lifecycle_event; //eslint-disable-line no-process-env
 
 var DEV = false;
-if (NPM_TARGET === 'run') {
+var FULLMAP = false;
+if (NPM_TARGET === 'run' || NPM_TARGET === 'run-fullmap') {
     DEV = true;
+    if (NPM_TARGET === 'run-fullmap') {
+        FULLMAP = true;
+    }
 }
 
 var config = {
@@ -94,7 +98,11 @@ var config = {
 
 // Development mode configuration
 if (DEV) {
-    config.devtool = 'eval-cheap-module-source-map';
+    if (FULLMAP) {
+        config.devtool = 'source-map';
+    } else {
+        config.devtool = 'eval-cheap-module-source-map';
+    }
 }
 
 // Production mode configuration