tokuhirom / Minilla

Authorizing tool for CPAN modules
https://metacpan.org/release/Minilla
Other
97 stars 65 forks source link

Fail to minil test #228

Open worthmine opened 6 years ago

worthmine commented 6 years ago

If there are unicode values in filename, minil test will fail.

Please see https://github.com/worthmine/Acme-Text-Vicevarsa and below:

Minilla::Error::CommandExit at ~/perl5/perlbrew/perls/perl-5.18.4/lib/site_perl/5.18.4/Minilla/Logger.pm line 56.
Minilla::Logger::errorf("%s\x{a}", "Can't call method \"name\" on an undefined value at (eval 32) l"...) called at ~/perl5/perlbrew/perls/perl-5.18.4/lib/site_perl/5.18.4/Minilla/CLI.pm line 60
Minilla::CLI::catch {...} ("Can't call method \"name\" on an undefined value at (eval 32) l"...) called at ~/perl5/perlbrew/perls/perl-5.18.4/lib/site_perl/5.18.4/Try/Tiny.pm line 124
Try::Tiny::try(CODE(0x7f99d3827cf0), Try::Tiny::Catch=REF(0x7f99d412a940)) called at ~/perl5/perlbrew/perls/perl-5.18.4/lib/site_perl/5.18.4/Minilla/CLI.pm line 63
Minilla::CLI::run(Minilla::CLI=HASH(0x7f99d3827ba0), "test") called at ~/perl5/perlbrew/perls/perl-5.18.4/bin/minil line 6

I don't care if there is no support, because I know that I did the ridiculous.

I just wish that It will be fixed until next April fool.

thanks anyway

skaji commented 6 years ago

I don't think minil should support such distributions.