v2.0.0 gives error when running ember deploy about broccoli-terser-sourcemap

[0xadada]

Workaround for this was to disable sourcemaps in production builds

ember-cli-build.js

// ...
const EmberApp = require('ember-cli/lib/broccoli/ember-app');

// ...
module.exports = function(defaults) {
  let app = new EmberApp(defaults, {
    sourcemaps: {
      enabled: EmberApp.env() !== 'production',
    },
   // ...

[rwjblue]

@urbany - #29 was not a bug in ember-cli-uglify (but was instead a bug in the actual source files that Ember generated). So far there is no evidence that an actual bug exists in this repo...

[akatov]

I think this is pretty much the same issue as ember-cli/ember-cli-terser#29

[urbany]

I added a comment there I think it's not solved for every case yet: ember-cli/ember-cli-terser#29 (comment)

[urbany]

Thank you for the quick reply Robert!
I'm not sure how all the pieces fit together, but shouldn't the broccoli-uglify-sourcemap remove the //# sourceMappingURL=popper.js.map when joining all the vendor files into the vendor.js?since the popper.js.map is already in the cendor.map there is no need to have the //# sourceMappingURL=popper.js.map comment right?