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
[PATCH] f8210d2 Add a simple .travis.yml #14558
Comments
From yoanlin93@gmail.comThis patch adds a simple .travis.yml config file Summary: People want to contribute to perl don't have a machine for contiuous This is a bug report for perl from yoanlin93@gmail.com, From f8210d2226c534ffd5b7701c27b6c6cd968df741 Mon Sep 17 00:00:00 2001 This is a multi-part message in MIME format. .travis.yml | 28 ++++++++++++++++++++++++++++ --------------2.1.2 Inline Patchdiff --git a/.travis.yml b/.travis.yml
new file mode 100644
index 0000000..aa8a493
--- /dev/null
+++ b/.travis.yml
@@ -0,0 +1,28 @@
+language: c
+
+compiler:
+ - gcc
+
+matrix:
+ include:
+ - compiler: gcc
+ env: CONFIGURE_OPTION='-Duse64bitall' COVERALLS=yes VALGRIND=no DEBUG=yes
+ - compiler: gcc
+ env: CONFIGURE_OPTION='-Duse64bitall -Dusethreads' COVERALLS=yes VALGRIND=no DEBUG=yes
+ - compiler: clang
+
+install:
+ - sudo apt-get update -qq
+ - sudo apt-get install -qq automake pkg-config build-essential libtool automake autoconf m4 gnulib
+ - sudo apt-get build-dep perl perl-base
+
+before_script:
+ - ./Configure -des -Dusedevel $CONFIGURE_OPTION -Dprefix=$HOME/perl-blead
+script:
+ - make
+after_script:
+cache:
+ apt: true
--------------2.1.2--
---
Site configuration information for perl 5.21.10: Configured by c9s at Fri Feb 27 12:59:15 CST 2015. Summary of my perl5 (revision 5 version 21 subversion 10) configuration: @INC for perl 5.21.10: Environment for perl 5.21.10: |
From @tonycozOn Wed Mar 04 00:21:41 2015, yoanlin93@gmail.com wrote:
git complained that the patch was corrupted when I extracted it from your message. I created the .travis.yml file and ran some tests and made some adjustments to produce the attached. The changes: - perl's build process doesn't pay any attention to the CC environment variable, - run the test suite and install and test that with perlivp - trim the installed dependencies, perl doesn't use the GNU auto* tools. Unfortunatly io/eintr.t fails on the travis VMs. The real question is whether we want this in the tree... Tony |
From @tonycoz0001-perl-123981-Travis-CI-support.patchFrom f8e9d3554ecf0d8542651c3e91a6102adf783990 Mon Sep 17 00:00:00 2001
From: Tony Cook <tony@develop-help.com>
Date: Thu, 5 Mar 2015 14:16:10 +1100
Subject: [perl #123981] Travis-CI support
Build and test perl under four different configurations.
This isn't compatible with "container based infrastructure"
I didn't use test_harness and TEST_JOBS since the four jobs specified
are already run in parallel on the virtual host.
io/eintr.t fails on the travis VMs.
---
.travis.yml | 26 ++++++++++++++++++++++++++
MANIFEST | 1 +
2 files changed, 27 insertions(+)
create mode 100644 .travis.yml
diff --git a/.travis.yml b/.travis.yml
new file mode 100644
index 0000000..f36617b
--- /dev/null
+++ b/.travis.yml
@@ -0,0 +1,26 @@
+language: c
+
+compiler:
+ - gcc
+
+matrix:
+ include:
+ - compiler: gcc
+ env: CONFIGURE_OPTION='-Duse64bitall'
+ - compiler: gcc
+ env: CONFIGURE_OPTION='-Duse64bitall -Dusethreads'
+ - compiler: clang
+
+install:
+ - sudo apt-get update -qq
+ - sudo apt-get install -qq build-essential libtool
+ - sudo apt-get build-dep perl perl-base
+
+before_script:
+ - ./Configure -des -Dusedevel -Uversiononly -Dcc=$CC $CONFIGURE_OPTION -Dprefix=$HOME/perl-blead -DDEBUGGING
+script:
+ - make test && make install && $HOME/perl-blead/bin/perlivp
+after_script:
+cache:
+ apt: true
+
diff --git a/MANIFEST b/MANIFEST
index aad1be4..5e0d672 100644
--- a/MANIFEST
+++ b/MANIFEST
@@ -5481,6 +5481,7 @@ t/porting/regen.t Check that regen.pl doesn't need running
t/porting/ss_dup.t Check that sv.c:ss_dup handle everything
t/porting/test_bootstrap.t Test that the instructions for test bootstrapping aren't accidentally overlooked.
t/porting/utils.t Check that utility scripts still compile
+.travis.yml continuous integration on github (where enabled)
t/README Instructions for regression tests
t/re/charset.t See if regex modifiers like /d, /u work properly
t/re/fold_grind.t See if case folding works properly
--
1.7.10.4
|
The RT System itself - Status changed from 'new' to 'open' |
From @tonycozSee https://travis-ci.org/tonycoz/perl for some sample builds. Tony |
From yoanlin93@gmail.comHi Tony, Thanks! since we have the perl5 repository mirror on GitHub, there is a About the io/eintr.t failure, I guess maybe we can just skip this test by Cheers, On Thu, Mar 5, 2015 at 11:16 AM, Tony Cook via RT <perlbug-followup@perl.org
|
From yoanlin93@gmail.comHi Tony, I added some improvements based on your changes: I am now waiting the tests to pass here:
|
From yoanlin93@gmail.comyada! the tests pass now! https://travis-ci.org/c9s/perl5/jobs/53149445 <https://travis-ci.org/c9s/perl5/jobs/53149445>
|
From yoanlin93@gmail.comThis is a bug report for perl from yoanlin93@gmail.com, From bd6c821336721c806bdac349cfda810734886c19 Mon Sep 17 00:00:00 2001 This is a multi-part message in MIME format. Summary: This is for skipping some unnecessary / container incompatible tests for - Add PERL_SKIP_AUTHORS_TEST, PERL_SKIP_MANIFEST_TEST environment variable - Skip eintr.t test when TRAVIS environment variable is defined. - Skip authors.t, manifest.t and pending-author.t when testing on Travis-CI - Clean up env vars - Document travis ci environment variable - Turn off optimization to get faster build .travis.yml | 14 +++++++++----- --------------2.1.2 Inline Patchdiff --git a/.travis.yml b/.travis.yml
index f36617b..38dd30b 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -3,6 +3,9 @@ language: c
compiler:
- gcc
+# Environment variable definitions can be found here:
+# http://docs.travis-ci.com/user/environment-variables/
+
matrix:
include:
- compiler: gcc
@@ -10,17 +13,18 @@ matrix:
- compiler: gcc
env: CONFIGURE_OPTION='-Duse64bitall -Dusethreads'
- compiler: clang
-
+ env: CONFIGURE_OPTION='-Duse64bitall -Dusethreads'
install:
- sudo apt-get update -qq
- sudo apt-get install -qq build-essential libtool
- sudo apt-get build-dep perl perl-base
-
before_script:
- - ./Configure -des -Dusedevel -Uversiononly -Dcc=$CC $CONFIGURE_OPTION -Dprefix=$HOME/perl-blead -DDEBUGGING
+ - ./Configure -des -Dusedevel -Uversiononly -Uoptimize -Dcc=$CC $CONFIGURE_OPTION -Dprefix=$HOME/perl-blead -DDEBUGGING
+ - make -j4
script:
- - make test && make install && $HOME/perl-blead/bin/perlivp
+ - PERL_SKIP_AUTHORS_TEST=1 PERL_SKIP_MANIFEST_TEST=1 TEST_JOBS=4 make test
+ - make install
+ - $HOME/perl-blead/bin/perlivp
after_script:
cache:
apt: true
-
diff --git a/t/io/eintr.t b/t/io/eintr.t
index ca15232..b2d800c 100644
--- a/t/io/eintr.t
+++ b/t/io/eintr.t
@@ -11,8 +11,12 @@ BEGIN {
@INC = '../lib';
require './test.pl';
skip_all_without_dynamic_extension('Fcntl');
+
+ # The environment variable is defined in http://docs.travis-ci.com/user/ci-environment/
+ skip_all("eintr tests can't be run inside travis-CI VMs") if $ENV{TRAVIS};
}
+
use warnings;
use strict;
use Config;
diff --git a/t/porting/authors.t b/t/porting/authors.t
index bc69e3a..58ec201 100644
--- a/t/porting/authors.t
+++ b/t/porting/authors.t
@@ -10,6 +10,8 @@ use strict;
require 't/test.pl';
find_git_or_skip('all');
+skip_all("Skip authors test") if $ENV{PERL_SKIP_AUTHORS_TEST};
+
# This is the subset of "pretty=fuller" that checkAUTHORS.pl actually needs:
my $quote = $^O =~ /^mswin/i ? q(") : q(');
system("git log --pretty=format:${quote}Author: %an <%ae>%n${quote} | $^X Porting/checkAUTHORS.pl --tap -");
diff --git a/t/porting/manifest.t b/t/porting/manifest.t
index 01cbd1c..6a9f83e 100644
--- a/t/porting/manifest.t
+++ b/t/porting/manifest.t
@@ -35,6 +35,7 @@ require 't/test.pl';
skip_all("Cross-compiling, the entire source might not be available")
if $Config{usecrosscompile};
+skip_all("Skip manifest tests") if $ENV{PERL_SKIP_MANIFEST_TEST};
plan('no_plan');
diff --git a/t/porting/pending-author.t b/t/porting/pending-author.t
index 4dceaf6..f2135ec 100644
--- a/t/porting/pending-author.t
+++ b/t/porting/pending-author.t
@@ -30,6 +30,8 @@ use File::Spec;
require 't/test.pl';
find_git_or_skip('all');
+skip_all("Skip author tests") if $ENV{PERL_SKIP_AUTHORS_TEST};
+
my $devnull = File::Spec->devnull;
my $changes;
foreach (`git status --porcelain 2>$devnull`) {
--------------2.1.2--
---
Site configuration information for perl 5.21.10: Configured by c9s at Thu Mar 5 12:30:20 CST 2015. Summary of my perl5 (revision 5 version 21 subversion 10) configuration: Locally applied patches: @INC for perl 5.21.10: Environment for perl 5.21.10: |
From yoanlin93@gmail.comI rebased the commits again on GitHub https://github.com/c9s/perl5/commit/a1cbaee25de2410442532373760ee6273478cbdc <c9s/perl5@a1cbaee25de2410442532373760ee6273478cbdc> Diff:
|
From net@profvince.comLe 05/03/2015 00:27, Yo-An Lin a écrit :
Please note that the github mirror is not maintained by the perl5 About that change particularly, the porters already maintains two ways Vincent |
From @rjbs* Vincent Pit <net@profvince.com> [2015-03-05T07:17:57]
Not addressing anything but the above: The mirror that GitHub was running has been shut down at our request, some time -- |
From yoanlin93@gmail.comHi Ricardo, Will we merge this patch? Cheers,
|
From @tonycozOn Wed Mar 04 22:34:15 2015, yoanlin93@gmail.com wrote:
Please attach patches rather than inlining them.
The manifest test shouldn't be skipped. Skipping the author tests can be a problem with older gits, since you can end up with nonsense email addresses in the commits. Also, I think a more general mechanism in t/TEST should be created to skip tests, so we might set: PERL_SKIP_TESTS='porting/authors.t io/eintr.t' to skip those two tests, rather than modifying each test we want to skip.
This should probably be: - ./Configure -des -Dusedevel -Uversiononly -Uoptimize -Dcc=$CC $CONFIGURE_OPTION -Dprefix=$HOME/perl-blead -DDEBUGGING && make -j4 so that the make is only run if the Configure is successful.
make test ignores TEST_JOBS, you need make test_harness for parallel testing.
Again, these should be joined by &&, since there's no point in running perlivp if make install failed. Tony |
From yoanlin93@gmail.comHI Tony, hmmm sadly I found that I can't reply on rt.perl.org, so I just copied the content and reply in a new mail:
I was using perlbug to send the patch, doesn't it work for you? if so, I can send another one.
Sounds great, I will do it instead of skipping the test separately.
Actually Travis-CI will stop to run the reset commands if any command failed, separating them into a list improves the readibility and does the same thing. :)
|
From yoanlin93@gmail.comI just updated the patch in the attachment. please take a look. |
From yoanlin93@gmail.com0001-Improvement-for-testing-on-Travis-CI.patchFrom 63874524d98a8b52048671e152ed01d637943d55 Mon Sep 17 00:00:00 2001
From: c9s <yoanlin93@gmail.com>
Date: Sat, 4 Apr 2015 19:36:11 +0800
Subject: [PATCH] Improvement for testing on Travis-CI
Summary:
This is for skipping some unnecessary / container incompatible tests for
perl5 forks on GitHub :
- Added PERL_SKIP_TESTS environment variable to specify tests to skip.
Usage:
PERL_SKIP_TESTS="t/porting/authors.t t/porting/manifest.t t/porting/pending-author.t" \
TEST_JOBS=4 make test_harness
- Skip eintr.t test when TRAVIS environment variable is defined.
the test isn't compatible with Travis-CI machine (container based infrastructure)
- Run `make test_harness` instead of `make test` since `make test`
doesn't know `TEST_JOBS`
- Update .travis.yml changes according to the suggestions from TonyC
- Clean up env vars
- Document travis ci environment variable
---
.travis.yml | 13 +++++++++----
t/io/eintr.t | 4 ++++
t/porting/authors.t | 2 ++
t/porting/manifest.t | 1 +
t/porting/pending-author.t | 2 ++
5 files changed, 18 insertions(+), 4 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index f36617b..48d923f 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -3,6 +3,9 @@ language: c
compiler:
- gcc
+# Environment variable definitions can be found here:
+# http://docs.travis-ci.com/user/environment-variables/
+
matrix:
include:
- compiler: gcc
@@ -10,17 +13,19 @@ matrix:
- compiler: gcc
env: CONFIGURE_OPTION='-Duse64bitall -Dusethreads'
- compiler: clang
-
+ env: CONFIGURE_OPTION='-Duse64bitall -Dusethreads'
+ - compiler: clang
install:
- sudo apt-get update -qq
- sudo apt-get install -qq build-essential libtool
- sudo apt-get build-dep perl perl-base
-
before_script:
- ./Configure -des -Dusedevel -Uversiononly -Dcc=$CC $CONFIGURE_OPTION -Dprefix=$HOME/perl-blead -DDEBUGGING
script:
- - make test && make install && $HOME/perl-blead/bin/perlivp
+ - make -j4
+ - PERL_SKIP_TESTS="t/porting/authors.t t/porting/manifest.t t/porting/pending-author.t" TEST_JOBS=4 make test_harness
+ - make install
+ - $HOME/perl-blead/bin/perlivp
after_script:
cache:
apt: true
-
diff --git a/t/io/eintr.t b/t/io/eintr.t
index ca15232..b2d800c 100644
--- a/t/io/eintr.t
+++ b/t/io/eintr.t
@@ -11,8 +11,12 @@ BEGIN {
@INC = '../lib';
require './test.pl';
skip_all_without_dynamic_extension('Fcntl');
+
+ # The environment variable is defined in http://docs.travis-ci.com/user/ci-environment/
+ skip_all("eintr tests can't be run inside travis-CI VMs") if $ENV{TRAVIS};
}
+
use warnings;
use strict;
use Config;
diff --git a/t/porting/authors.t b/t/porting/authors.t
index bc69e3a..2661b90 100644
--- a/t/porting/authors.t
+++ b/t/porting/authors.t
@@ -10,6 +10,8 @@ use strict;
require 't/test.pl';
find_git_or_skip('all');
+skip_all("Skip authors test") if grep /$0/i, split(/\s+/, $ENV{PERL_SKIP_TESTS});
+
# This is the subset of "pretty=fuller" that checkAUTHORS.pl actually needs:
my $quote = $^O =~ /^mswin/i ? q(") : q(');
system("git log --pretty=format:${quote}Author: %an <%ae>%n${quote} | $^X Porting/checkAUTHORS.pl --tap -");
diff --git a/t/porting/manifest.t b/t/porting/manifest.t
index 01cbd1c..bb627d2 100644
--- a/t/porting/manifest.t
+++ b/t/porting/manifest.t
@@ -35,6 +35,7 @@ require 't/test.pl';
skip_all("Cross-compiling, the entire source might not be available")
if $Config{usecrosscompile};
+skip_all("Skip manifest tests") if grep /$0/i, split(/\s+/, $ENV{PERL_SKIP_TESTS});
plan('no_plan');
diff --git a/t/porting/pending-author.t b/t/porting/pending-author.t
index 4dceaf6..c5774f5 100644
--- a/t/porting/pending-author.t
+++ b/t/porting/pending-author.t
@@ -30,6 +30,8 @@ use File::Spec;
require 't/test.pl';
find_git_or_skip('all');
+skip_all("Skip author tests") if grep /$0/i, split(/\s+/, $ENV{PERL_SKIP_TESTS});
+
my $devnull = File::Spec->devnull;
my $changes;
foreach (`git status --porcelain 2>$devnull`) {
--
2.1.2
|
From yoanlin93@gmail.com
|
From @jkeenanOn Sat Apr 04 04:43:59 2015, yoanlin93@gmail.com wrote:
TonyC: Could we get an update on this ticket (which is in the "consider for 5.23.1" meta ticket (https://rt-archive.perl.org/perl5/Ticket/Display.html?id=124092)? Thank you very much. -- |
From @rjbsI am probably on board with this patch. I don't think it will hurt, and might help. Why does it skip some porting tests? I thought they'd work with or without a git checkout present, and if possible I'd like to see people being told they didn't update AUTHORS, for example. -- |
From @tonycozOn Sat Apr 04 04:31:39 2015, yoanlin93@gmail.com wrote:
Actually it continues to run, see https://travis-ci.org/tonycoz/perl/jobs/79207906 where I deliberately failed, and the process continued to install and run perlivp. Tony |
From @tonycozOn Fri Jun 12 15:12:28 2015, rjbs wrote:
The attached updates the whole patch: - don't skip porting tests - make the stages of the build require the previous stages success Tony |
From @tonycoz0001-perl-123981-Travis-CI-support.patchFrom 9bfad6c14935dd0be3c6d03cfe8ad60209b49b58 Mon Sep 17 00:00:00 2001
From: Tony Cook <tony@develop-help.com>
Date: Tue, 8 Sep 2015 15:40:54 +1000
Subject: [perl #123981] Travis-CI support
Build and test perl under four different configurations.
This isn't compatible with "container based infrastructure"
Skip io/eintr.t since it fails on the travis VMs.
make the build process depend on the previous steps, since travis does
NOT stop the build process if a step fails
This is largely based on work done by "c9s <yoanlin93@gmail.com>"
---
.travis.yml | 26 ++++++++++++++++++++++++++
MANIFEST | 1 +
t/io/eintr.t | 3 +++
3 files changed, 30 insertions(+)
create mode 100644 .travis.yml
diff --git a/.travis.yml b/.travis.yml
new file mode 100644
index 0000000..842931e
--- /dev/null
+++ b/.travis.yml
@@ -0,0 +1,26 @@
+language: c
+
+compiler:
+ - gcc
+
+matrix:
+ include:
+ - compiler: gcc
+ env: CONFIGURE_OPTION='-Duselongdouble'
+ - compiler: gcc
+ env: CONFIGURE_OPTION='-Duse64bitall -Dusethreads'
+ - compiler: clang
+ env: CONFIGURE_OPTION='-Duselongdouble'
+ - compiler: clang
+ env: CONFIGURE_OPTION='-Duse64bitall -Dusethreads'
+
+install:
+ - sudo apt-get update -qq
+ - sudo apt-get install -qq build-essential libtool
+
+script:
+ - ./Configure -des -Dusedevel -Uversiononly -Dcc=$CC $CONFIGURE_OPTION -Dprefix=$HOME/perl-blead -DDEBUGGING && TEST_JOBS=2 make -j2 test_harness && make install && $HOME/perl-blead/bin/perlivp
+after_script:
+cache:
+ apt: true
+
diff --git a/MANIFEST b/MANIFEST
index 870d729..7ac09e9 100644
--- a/MANIFEST
+++ b/MANIFEST
@@ -5420,6 +5420,7 @@ t/porting/regen.t Check that regen.pl doesn't need running
t/porting/ss_dup.t Check that sv.c:ss_dup handle everything
t/porting/test_bootstrap.t Test that the instructions for test bootstrapping aren't accidentally overlooked.
t/porting/utils.t Check that utility scripts still compile
+.travis.yml continuous integration on github (where enabled)
t/README Instructions for regression tests
t/re/charset.t See if regex modifiers like /d, /u work properly
t/re/fold_grind.t See if case folding works properly
diff --git a/t/io/eintr.t b/t/io/eintr.t
index 1a81cdf..19527c2 100644
--- a/t/io/eintr.t
+++ b/t/io/eintr.t
@@ -11,6 +11,9 @@ BEGIN {
@INC = '../lib';
require './test.pl';
skip_all_without_dynamic_extension('Fcntl');
+
+ # The environment variable is defined in http://docs.travis-ci.com/user/ci-environment/
+ skip_all("eintr tests can't be run inside travis-CI VMs") if $ENV{TRAVIS};
}
use warnings;
--
2.1.4
|
From dennis@kaarsemaker.netOn ma, 2015-09-07 at 22:50 -0700, Tony Cook via RT wrote:
Having played a bit with travis, I think the patch could use some - Uses the container based builds and better dependencies -- |
From dennis@kaarsemaker.net0001-Travis-integration.patchFrom 8ee8fa0918fa0bc1ec555ee7fd0f797167c05d23 Mon Sep 17 00:00:00 2001
From: Dennis Kaarsemaker <dennis@kaarsemaker.net>
Date: Thu, 22 Oct 2015 18:30:11 +0200
Subject: [PATCH] Travis integration
---
.travis.yml | 55 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
MANIFEST | 1 +
2 files changed, 56 insertions(+)
create mode 100644 .travis.yml
diff --git a/.travis.yml b/.travis.yml
new file mode 100644
index 0000000..721fa2c
--- /dev/null
+++ b/.travis.yml
@@ -0,0 +1,55 @@
+language: c
+
+# Here we build our build matrix
+os:
+ - linux
+ - osx
+
+compiler:
+ - clang
+ - gcc
+
+env:
+ - CONFIGURE_ARGS='-Uusethreads'
+ - CONFIGURE_ARGS='-Dusethreads'
+ - CONFIGURE_ARGS='-Duselongdouble'
+ - CONFIGURE_ARGS='-Duse64bitall'
+ - CONFIGURE_ARGS='-Dusethreads -Duselongdouble'
+ - CONFIGURE_ARGS='-Dusethreads -Duse64bitall'
+ - CONFIGURE_ARGS='-Dusethreads -Duse64bitall -Duselongdouble'
+
+script:
+ # CC is exported by travis based on compuler section above
+ # CONFIGURE_ARGS is set in the env section above
+ - ./Configure -de -Dusedevel -Dcc=$CC -Dld=$CC $CONFIGURE_ARGS -Dprefix=$HOME/perl-blead && make -j4 && TEST_JOBS=4 make test_harness && make install && $HOME/perl-blead/bin/perlivp
+
+# No testing of random branches, just blead, maint and smoke-me/*
+branches:
+ only:
+ - blead
+ - /^maint-5.*$/
+ - /^smoke-me.*$/
+
+# Minimal notifications (well, almost: email: false and leaving out the irc
+# section would disable notifications completely)
+notifications:
+ email:
+ recipients:
+ - perl5-porters@perl.org
+ on_failure: change
+ irc:
+ channels:
+ - "irc.perl.org#p5p"
+ on_failure: change
+
+# The linux builds run on Ubuntu 12.04 docker instances and may need additional
+# packages installed.
+addons:
+ apt:
+ packages:
+ - file
+ - cpio
+ - libdb-dev
+ - libgdbm-dev
+ - zlib1g-dev
+ - libbz2-dev
diff --git a/MANIFEST b/MANIFEST
index 3c788d9..ff34eb5 100644
--- a/MANIFEST
+++ b/MANIFEST
@@ -5438,6 +5438,7 @@ t/porting/regen.t Check that regen.pl doesn't need running
t/porting/ss_dup.t Check that sv.c:ss_dup handle everything
t/porting/test_bootstrap.t Test that the instructions for test bootstrapping aren't accidentally overlooked.
t/porting/utils.t Check that utility scripts still compile
+.travis.yml Travis CI integration
t/README Instructions for regression tests
t/re/charset.t See if regex modifiers like /d, /u work properly
t/re/fold_grind.t See if case folding works properly
--
2.6.2-327-gb5fc588
|
From @tonycozOn Thu Oct 22 11:30:43 2015, dennis@kaarsemaker.net wrote:
+# No testing of random branches, just blead, maint and smoke-me/* This won't necessarily be used by perl5-porters members, it might be used by someone outside who's preparing a patch to be submitted. I don't think we should expect such a user to name their branch to match the above to get travis coverage. +# Minimal notifications (well, almost: email: false and leaving out the irc Similar to the above, the user might not know about the IRC channel and might not be a member of the list. + # CC is exported by travis based on compuler section above "compiler". Tony |
From @khwilliamsonOn Sun Dec 06 15:54:29 2015, tonyc wrote:
Any progress on this? |
From @tonycozOn Sat Aug 06 16:47:46 2016, khw wrote:
Here's a new patch. One issue the restrictions in Seveas's patch brings up - if we add Travis doesn't seem to be smart enough to avoid re-testing duplicate So if we add this, and someone enables travis on their repository and does a mirror, it's going to start a ridiculous number of probably The restrictions from Seveas' patch limits that to blead/maint/smoke-me, but that last can be a fairly large list. Tony |
From @tonycoz0001-perl-123981-Travis-CI-support.patchFrom 57e4d871864da993f757f3d6079c49793602d534 Mon Sep 17 00:00:00 2001
From: Tony Cook <tony@develop-help.com>
Date: Mon, 8 Aug 2016 15:11:55 +1000
Subject: [perl #123981] Travis-CI support
This is largely based on work done by "c9s <yoanlin93@gmail.com>" and
Dennis Kaarsemaker (Seveas)
---
.travis.yml | 31 +++++++++++++++++++++++++++++++
MANIFEST | 1 +
2 files changed, 32 insertions(+)
create mode 100644 .travis.yml
diff --git a/.travis.yml b/.travis.yml
new file mode 100644
index 0000000..d1fdd23
--- /dev/null
+++ b/.travis.yml
@@ -0,0 +1,31 @@
+language: c
+
+os:
+ - linux
+ - osx
+
+compiler:
+ - gcc
+ - clang
+
+env:
+ - CONFIGURE_ARGS='-Uusethreads'
+ - CONFIGURE_ARGS='-Dusethreads'
+ - CONFIGURE_ARGS='-Duselongdouble'
+ - CONFIGURE_ARGS='-Duse64bitall'
+ - CONFIGURE_ARGS='-Dusethreads -Duselongdouble'
+ - CONFIGURE_ARGS='-Dusethreads -Duse64bitall'
+ - CONFIGURE_ARGS='-Dusethreads -Duse64bitall -Duselongdouble'
+
+script:
+ - ./Configure -des -Dusedevel -Uversiononly -Dcc=$CC $CONFIGURE_ARGS -Dprefix=$HOME/perl-blead -DDEBUGGING && TEST_JOBS=4 make -j4 test_harness && make install && $HOME/perl-blead/bin/perlivp
+
+addons:
+ apt:
+ packages:
+ - file
+ - cpio
+ - libdb-dev
+ - libgdbm-dev
+ - zlib1g-dev
+ - libbz2-dev
diff --git a/MANIFEST b/MANIFEST
index 4cec28b..ead1f9a 100644
--- a/MANIFEST
+++ b/MANIFEST
@@ -1,4 +1,5 @@
.dir-locals.el Emacs control file
+.travis.yml continuous integration on github (where enabled)
amigaos4/amigaio.c AmigaOS4 port
amigaos4/amigaio.h AmigaOS4 port
amigaos4/amigaos.c AmigaOS4 port
--
2.1.4
|
From @tonycozOn Sun, 07 Aug 2016 23:41:27 -0700, tonyc wrote:
This doesn't appear to be a problem unless maybe if you try to git push --mirror. Forking a repo using the github UI won't run travis (since you can't have travis enabled yet.) I've pushed my patch as 3135e2b. Tony |
@tonycoz - Status changed from 'open' to 'pending release' |
From @khwilliamsonThank you for filing this report. You have helped make Perl better. With the release yesterday of Perl 5.28.0, this and 185 other issues have been Perl 5.28.0 may be downloaded via: If you find that the problem persists, feel free to reopen this ticket. |
@khwilliamson - Status changed from 'pending release' to 'resolved' |
Migrated from rt.perl.org#123981 (status was 'resolved')
Searchable as RT123981$
The text was updated successfully, but these errors were encountered: