Add a travis-yml to create a travis-ci build for jmake

Review Request #1833 — Created Feb. 25, 2015 and submitted

zundel
jmake
zundel/add-travis-yml
77a91a1...
pants-reviews
dturner-tw, jsirois
Add a travis-yml to create a travis-ci build for this project

Validated the file on http://docs.travis-ci.com/user/travis-lint/

ZU
JS
  1. I turned on Travis for pantsbuild/jmake here: https://travis-ci.org/profile/pantsbuild
    You should be able to do a pull request now and make sure it goes green before this hits master.
    1. Meh, creating a Pull Request isn't triggering travis-ci. I pushed to master.

    2. I pushed this change to master @ 1eeefb9 It will probably not work, I'll keep banging on it.

  2. .travis.yml (Diff revision 1)
     
     
    The last sentence should go.
  3. 
      
ZU
JS
  1. Ship It!
  2. 
      
DT
  1. 
      
  2. .travis.yml (Diff revision 2)
     
     

    nit: unconditionally

  3. .travis.yml (Diff revision 2)
     
     

    For me, ant test wants a findbugs jar to be at a certain location. Does it not do that for you?

    1. Ack - should be ant import test

    2. Actually: `ant -Dimport.dir=build-support import test` and cache the build-support dir.
  4. 
      
DT
  1. Ship It!
  2. 
      
ZU
ZU
  1. Travis seems pretty confused about this project. The pull requests aren't running.

    1. IIRC there is an "after-the-switch-flip" correlation.  I'd try creating a new PR with a new sha-set if you haven't already.  The new sha-set could just come from patching this RB into a new branch and tweaking the commit message.
    2. Maybe its catching up. As soon as I made a new PR, I got a message in email for the master build that failed over an hour ago. I did already create a new PR but its still in limbo. I'll wait longer this time.

  2. 
      
JS
  1. The import/cache changes LGTM.
  2. 
      
ZU
Review request changed

Status: Closed (submitted)

Change Summary:

This is starting to catch real problems. Submitted as cd70a62

Loading...