"zef install A B C" fails to solve missing dependencies

thibaultduponchelle commented 3 years ago

Hello :smiley:

zef says "Failed to resolve some missing dependencies" when installing multiple modules in one time but installs them well when install one after the other.

This 3 lines one after the others are working OK :

$ zef install --/test https://github.com/melezhik/Sparrow6.git
$ zef install --/test https://github.com/melezhik/sparrowdo.git
$ zef install --/test https://github.com/melezhik/sparky.git

But gathering all in one ends in error ("Failed to resolve some missing dependencies"):

$ zef install --/test https://github.com/melezhik/Sparrow6.git https://github.com/melezhik/sparrowdo.git https://github.com/melezhik/sparky.git

I think it is because sparky requires Data::Dump:ver<2.0.0+> when Sparrow6 requires Data::Dump:ver<1.0.0>.

Expected Behavior

I expect to have the modules installed as they are when typing independent commands.

Actual Behavior

"Failed to resolve some missing dependencies"

Environment

raku -v

Welcome to Rakudo(tm) v2020.12-21-g94d685d95.
Implementing the Raku(tm) programming language v6.d.
Built on MoarVM version 2020.12-9-gb8663f53a.

zef list --installed

===> Found via /opt/raku/share/perl6/core
CORE:ver<6.d>:auth<perl>
===> Found via /opt/raku/share/perl6/site
zef:ver<0.9.1>:auth<github:ugexe>:api<0>
===> Found via /home/tduponchelle/.raku
Bailador:ver<0.0.19>:auth<github:Bailador>
DBIish:ver<0.6.2>:auth<github:raku-community-modules>
Data::Dump:ver<0.0.12>:auth<github:tony-o>
Digest::HMAC:ver<1.0.2>:auth<github:retupmoca>
Digest:ver<0.3.4>:auth<Lucien Grondin>
File::Directory::Tree:auth<labster>
File::Find:ver<0.1.1>
File::Temp:ver<0.0.8>
HTML::Escape:ver<0.0.1>
HTTP::Easy:ver<1.1.0>
HTTP::HPACK:ver<0.9.2>
HTTP::MultiPartParser
HTTP::Server::Ogre:ver<0.0.4>
HTTP::Status:ver<0.0.1>
HTTP::Supply:ver<0.5.0>:auth<github:zostay>
Hash::Merge:ver<1.0.0>:auth<github:scriptkitties>:api<1>
Hash::Merge:ver<2.0.0>:auth<github:scriptkitties>:api<2>
IO::Path::ChildSecure:ver<1.001011>
JSON::Class:ver<0.0.15>:auth<github:jonathanstowe>:api<1.0>
JSON::Fast:ver<0.15>
JSON::Marshal:ver<0.0.19>:auth<github:jonathanstowe>:api<1.0>
JSON::Name:ver<0.0.4>:auth<github:jonathanstowe>:api<1.0>
JSON::Tiny:ver<1.0>
JSON::Unmarshal:ver<0.08>
License::SPDX:ver<3.11.0>:auth<github:jonathanstowe>:api<1.0>
Lingua::Conjunction:ver<1.001001>
Log::Any:ver<0.9.5>
META6:ver<0.0.24>:auth<github:jonathanstowe>:api<1.0>
MIME::Base64:ver<1.2.1>:auth<github:retupmoca>
NativeHelpers::Blob:ver<0.1.12>:auth<github:salortiz>
NativeLibs:ver<0.0.8>:auth<github:salortiz>
Number::Denominate:ver<2.002001>
PSGI:ver<1.2.0>:auth<github:raku-community-modules>
Path::Finder:ver<0.3.1>
Sparky:ver<0.0.26>
Sparrow6:ver<0.0.32>
Sparrowdo:ver<0.1.12>
TAP:ver<0.1.0>
Template::Mojo:ver<v.0.1>
Template::Mustache:ver<1.2.2>:auth<github:softmoth>
Terminal::ANSIColor:ver<0.5>
Test::META:ver<0.0.16>:auth<github:jonathanstowe>:api<1.0>
Text::Markdown:ver<1.1.0>:auth<github:retupmoca>
Time::Crontab:ver<1.0.0>
URI::Encode:ver<0.08>
URI:ver<0.3.0>
YAMLish:ver<0.0.5>
zef:ver<0.9.4>:auth<github:ugexe>:api<0>

Thank you

Thibault

ugexe commented 3 years ago

Could be getting swallowed up here: https://github.com/ugexe/zef/blob/3c756183d9490cf6ffe8a854e09b526aab22cd65/lib/Zef/Repository.pm6#L23

a naive solution might be to change it to

    my @unsorted-grouped-candis = @unsorted-candis.categorize({.as}).values;

but I suspect that could have some edge cases around when .as is a name and not a uri (e.g. zef install $url or zef install $path)

thibaultduponchelle commented 3 years ago

(I only tested with url)

ugexe / zef