-
Notifications
You must be signed in to change notification settings - Fork 97
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
minimum perl version not always respected #249
Comments
(possibly related to #232? although this doesn't explain the different behaviour in different environments.) |
also possibly related to #177 |
karenetheridge
added a commit
to joyent/conch-api
that referenced
this issue
Mar 1, 2019
karenetheridge
added a commit
to joyent/conch-api
that referenced
this issue
Mar 1, 2019
karenetheridge
added a commit
to joyent/conch-api
that referenced
this issue
Mar 1, 2019
see perl-carton/carton#249 (cherry picked from commit 184d430)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I haven't been able to trace the cause of this yet, but I'm opening this issue now in the hopes that someone else has seen this or has some insight.
I have a cpanfile with an entry:
requires 'perl', '5.026';
to state a minimum perl requirement of 5.26.x. On a modern system (darwin running 5.26.3 under perlbrew with local::lib and all modules up to date), if I change the minimum perl to something like '5.030', I get an error like:! Installing the dependencies failed: Your Perl (5.026003) is not in the range '5.030'
However, on another system (virtual environment using a prebuilt linux image with perl 5.24 and carton pre-installed), the perl requirement in the cpanfile is totally ignored. I cannot figure out why, as it seems that the installed versions of Carton, cpanm and Menlo are all the same (current). Was there a change to how the perl requirement is detected and checked, perhaps in a different module that is not up to date? If I knew which one, I can check for that at the top of cpanfile as a base tooling requirement.
The text was updated successfully, but these errors were encountered: