-
-
Notifications
You must be signed in to change notification settings - Fork 5.6k
build: cleanup base files inclusion #45441
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
Conversation
195cad3
to
112e7e6
Compare
Ensures the out-of-tree build has the correct files installed, and simplifies some of the `include`s used in Base. Fixes a return value that was relied on (largely unnecessarily) by a loading test, despite being invalid before now. Fixes #31140
112e7e6
to
075e3db
Compare
I will assume people are okay with this, and take review comments afterwards if not |
This reverts commit f0b1c5f.
I don't really understand what the point of moving the files from |
It sort of is random, in that the alternate versions I tried seemed to have more problems with make or Revise getting confused. I didn't really want to nest them this way for any design reason. |
Revise is very confused right now :). |
Revise is fixable (see timholy/Revise.jl#694), but it would be worth knowing whether this change is going to be reconsidered. I wouldn't choose an unfortunate directory structure in a (failed) attempt to keep Revise happy, I'd rather see this solved as it should be and Revise can adapt accordingly. Revise always breaks with each point release, and this is comparatively easy. It's the changes that affect JuliaInterpreter that are scary. |
Quick thoughts from @vtjnash? Should we change this directory organization or should I merge timholy/Revise.jl#694? |
I would very much like to change the directory structure. This change confuses not just Revise but also VSCode, grep, and other tools. |
The directory structure is probably fine, but symlinking |
I'll hold off on the Revise change then. I'll also defer to others in implementing the change since everyone involved in this conversation has a better sense for what needs to be done than me. We'll update Revise when it's done. (@pfitzseb feel free to just modify your PR and merge it whenever you wish.) |
This reverts commit f0b1c5f.
This reverts commit f0b1c5f.
Ensures the out-of-tree build has the correct src files installed for the base image, and simplifies some of the
include
s used in Base.Fixes #31140