-
-
Notifications
You must be signed in to change notification settings - Fork 180
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Harder to install 0.16.0 due to dependency on brotli-size #202
Comments
for extra brotli-size build requirement ref: siddharthkp/bundlesize#202
These pastebin links are 404ing right now. Here is a 2019 update: If the current behavior is a bug, please provide the steps to reproduce. The following log
What is the expected behavior? |
Brotli is available in core since Node 11 https://nodejs.org/docs/latest-v11.x/api/zlib.html |
@styfle There's great. That's going to help make it optional 🎉 |
Hi! Thank you for a great tool :-)
Do you want to request a feature or report a bug?
Ease of use regression between 0.15.3 and 0.16.0.
What is the current behavior?
#194 (released in 0.16.0) added support for tracking the brotli sizes of files, however to do so it had to add a dependency on brotli-size, which itself has a dependency on iltorb.
If iltorb isn't able to find a pre-compiled binary, it has to build using node-gyp, which requires Python and g++ be installed - which often aren't in Docker images, or in Windows environments.
For users that aren't making use of the brotli size feature, it would be a shame to have to install Python/g++ just to allow bundlesize to install. Perhaps
brotli-size
could be made an optional dependency?It's worth also noting that:
brotli-size
dependency doubled the install size and package dependency count of bundlesize (taking it up to ~200 dependencies!)no-bin-links
feature (necessary when using npm/yarn inside a linux Vagrant environment using shared folders with a Windows host). I'll be filing an issue against iltorb shortly for this. Edit: filed "detect-libc: not found" error when using npm/yarn --no-bin-links option nstepien/iltorb#58If the current behavior is a bug, please provide the steps to reproduce.
docker run --rm -it node:9-slim yarn add [email protected] --dev
...which results in: https://emorley.pastebin.mozilla.org/9078461
What is the expected behavior?
Successful install, like with 0.15.3:
docker run --rm -it node:9-slim yarn add [email protected] --dev
...which results in: https://emorley.pastebin.mozilla.org/9078471
Please mention other relevant information.
The text was updated successfully, but these errors were encountered: