add .travis.yml to build on TravisCI #48

Open
wants to merge 2 commits into
from

Projects

None yet

4 participants

@KengoTODA
Contributor

This PR is a suggestion to introduce TravisCI to this project. Here is an example:

To build on Travis CI, organization owner needs to sign-in to Travis CI and configure to enable Travis CI for findbugsproject organization.

@KengoTODA KengoTODA add .travis.yml to build on TravisCI
To build on Travis CI, account owner needs to sign-in and configure
to enable Travis CI for findbugsproject organization.
48eb99f
@KengoTODA KengoTODA and 2 others commented on an outdated diff Sep 20, 2015
@@ -0,0 +1,13 @@
+language: java
+sudo: false
+jdk:
+ - oraclejdk7
+install:
+ - wget http://www.eclipse.org/downloads/download.php?file=/eclipse/downloads/drops/R-3.6.2-201102101200/eclipse-SDK-3.6.2-linux-gtk-x86_64.tar.gz -O eclipse-3.6.2-linux64.tar.gz
+ - tar xzvf eclipse-3.6.2-linux64.tar.gz eclipse
+ - echo eclipsePlugin.dir=$(pwd)/eclipse/plugins > eclipsePlugin/local.properties
+script: ant build test
+notifications:
+ email:
+ recipients:
+ - skypencil+travis@gmail.com
@KengoTODA
KengoTODA Sep 20, 2015 Contributor

Travis CI can send a mail to report build result, but we need to give mail address here. If we do not need it, simply remove notification.

@iloveeclipse
iloveeclipse Sep 21, 2015 Member

I like this idea. One question: currently we compile with 1.7 but test on both vm versions (1.8 and 1.7). Is this configurable?

@iloveeclipse
iloveeclipse Sep 21, 2015 Member

The mail if possible should go to the developers mailing list or github team account.

@amaembo
amaembo Sep 21, 2015 Contributor

In my personal project I have no mails configured, but they are nevertheless sent automatically to me when build is failed or new branch is built. I guess, project members will receive the mails automatically here as well.

@KengoTODA
KengoTODA Sep 21, 2015 Contributor

we compile with 1.7 but test on both vm versions (1.8 and 1.7). Is this configurable?

Technically possible, but now our Ant build fails on Java 1.8.

@iloveeclipse
iloveeclipse Sep 21, 2015 Member

The compile task fails. Tests should be OK to run with Java 8.

@amaembo
amaembo Sep 21, 2015 Contributor

As I understand, every single build on Travis starts from the scratch. Thus probably we cannot use FindBugs binary built on Java 1.7 to test it with Java 1.8. Probably it would be good to enable build with Java 1.8 when some ant property is set (like -DENABLE_JAVA_8_BUILD=true). I guess it fails currently just because we should not accidentally publish FindBugs binaries which are built by Java-8. With some explicit property it should be ok.

@iloveeclipse
iloveeclipse Sep 21, 2015 Member

Yep. The intent was to avoid any unforeseen publishing of Java 8 build artifacts.

@KengoTODA
KengoTODA Sep 21, 2015 Contributor

Or we may want to jdk_switcher to test on Java 8 after we build on Java 7. I will try it later.

@amaembo
Contributor
amaembo commented Sep 21, 2015

I use travis-ci on my personal GitHub project and have positive experience with it. Thus I support this change, though I guess only project administrator (Bill Pugh) can properly setup the travis account.

@KengoTODA
Contributor

I have updated .travis.yml, to test on both of oraclejdk7 and oraclejdk8. Here is result of build.

@farnulfo

Nice !

@amaembo
Contributor
amaembo commented Oct 6, 2015

I requested travis access from project owner (Bill Pugh). No response so far.

@iloveeclipse iloveeclipse referenced this pull request in spotbugs/spotbugs Nov 6, 2016
Closed

Build tools #4

@jsotuyod jsotuyod added a commit to spotbugs/spotbugs that referenced this pull request Nov 6, 2016
@jsotuyod jsotuyod Add initial Travis support 9a92ae5
@jsotuyod jsotuyod referenced this pull request in spotbugs/spotbugs Nov 6, 2016
Merged

Add initial Travis support #12

@jsotuyod jsotuyod added a commit to spotbugs/spotbugs that referenced this pull request Nov 6, 2016
@jsotuyod jsotuyod Add initial Travis support d24518f
@iloveeclipse iloveeclipse added a commit to spotbugs/spotbugs that referenced this pull request Nov 6, 2016
@jsotuyod @iloveeclipse jsotuyod + iloveeclipse Add initial Travis support (#12) afc1ecb
@jsotuyod jsotuyod referenced this pull request in spotbugs/spotbugs Feb 9, 2017
Open

Absorb existing findbugs pull requests? #120

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment