fixup debug X sourcemap intersect

Author: archmoj

tasks/partial_bundle.js

@@ -62,13 +62,13 @@ if(process.argv.length > 2) {
 
         name: out,
         index: path.join(constants.pathToLib, 'index-' + out + '.js'),
+        sourceMap: path.join(constants.pathToDist, 'plotly-' + out + '.js.map'),
         dist: path.join(constants.pathToDist, 'plotly-' + out + '.js'),
         distMin: path.join(constants.pathToDist, 'plotly-' + out + '.min.js')
     };
 
     console.log(opts);
 
-    opts.sourcemap = true;
     opts.calendars = true;
 
     var tasks = [];
@@ -89,7 +89,7 @@ function partialBundle(tasks, opts) {
     var traceList = opts.traceList;
     var transformList = opts.transformList;
     var calendars = opts.calendars;
-    var sourcemap = opts.sourcemap;
+    var sourceMap = opts.sourceMap;
 
     tasks.push(function(done) {
         var partialIndex = mainIndex;
@@ -123,7 +123,7 @@ function partialBundle(tasks, opts) {
 
     tasks.push(function(done) {
         var bundleOpts = {
-            debug: sourcemap,
+            pathToSourceMap: sourceMap,
             standalone: 'Plotly',
             pathToMinBundle: distMin
         };

tasks/util/browserify_wrapper.js

@@ -23,9 +23,8 @@ var strictD3 = require('./strict_d3');
  *  - noCompress {boolean} skip attribute meta compression?
  * @param {function} cb callback
  *
- * Outputs one bundle (un-minified) file if opts.pathToMinBundle is omitted
- * or opts.debug is true. Otherwise outputs two file: one un-minified bundle and
- * one minified bundle.
+ * Outputs one bundle (un-minified) file if opts.pathToMinBundle is omitted.
+ * Otherwise outputs two file: one un-minified bundle and one minified bundle.
  *
  * Logs basename of bundle when completed.
  */
@@ -36,7 +35,8 @@ module.exports = function _bundle(pathToIndex, pathToBundle, opts, cb) {
 
     var browserifyOpts = {};
     browserifyOpts.standalone = opts.standalone;
-    browserifyOpts.debug = opts.debug;
+    var sourceMap = opts.pathToSourceMap;
+    browserifyOpts.debug = opts.debug || sourceMap;
 
     if(opts.noCompress) {
         browserifyOpts.ignoreTransform = './tasks/compress_attributes.js';
@@ -72,10 +72,18 @@ module.exports = function _bundle(pathToIndex, pathToBundle, opts, cb) {
             });
     }
 
-    if(pathToBundle) {
+    if(sourceMap) {
+        bundleStream
+            .pipe(applyDerequire())
+            .pipe(exorcist(sourceMap))
+            .pipe(fs.createWriteStream(pathToBundle))
+            .on('finish', function() {
+                logger(pathToBundle);
+                done();
+            });
+    } else {
         bundleStream
             .pipe(applyDerequire())
-            .pipe(exorcist(pathToBundle + '.map'))
             .pipe(fs.createWriteStream(pathToBundle))
             .on('finish', function() {
                 logger(pathToBundle);