docker-perl-tester icon indicating copy to clipboard operation
docker-perl-tester copied to clipboard

Provide App::perlimports to the image

Open bschmalhofer opened this issue 1 year ago • 3 comments

Hi,

I am fond of perlimports for cleaning up my import statements. Would it make sense to add that module to the image? The list of dependencies looks a bit daunting though.

Edit: The issue with DBD::ODBC was unrelated.

bschmalhofer avatar Mar 19 '24 10:03 bschmalhofer

I see no objections to add it, but also note that you can add extra packages using a cpanfile with cpm or cpanm actions

Note: there is one example described here https://github.com/Perl/docker-perl-tester?tab=readme-ov-file#using-the-images-with-github-workflow and the two actions are documented there:

  • https://github.com/perl-actions/install-with-cpanm
  • https://github.com/perl-actions/install-with-cpm

atoomic avatar Mar 19 '24 16:03 atoomic

I am fond of perlimports for cleaning up my import statements

❤️

We could add it for perl >= '5.18.0'. It would pull in a lot of dependencies, but I'm not sure how bad that would be, considering how many dependencies these builds already contain.

oalders avatar Mar 19 '24 17:03 oalders

I see no objections to add it, but also note that you can add extra packages using a cpanfile with cpm or cpanm actions

Yes, that is what I do in https://github.com/RotherOSS/otobo/blob/5c899d77c7d9e33b9471ca26fcbfbc60a9ed68d1/.github/workflows/code_policy.yml#L60 . This works fine, so adding App::perlimports would be just for convenience.

bschmalhofer avatar Mar 19 '24 19:03 bschmalhofer