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
Remove buildpath from line information #54134
Comments
FWIW, we already do some dynamic rewriting of stdlib path so that they are valid on the local system: # Path is valid locally
julia> @which REPL.symbol_latex("s")
symbol_latex(s::String)
@ REPL ~/.julia/juliaup/julia-1.10.2+0.aarch64.apple.darwin14/share/julia/stdlib/v1.10/REPL/src/docview.jl:403
julia> m = @which REPL.symbol_latex("s");
# What is actually stored is the build path
julia> m.file
Symbol("/Users/julia/.julia/scratchspaces/a66863c6-20e8-4ff4-8a62-49f30b1f605e/agent-cache/default-honeycrisp-HL2F7YQ3XH.0/build/default-honeycrisp-HL2F7YQ3XH-0/julialang/julia-release-1-dot-10/usr/share/julia/stdlib/v1.10/REPL/src/docview.jl") |
We already do that too, but the problem is the paths are wrong for the REPL files in the distributed code (as noted in this issue exactly) so the replacement doesn't happen. And it only applies to Julia code, not native code (which may have come from binary builder or system library built on some deb-builder cluster):
|
So we currently store absolut paths in debuginfo for Julia and it's library. Locally this might be fine, but for our build artifacts this is rather pointless.
@KristofferC raised this recently as:
The buildpath of
/Users/julia/.julia/scratchspaces/a66863c6-20e8-4ff4-8a62-49f30b1f605e/agent-cache/default-honeycrisp-HL2F7YQ3XH.0/build/default-honeycrisp-HL2F7YQ3XH-0/julialang/julia-release-1-dot-11
is long and at least for me unecessary.There are two parts of this.
An example for 2 comes from:
We already do this for
Base
so maybe we can change that latter path to@REPL
?The bigger challenge here is to make sure that system profilers, "click on path" in REPL and other things keep working.
So we might need to remap files to local directories for presentation.
While discussing with @KristofferC I noticed that GCC/Clang support remapping paths using
-ffile-prefix-map
.I locally tried the following patch:
and the test-case:
Opening this as an issue instead of a PR since I suspect this will need a larger conversation that also includes changes to BinaryBuilder.
cc: @topolarity @vtjnash @giordano @staticfloat
The text was updated successfully, but these errors were encountered: