-
Notifications
You must be signed in to change notification settings - Fork 29
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
Delete selected libs from cache to shrink size #43
Delete selected libs from cache to shrink size #43
Conversation
Delete these libs on windows, where we are over the nuget size limit. Context: dotnet#34 (comment) That should affect build times when using [debug sanitizers](https://emscripten.org/docs/debugging/Sanitizers.html), which is not a common scenario I think.
We are still too big, 251MB |
@lewing that gets us to 250MB. Is it enough or do we need remove more libs? |
Now I noticed some of the |
@lewing ok, that gets us to 242MB |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm assuming this will cause no problems with the default build but have we actually tested it?
Ok now that we have a way to ship should we try putting the cache in a separate package and populating it with everything? |
Delete these libs on windows, where we are over the nuget size limit.
Context: #34 (comment)
That should affect build times when using
debug sanitizers
which is not used by default builds.
Also remove
mt
libs. We don't support threads yet anyway.