-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
trim dist #1744
Comments
i prefer leaving the docs in, so they're usable locally (for searching for example). |
Yes, we should include the docs in dist, but rest all should be fixed. For docs, we should also include a pdf of the docs in the distribution. |
I think we can close this, since we got our major gains from generating the unicode files on the fly, rather than packaging them. @JeffBezanson Unless you strongly feel about all the other stuff you mention, could you close this? |
No, we can't close this until we stop packaging multiple copies of libraries, or anything else that is not needed. |
For the static libraries, the simplest thing to do is to |
cc @staticfloat |
related perhaps: I just downloaded rc-4 today (on OSX) and it was ~50MB bigger than a 60 day old nightly I had been using |
We also recently added debug symbols for release builds. I like this when building from source, but for distributions it might make sense to strip them. |
Is there a point to having separate julia and julia-debug if both have debug symbols included? I think the osx nightly from 60 days ago also was using accelerate instead of openblas |
@tkelman: Sorry if this is a dump question but are the debug symbols in a release build really valid? How can that be when the optimizer moves statements wildly around... |
@JeffBezanson We get multiple copies because Debug symbols shouldn't be adding anything significant to the download size. We're duplicating every library that has a tiered versioned name. Copying these as symlinks instead of files should cut down our install size significantly. OSX nightlies will necessarily be at least 30MB larger now that we're packing OpenBLAS alongside. |
@tkelman That's pretty different from other platforms. On Linux, for example, |
PR for fixing some of these issues here: #8024 |
There is a lot of unnecessary stuff in
make dist
:.so
,.so.2
, and.so.2.0.0
. Might be a problem in the arpack build, copying instead of making links perhaps.make install
specifically runsmake -C test/unicode
. Why?We could also remove the docs, since they are available online.
The text was updated successfully, but these errors were encountered: