-
Notifications
You must be signed in to change notification settings - Fork 256
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
Reducing the size of compiled extensions #130
Comments
Is this do to |
@MSP-Greg I am not aware of |
|
@MSP-Greg Thanks! Regarding point 3, I agree but not sure how to patch rbconfig.rb in the context of rubyinstaller2. Is there an example of something like this already being done in this project that I could use as a starting point to make a PR? (I.e. some other files in Ruby distribution getting patched/overwritten?) |
This is regarding oneclick/rubyinstaller2#130 Changing the debugflags didn't have an effect to the binary size.
This is regarding oneclick/rubyinstaller2#130 Changing the debugflags didn't have an effect to the binary size.
This is regarding oneclick/rubyinstaller2#130 Changing the debugflags didn't have an effect to the binary size.
This is regarding oneclick/rubyinstaller2#130 Changing the debugflags didn't have an effect to the binary size.
This is regarding oneclick/rubyinstaller2#130 Changing the debugflags didn't have an effect to the binary size.
This is regarding oneclick/rubyinstaller2#130 Changing the debugflags didn't have an effect to the binary size.
What problems are you experiencing?
I've just noticed that the size of compiled extensions (
.so
files) jumped an order of magnitude going from ruby 2.3.3 to 2.4.4. This seems to be caused by this change inrbconfig.rb
:Reverting this change makes
.so
files an order of magnitude smaller.With
CONFIG["debugflags"] = "-ggdb3"
we get this:With
CONFIG["debugflags"] = "-g"
we get this:Should
debugflags
be changed in rubyinstaller2, or does this have to be done upstream? I'm concerned that getting this into Ruby trunk may be much more cumbersome given multiple platforms etc...The text was updated successfully, but these errors were encountered: