Skip to content
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

02packages resolver should fail loudly if URI doesn't match /(file|https?):\/\// #49

Closed
oschwald opened this issue Nov 7, 2016 · 4 comments

Comments

@oschwald
Copy link

oschwald commented Nov 7, 2016

When using the 02packages resolver with a minicpan on the local filesystem, I accidentally forgot the file:// prefix. cpm install appeared to mostly work except it would fail on FETCH for no specific reason due to Menlo::CLI::Compat thinking that the URI should be requested using the HTTP mirroring rather than the file mirroring. It took embarrassingly long time for me to figure out what was wrong.

@skaji
Copy link
Owner

skaji commented Nov 7, 2016

Can I see the output?

@oschwald
Copy link
Author

oschwald commented Nov 7, 2016

cpm install --verbose -g --cpanfile /home/greg/mm_website/cpanfile --resolver 02packages,/home/greg/mm_website/infrastructure/cpan/minicpan 
1694 DONE resolve   (0.002sec) Math::BigInt -> Math-BigInt-1.999727 (from 02Packages)
1695 DONE resolve   (0.002sec) Pod::Checker -> Pod-Checker-1.73 (from 02Packages)
1696 DONE resolve   (0.002sec) Data::Dumper -> Data-Dumper-2.161 (from 02Packages)
1693 DONE resolve   (0.003sec) Locale::Codes -> Locale-Codes-3.40 (from 02Packages)
1694 FAIL fetch     (0.012sec) Math-BigInt-1.999727
1693 FAIL fetch     (0.011sec) Locale-Codes-3.40
1696 FAIL fetch     (0.014sec) Pod-Checker-1.73
1695 FAIL fetch     (0.015sec) Data-Dumper-2.161
FAIL install D/DU/DUMMY/Data-Dumper-2.161.tar.gz
FAIL install D/DU/DUMMY/Locale-Codes-3.40.tar.gz
FAIL install D/DU/DUMMY/Math-BigInt-1.999727.tar.gz
FAIL install D/DU/DUMMY/Pod-Checker-1.73.tar.gz

@skaji
Copy link
Owner

skaji commented Nov 7, 2016

thanks. I will look into it today.

@skaji skaji closed this as completed in 52b3bee Nov 8, 2016
@oschwald
Copy link
Author

oschwald commented Nov 8, 2016

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants