-
Notifications
You must be signed in to change notification settings - Fork 464
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
Normalize the registry files #65997
Normalize the registry files #65997
Conversation
7f6f6b0
to
b7f2349
Compare
I'll create a whitespace only change first so that the diff here will be easier to see. Edit: Done. |
b7f2349
to
c026f90
Compare
I've verified the compat read from this PR is the same as the other one by running the following function on this branch and the commit before it: using Pkg
function compare(regpath1, regpath2)
reg1 = Pkg.Registry.RegistryInstance(regpath1)
reg2 = Pkg.Registry.RegistryInstance(regpath2)
for (uuid, _) in reg1
entry_1 = reg1[uuid]
entry_2 = reg2[uuid]
info_1 = Pkg.Registry.registry_info(entry_1)
info_2 = Pkg.Registry.registry_info(entry_2)
compat_info_1 = Pkg.Registry.compat_info(info_1)
compat_info_2 = Pkg.Registry.compat_info(info_2)
if compat_info_1 != compat_info_2
error("mismatch in pkg $(entry_1.name)")
end
end
end We have also run a PkgEval run with this branch with no registry surprises (JuliaLang/julia#45540 (comment)). I would therefore claim this is good to merge. |
|
458: Registry consistency tests: the compress roundtrip check should compare `VersionRange`s (not `String`s) r=DilumAluthge a=DilumAluthge Fixes #457 Will likely fail until JuliaRegistries/General#65997 is merged. Co-authored-by: Dilum Aluthge <dilum@aluthge.com>
Various manual changes and tools have made the registry a bit un-normalized in how whitespace etc is used. This normalizes it by loading all files and resaving them.
Perhaps this should be made into RegistryCI to check that doing this is a no-op.
We should hold off on merging this for a little bit until we verify that things look good.
The code for this is: