Warning "The Julia package 'Downloads' cannot be loaded" when starting GAP.jl in CI

[fingolfin]

This is printed by JuliaImportPackage.

That in turn is called by GAP's PackageManage which has a Download.g file containing this:

if IsBound(Julia) and JuliaImportPackage("Downloads") = true then

[fingolfin]

Perhaps turning up the PackageLoadingMessage to PACKAGE_DEBUG will give some insight.

Also perhaps we should take advantage of that code in PackageManager to remove the hack where we patch PackageManager in src/packages.jl, function init_packagemanager.

[ThomasBreuer]

Thanks for the hint about the JuliaImportPackage call in a GAP package.
This explains why a git grep JuliaImportPackage in GAP.jl did not answer the question who calls this function with "Downloads".

I think that this problem will disappear when we simply omit the JuliaImportPackage call from the function in PackageManager, and change its code such that it calls Julia.GAP.Packages.Downloads.downloads instead of Julia.Downloads.downloads.
And I think that this function must be changed anyhow because it does not contain the change from /pull/961.

(Still it would be interesting why the warning appears only in certain CI runs.)

[thofma]

I also saw this locally:

Precompiling project...
  11 dependencies successfully precompiled in 76 seconds. 77 already precompiled. 1 skipped during auto due to previous errors.
  1 dependency had output during precompilation:
┌ Hecke → GAPExt
│  #I  The Julia package 'Downloads' cannot be loaded.
└