Skip to content
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

Node.js needs to lose weight #43811

Closed
sbfkcel opened this issue Jul 13, 2022 · 3 comments
Closed

Node.js needs to lose weight #43811

sbfkcel opened this issue Jul 13, 2022 · 3 comments
Labels
feature request Issues that request new features to be added to Node.js.

Comments

@sbfkcel
Copy link

sbfkcel commented Jul 13, 2022

What is the problem this feature will solve?

Binaries are getting bigger and bigger.
From the original 30MB to nearly 100MB,

The warehouse image on docker contains too much, from the original 100MB to nearly 1GB, please remove those programs except Node.js. (Example: Python)

Overly large programs seem to be far from elegant runtimes.

What is the feature you are proposing to solve the problem?

Reduce the size of Node.js and exclude other programs.

What alternatives have you considered?

No response

@sbfkcel sbfkcel added the feature request Issues that request new features to be added to Node.js. label Jul 13, 2022
@bnoordhuis
Copy link
Member

Chesterton's fence: those megabytes and programs are probably there for a reason. The beauty of open source is that with some code spelunking you can find out why. I'll provide clues: V8, i18n, add-ons.

@mvduin
Copy link

mvduin commented Jul 13, 2022

A while back I did some testing to try to figure out why the nodejs binary got so much larger between 12 and 14, and unicode tables turned out to be a ridiculously large part of it. These are some nodejs v14.17.2 armhf (armv7+neon, thumb) executable sizes I got while fiddling with various options:

61M v8_enable_object_print=0
60M v8_enable_object_print=0 v8_enable_inspector=0
29M v8_enable_object_print=0 v8_enable_inspector=0 v8_enable_i18n_support=0
36M icu_small=true v8_enable_inspector=0
37M icu_small=true

@bnoordhuis
Copy link
Member

OP has had some time to follow up with more concrete proposals but didn't so I'll go ahead and close this.

@bnoordhuis bnoordhuis closed this as not planned Won't fix, can't repro, duplicate, stale Jul 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Issues that request new features to be added to Node.js.
Projects
None yet
Development

No branches or pull requests

3 participants