Skip to content
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

Precompilation workload: set "JULIA_PKG_UNPACK_REGISTRY" => nothing #3662

Merged
merged 2 commits into from
Oct 17, 2023

Conversation

DilumAluthge
Copy link
Member

@DilumAluthge DilumAluthge commented Oct 16, 2023

Fixes #3663

@DilumAluthge DilumAluthge marked this pull request as draft October 16, 2023 04:49
@DilumAluthge DilumAluthge marked this pull request as ready for review October 16, 2023 05:06
@DilumAluthge DilumAluthge force-pushed the dpa/fake-registry-precomp-workload branch from a8753cc to 53b2b0c Compare October 16, 2023 05:08
@DilumAluthge DilumAluthge changed the title Precompilation workload: set "JULIA_PKG_UNPACK_REGISTRY" => nothing Precompilation workload: set "JULIA_PKG_UNPACK_REGISTRY" => nothing Oct 16, 2023
@DilumAluthge DilumAluthge force-pushed the dpa/fake-registry-precomp-workload branch from 53b2b0c to c39f6e2 Compare October 16, 2023 18:30
@KristofferC
Copy link
Member

KristofferC commented Oct 16, 2023

How does JULIA_PKG_UNPACK_REGISTRY have anything to do with a modified DEPOT_PATH?

@DilumAluthge
Copy link
Member Author

If you don't modify the depot path, there's no need to precompile Pkg (because you can use the existing cache files), so the precompilation workload never gets run.

If you do modify the depot path, Pkg needs to be precompiled, so the precompilation workload gets run. If the "unpack registries" environment variable is set, this clashes with the "setup" function inside the precompilation workflow, which specifically sets up a non-unpacked registry.

@KristofferC
Copy link
Member

To me, it seems like we should just set some kind of default DEPOT_PATH when running the precompile workload? Otherwise, you won't precompile the whole unpack part, no?

@DilumAluthge
Copy link
Member Author

To me, it seems like we should just set some kind of default DEPOT_PATH when running the precompile workload

I think we already do that:

Pkg.jl/src/precompile.jl

Lines 10 to 11 in ee39026

empty!(DEPOT_PATH)
pushfirst!(DEPOT_PATH, tmp)

@DilumAluthge
Copy link
Member Author

Otherwise, you won't precompile the whole unpack part, no?

Which code path are you talking about precompiling? The default code path (the user doesn't set the JULIA_PKG_UNPACK_REGISTRY environment variable), or the code path I'm hitting (the user has set the JULIA_PKG_UNPACK_REGISTRY environment variable to true)?

@DilumAluthge DilumAluthge force-pushed the dpa/fake-registry-precomp-workload branch from c39f6e2 to 0fd7ed9 Compare October 16, 2023 23:12
@DilumAluthge DilumAluthge force-pushed the dpa/fake-registry-precomp-workload branch from 0fd7ed9 to 3bd14e7 Compare October 17, 2023 02:26
@DilumAluthge DilumAluthge merged commit b39ba05 into master Oct 17, 2023
12 of 13 checks passed
@DilumAluthge DilumAluthge deleted the dpa/fake-registry-precomp-workload branch October 17, 2023 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants