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

Travis-CI changes and build improvements #18

Merged
merged 15 commits into from Mar 2, 2015
Merged

Travis-CI changes and build improvements #18

merged 15 commits into from Mar 2, 2015

Conversation

zmughal
Copy link
Member

@zmughal zmughal commented Mar 1, 2015

The branch travis-ci-more makes several changes:

View code:

Summary:

  1. @mohawk2 has worked on incorporating the code for the pre-req report from Dist::Zilla::Plugin::Test::ReportPrereqs into a test that can be used to dump the versions of the pre-reqs used. This also helps with finding the versions of the packages during the Travis-CI build.
  2. There is an additional Travis-CI build which skips installing gfortran so that PDL can be tested in the absence of a Fortran compiler. This addresses the issue SF#346 ExtUtils::F77 dependency causing problems for CPAN install.
  3. @mohawk2 made fixes the tests so that they behave properly when WITH_BADVAL=0 and all the corresponding Travis-CI builds now succeed which addresses SF#347, GH#39.
  4. Travis-CI now gives IRC notifications to irc.perl.org/#pdl when the build succeeds or fails.
  5. Several changes including parallel builds and testing so that Travis-CI finishes faster. This speeds up the overall build by 25%.

Code review thread on pdl-devel: http://thread.gmane.org/gmane.comp.lang.perl.pdl.devel/6328

mohawk2 and others added 15 commits February 28, 2015 22:07
The current test suite does not work well when BADVAL is disabled, so
for now, we allow them to fail without causing the whole build to be
marked as failed.
This will allow it to start before all the other tests so that it has
more time to run while the other builds finish in parallel.
This greatly decreases the time to finish all the Travis-CI builds.
Though the output will appear out-of-order, which interferes with using
a diff on the build log, this can always be fixed after the fact by
using a TAP parser.
We had previously allowed builds to succeed even though some tests under
BADVAL=0 were failing. Now that Ed has made fixes to those tests, the
builds are now re-enabled to cause failures.
This will allow it to start before the rest of the builds and work in
parallel with the faster non-coverage builds.
The default build for Perl v5.20 is duplicated by one of the variations
of the BADVAL configurations.
@coveralls
Copy link

Coverage Status

Coverage increased (+1.6%) to 51.84% when pulling aea6254 on travis-ci-more into 255adcf on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+1.6%) to 51.84% when pulling aea6254 on travis-ci-more into 255adcf on master.

@mohawk2
Copy link
Member

mohawk2 commented Mar 2, 2015

Builds and passes on Win 7, 5.20.

@mohawk2 mohawk2 merged commit aea6254 into master Mar 2, 2015
@mohawk2 mohawk2 deleted the travis-ci-more branch March 2, 2015 23:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants