Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Add build config for Travis CI #127

Closed
wants to merge 1 commit into from

5 participants

@eirslett

Let's see if Finagle can pass green outside of Twitter?

challenge

@caniszczyk
Owner

Funny, I've actually been working on this today too...

https://travis-ci.org/caniszczyk/finagle/builds/3746169

@caniszczyk
Owner

I believe there's an issue of Travis not supporting 64bit VMs so we're dead in the water ATM

@eirslett

Speaking of the devil! :+1:

Strange... what happens if you make Travis build it with maven instead of sbt? If that's relevant?

@mariusae
Owner

We should just adjust the options in sbt. There's no reason this can't run on a 32-bit JVM.

@caniszczyk
Owner

@mariusaeriksen, is there a reason we should even bundle SBT?

I think it's reasonable to expect people to have it installed locally.

@mariusae
Owner

@caniszczyk yes. First, we're not actually bundling sbt; the included sbt script is a bootstrapping script. So, in order to build these projects, all that is required is a JRE install. Pretty nice.

But more importantly: sbt versioning is really confusing. You might have the simultaneous need of sbts 0.7.x, 0.11.x, 0.12.x. And not only that, sbt will output only cryptic errors if you use one sbt version when you should have used another.

Simply being able to build finagle with

$ ./sbt compile

is a wonderfully simple thing, in my mind.

@eirslett

https://travis-ci.org/eirslett/finagle/builds/3761932

[info] Loading project definition from /home/travis/builds/eirslett/finagle/project

19[info] Updating {file:/home/travis/builds/eirslett/finagle/project/}default-d291f6...

20[info] Resolving com.twitter#sbt-package-dist;1.0.5 ...

21[warn] module not found: com.twitter#sbt-package-dist;1.0.5

@mosesn
Owner

This change is very similar to #252 which will be merged shortly. Closing for now, thanks for kicking this off!

@mosesn mosesn closed this
@evnm

Better late than never.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 19, 2012
  1. @eirslett
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 0 deletions.
  1. +3 −0  .travis.yml
View
3  .travis.yml
@@ -0,0 +1,3 @@
+language: scala
+scala:
+ - 2.9.2
Something went wrong with that request. Please try again.