Another --mirror-only 02packages.details.txt parsing problem

[brycebaril]

This is another carton-related issue triggered by the --mirror-only flag.

It looks like the fix for #188 may have broken parsing for a different class of pathological entries in the package details files:

02packages.details.txt

TimeDate                              undef  G/GB/GBARR/TimeDate-1.20.tar.gz

It looks like it may need to special case the string "undef" when choosing what the version is.

Searching TimeDate on mirror index /home/bryce/carton2/.carton/02packages.details.txt ...
Found TimeDate version  < 1.20.
Searching TimeDate on mirror http://cpan.metacpan.org ...
Downloading index file http://cpan.metacpan.org/modules/02packages.details.txt.gz ...
-> FAIL Finding TimeDate (1.20) on mirror http://cpan.metacpan.org failed.
Searching TimeDate on mirror http://backpan.perl.org ...
Downloading index file http://backpan.perl.org/modules/02packages.details.txt.gz ...
-> FAIL Finding TimeDate (1.20) on mirror http://backpan.perl.org failed.
-> FAIL Couldn't find module or a distribution TimeDate (1.20)
Searching Date::Format on mirror index /home/bryce/carton2/.carton/02packages.details.txt ...

[miyagawa]

i think this is rather a bug in carton side which creates a bogus 1.20 requirement for a module/package that doesn't actually exist (TimeDate).

[brycebaril]

This one might be a carton bug-- I ended up with the TimeDate requirement because my cpanfile requiring Date::Format -- that installs fine by installing TimeDate 1.20, but then the carton.lock file that is written as requiring TimeDate instead of Date::Format.

Subsequent installs using carton install based on the carton.lock file will look like this, i.e. they actually work but the return status is a failure:

Installing modules using cpanfile
! Couldn't find module or a distribution TimeDate (1.20)
Successfully installed TimeDate-1.20
1 distribution installed
Installing modules failed

[miyagawa]

This is an unfortunate bug in the toolchain that creates a module "name" out of MYMETA which has the value of "TimeDate" which actually doesn't exist as a module/package.

There's a couple of modules that are known to provide such a bogus NAME, and carton should probably have to work around that.

[miyagawa]

I reported this as a bug to TimeDate: https://rt.cpan.org/Public/Bug/Display.html?id=83141

but i will need to work around on carton level for modules like this.

[miyagawa]

Should probably be addressed by Carton v.0.9_8

[brycebaril]

Thanks! I'll try it out.