search

thaljef / Pinto

Curate your own repository of Perl modules
https://metacpan.org/module/Pinto::Manual
66 stars 49 forks source link

proposal on the new format of `cpanminus options` parameter for pinto install command #101

Open melezhik opened 11 years ago

melezhik commented 11 years ago

Right now to add cpanminus options into pinto install command one should say something like that:

 pinto install --cpanm-options 'verbose test-only curl'  <some-module>

I'd like to propose alternative or substitution for the given format:

 pinto install --cpanm-options '--verbose --test-only --curl ' <some-module>

IMHO, this way is more convenient because it passes cpanm options as is, without a bit confusion subtraction of dash symbols.

thaljef commented 11 years ago

I like it. Do you think we should continue to support the -L and -l shortcuts that we have now? Or will you have to use the --cpanm-options to set those?

melezhik commented 11 years ago

Do you think we should continue to support the -L and -l shortcuts that we have now? Or will you have to use the --cpanm-options to set those?

this is good question ... I think it's good to leave pinto install -L|-l . As for me when I think about pinto install command firstly I think about installing something into given install-base (then I add -l or -L bits) ... and then I adjust the behavior of install process (if it be more verbose?, or to use curl as http client? or something ... ) - and I add cpanm options (with --cpanm-options string). I'd say that -L and -l shortcuts in someway cpan client agnostic on pinto install side ... of course one can use --cpanm-options to setup install base directly but IMHO it's less natural when installing with pinto install