Comments (4)
Thanks for your feedback!
There's two main problems right now:
- during my quick tests, brotli,js was at average 3x slower than iltorb
- brotli.js exposes only synchronous interface (due to emscripten-generated code)
While the first one could be solved using optionalDependencies
and runtime require()
s, the secound could be tricky and degrade overall webpack performance.
I'll look into it soon, but PRs are welcome :)
from brotli-webpack-plugin.
@unsafecode sorry for the delay! I've created PR #5 with conditional logic.
Could you please try it on your Windows setup with npm install --save-dev brotli-webpack-plugin@next
and leave feedback?
from brotli-webpack-plugin.
@mynameiswhm Nice, it works! I'd just add in the docs the need of a peerDep for brotli, so to state it clearly.
from brotli-webpack-plugin.
@unsafecode both iltorb
and brotli
is now included in optionalDependencies
, so npm will try to install all of them, but fail silently if something goes wrong. Worst case scenario: both installations failed, but it's hard to imagine that for brotli.js, which is precompiled emscripten port.
from brotli-webpack-plugin.
Related Issues (20)
- An in-range update of webpack is breaking the build 🚨 HOT 3
- An in-range update of iltorb is breaking the build 🚨 HOT 7
- Webpack 4 support HOT 2
- An in-range update of webpack is breaking the build 🚨 HOT 1
- An in-range update of webpack is breaking the build 🚨 HOT 6
- An in-range update of webpack is breaking the build 🚨 HOT 4
- An in-range update of iltorb is breaking the build 🚨 HOT 1
- An in-range update of webpack is breaking the build 🚨 HOT 4
- Prebuild with iltorb is broken HOT 2
- Use native node brotli support HOT 3
- Segmentation fault: 11 HOT 1
- Noisy output in `compress.js` HOT 4
- dyld: lazy symbol binding failed: Symbol not found HOT 2
- An in-range update of webpack is breaking the build 🚨 HOT 21
- Support for 'asset' method which can let you customize the output file name HOT 1
- error in the node v12.16.3 HOT 2
- adding deleteOriginalAssets flag causing build failure HOT 1
- High risk vulnerability in dependencies HOT 3
- Use `engine` field to enforce node version
- Not working since I upgraded to Angular 14
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from brotli-webpack-plugin.