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
CompUnit::Repository::Installation.files appears to be broken #4820
Comments
From @jonathanstoweHi, [jonathan@cannibal p6-Sys-Utmp]$ perl6 -e 'for $*REPO.repo-chain -> $v { say $v.perl; say $v.files("foo").perl }' This will, as far as I understand it, need to work in order for modules to be able to find native .so files they are providing for themselves. |
From @jonathanstoweIt seems that this is https://github.com/rakudo/rakudo/blob/nom/src/core/CompUnit/Repository/Installation.pm#L198 I'm not sure whether $name should be defined or should be set to some sensible default, |
Just for the record, this is probably the stable link to the relevant line: https://github.com/rakudo/rakudo/blob/104623a21a/src/core/CompUnit/Repository/Installation.pm#L198 The whole class I can't reproduce the error with a current Rakudo. (I've added a conditional to the last
IMHO this is closable without adding a test. If you think otherwise, please reopen (or ideally add such a test). |
As you say there is a lot of water under the bridge since then. With the application of a |
Migrated from rt.perl.org#126806 (status was 'new')
Searchable as RT126806$
The text was updated successfully, but these errors were encountered: