search

Perl5-Alien / Alien-Base

Base classes for Alien:: modules (deprecated, see Alien-Build)
Other
35 stars 19 forks source link

Allow overriding repo config from ENV #180

Closed nshp closed 7 years ago

nshp commented 7 years ago

Allows e.g.:

ALIEN_OPENSSL_REPO_FTP_HOST=example.com cpanm Alien::OpenSSL
wbraswell commented 7 years ago

Seems legit?

plicease commented 7 years ago

Going to do some testing, but I think I largely approve. I appreciate the documentation on the environment variables, which are also buried in the documentation here:

https://metacpan.org/pod/distribution/Alien-Base/lib/Alien/Base/ModuleBuild/API.pod#PACKAGE-AND-ENVIRONMENT-VARIABLES

plicease commented 7 years ago

I did some testing with my own Alien::FFI, and I vote for inclusion. I will merge this Thursday if there aren't any objections before then.

plicease commented 7 years ago

Rebased and merged. I am going to send this to CPAN now.