Skip to content
This repository has been archived by the owner on Dec 6, 2021. It is now read-only.

v5 breaking on "extract-text-webpack-plugin" #94

Closed
marvinroger opened this issue Jan 6, 2017 · 11 comments
Closed

v5 breaking on "extract-text-webpack-plugin" #94

marvinroger opened this issue Jan 6, 2017 · 11 comments

Comments

@marvinroger
Copy link

greenkeeper just notified me the v5 was breaking my build.

See, for the same code:

It only fails with vbuild production releases. vbuild --dev builds correctly

@egoist
Copy link
Owner

egoist commented Jan 6, 2017

webpack 2 rc.3 introduces performance check, some of your bundles are way too big:

This can impact web performance.
Assets: 
  static/fontawesome-webfont.svg (444 kB)
  client.90bc0cc4.js (1.79 MB)

Check out https://webpack.js.org/configuration/performance/ for this

Currently you can disable it by:

export default {
  mergeConfig: {
    performance: {hints: false}
  }
}

@marvinroger
Copy link
Author

I was not expecting a warning to actually exit with a non-0 status code. Anyway it works now, thanks!

@marvinroger
Copy link
Author

@egoist the client.js is way too big, indeed. After checking out the content of the file (built for production with vbuild, I see process.env.NODE_ENV is not replaced, so the bundle is not optimized. Although it works with the template. I have no idea what's going on.

@marvinroger marvinroger reopened this Jan 9, 2017
@egoist
Copy link
Owner

egoist commented Jan 9, 2017

ah weird, no doubt it's set correctly https://github.com/egoist/vbuild/blob/master/lib/webpack/config.prod.js#L11

@marvinroger
Copy link
Author

I know, it's weird.

If you want to see it by yourself:

git clone https://github.com/INTECH-RGBH/homie-dashboard.git \
  && cd homie-dashboard \
  && git checkout e3225f9a7607f76e863946fdc7d64a31b98cb4f5 \
  && yarn \
  && yarn app-build

(this is a pretty big project, so it'll take some time)

Then see the client-*.js file, you'll see it contains some process.env.NODE_ENV.

@egoist
Copy link
Owner

egoist commented Jan 9, 2017

I use sao INTECH-RGBH/homie-dashboard foo --install to download it, will be faster than git clone 😎

eh, node-sass is still slow 😂

@egoist
Copy link
Owner

egoist commented Jan 9, 2017

2017-01-09 9 01 24

seems the vue-grid-layout contains inline sourcemaps, I don't know why webpack didn't remove it

@egoist
Copy link
Owner

egoist commented Jan 9, 2017

2017-01-09 9 16 16

why vue-grid-layout contains the eval thing 😂

@egoist
Copy link
Owner

egoist commented Jan 9, 2017

I think they had to change the devtool for umd build https://github.com/jbaysolutions/vue-grid-layout/blob/master/webpack.config.js#L57

@egoist egoist closed this as completed Jan 9, 2017
@marvinroger
Copy link
Author

Oh, awesome, thanks for the finding! What tool did you use to generate the tree? Might be helpful to avoid this kind of non-related-to-vbuild issues. 😒

@egoist
Copy link
Owner

egoist commented Jan 9, 2017

https://github.com/th0r/webpack-bundle-analyzer

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants