Most packages with artifacts are not relocatable

[jonathan-conder-sm]

Would anything break if I added track_content = true to this line?

julia/stdlib/Artifacts/src/Artifacts.jl

Line 696 in 999dde7

Base.include_dependency(artifacts_toml)

Related: #53906 and/or #51798

[KristofferC]

I don't see why anything should break.

[vchuravy]

@jonathan-conder-sm do you want to open that PR?

[jonathan-conder-sm]

Sure, sorry I've been busy and forgot about this. See #54426

I'm curious, if this is unlikely to break anything then why is track_content not the default?

[fatteneder]

The default track_content = false was initially chosen, because that's how it used to work before. In particular, some folks on discourse, and I think also Oscar.jl, relied on the mtime check to trigger recompilation.
Admittedly, its a moot argument, because we already changed the behavior for include away from using mtime, and it was also changed how include_dependency itself deals with directories.

I think that not many pkgs would notice the change, because its seldomly used.
If someone would stand up to represent this as a bug fix, then I guess we could change the default to true.