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

Updated .travis.yml. #1

Merged
merged 2 commits into from Aug 13, 2019

Conversation

@hakonhagland
Copy link
Contributor

commented Aug 13, 2019

Hello, this is from pullrequest.club!

I have updated .travis.yml to use the Ubuntu 16.04 (Xenial) build environment. In this environment perl versions before 5.22 are not available, see
https://travis-ci.community/t/failure-with-perl-5-16-5-18-5-20/2458

Also from Perl 5.22, Module::Build is not in core, see:
https://metacpan.org/pod/release/RJBS/perl-5.22.0-RC2/pod/perldelta.pod
so added that as a before_install dependency in .travis.yml.

Updated .travis.yml.
Updated .travis.yml to use the Ubuntu 16.04 (Xenial) build environment.
In this environment perl versions before 5.22 is not available, see
https://travis-ci.community/t/failure-with-perl-5-16-5-18-5-20/2458

Also from Perl 5.22, Module::Build is not in core, see:
https://metacpan.org/pod/release/RJBS/perl-5.22.0-RC2/pod/perldelta.pod
so added that as a before_install dependency in .travis.yml.
@davorg

This comment has been minimized.

Copy link
Owner

commented Aug 13, 2019

Hi. Thanks for taking on one of my most unused CPAN modules :-)

I'd really like to keep testing my modules on those older versions of Perl, so I'd prefer to add this line:

dist: trusty

which tells Travis to use an older distribution where those versions are still available. Alternatively, I've heard about something called "the matrix" which seems to solve the problem in a more flexible way. I haven't read much about it yet, but here's an example:

matrix:
  include:
    - perl: “5.28”
    - perl: “5.20”
      dist: trusty

I guess that means "test on 5.28 on the default distribution, but use Trusty for 5.20". That would be my preferred solution.

@davorg

This comment has been minimized.

Copy link
Owner

commented Aug 13, 2019

In fact, here's a version we could steal :-)

https://github.com/mojolicious/mojo/pull/1396/files

Added build matrix.
Added build matrix in travis.yml to be able to test older versions of perl.
@hakonhagland

This comment has been minimized.

Copy link
Contributor Author

commented Aug 13, 2019

Thanks for the tip! Sound like a good idea, let's see if this works :)

@davorg

This comment has been minimized.

Copy link
Owner

commented Aug 13, 2019

@davorg davorg merged commit e3d1035 into davorg:master Aug 13, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.