enso-org / enso

Hybrid visual and textual functional programming.
https://enso.org
Apache License 2.0
7.31k stars 317 forks source link

Lower the size of the parser library in VSCode #10295

Closed JaroslavTulach closed 2 weeks ago

JaroslavTulach commented 2 weeks ago

Pull Request Description

Uses --release when building the enso_parser libraries to make the code smaller:

when compressed.

Checklist

Please ensure that the following checklist has been satisfied before submitting the PR:

JaroslavTulach commented 2 weeks ago

Based on this advice by Kaz.

JaroslavTulach commented 2 weeks ago

The current sizes of the libraries are:

-rw-r--r-- 1 runner docker 1487528 Jun 17 06:46 ./enso_parser/aarch64/libenso_parser.dylib
-rw-r--r-- 1 runner docker 1169920 Jun 17 06:46 ./enso_parser/enso_parser.dll
-rw-r--r-- 1 runner docker    2520 Jun 17 06:46 ./enso_parser/enso_parser.dll.exp
-rw-r--r-- 1 runner docker    4582 Jun 17 06:46 ./enso_parser/enso_parser.dll.lib
-rw-r--r-- 1 runner docker 3324936 Jun 17 06:46 ./enso_parser/libenso_parser.so
-rw-r--r-- 1 runner docker 1560048 Jun 17 06:46 ./enso_parser/x86_64/libenso_parser.dylib

still the ./enso_parser/libenso_parser.so is bigger than it should be...

JaroslavTulach commented 2 weeks ago

still the ./enso_parser/libenso_parser.so is bigger than it should be...

Is that the result of using musl?

kazcw commented 2 weeks ago

still the ./enso_parser/libenso_parser.so is bigger than it should be...

Is that the result of using musl?

Probably, yes. It's the cost of wide Linux portability.