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

feat: utilize cpanminus for perl dependencies if available #3188

Closed
wants to merge 3 commits into from

Conversation

Stealthii
Copy link

cpanminus is broadly compatible with our syntax use of cpan, but can offer additional benefits such as Pkg::Name@Version definitions, as well as built-in modules to reduce the environment requirements on the system when resolving dependencies.

This PR updates the perl dependency resolver to use cpanm if available in the environment. Tests have been updated to ensure operation with either utility, and the Github Actions workflow updated to make the utility available on Ubuntu runners (being already provided in the Windows Perl distribution).

cpanminus is already available in the Windows Perl distribution, but was
missing from the runner for Ubuntu. This updates the runner for the perl
jobs to make cpanm available.
cpanminus is broadly compatible with our syntax use of cpan, but can
offer additional benefits such as Pkg::Name@Version definitions, as well
as built-in modules to reduce the environment requirements on the system
when resolving dependencies.

Tests have been updated to ensure functionality works with either
utility.
Ensure perl dependencies that use Pkg::Name@Version definitions work
with the cpanm utility.
@asottile
Copy link
Member

why would one want to use this? I'm inclined to decline unless there's a very good reason and that should start with an issue first rather than a pull request

@asottile asottile closed this Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants