Skip to content

Conversation

@SethTisue
Copy link
Member

No description provided.

@SethTisue
Copy link
Member Author

oops, a second run still gives:

change detected (content changed, file is created, or file is deleted):
/home/travis/.ivy2/local/org.scala-lang.modules/scala-xml_2.11/1.0.6-SNAPSHOT/bundles/scala-xml_2.11.jar
/home/travis/.ivy2/local/org.scala-lang.modules/scala-xml_2.11/1.0.6-SNAPSHOT/bundles/scala-xml_2.11.jar.md5
/home/travis/.ivy2/local/org.scala-lang.modules/scala-xml_2.11/1.0.6-SNAPSHOT/bundles/scala-xml_2.11.jar.sha1
/home/travis/.ivy2/local/org.scala-lang.modules/scala-xml_2.11/1.0.6-SNAPSHOT/docs/scala-xml_2.11-javadoc.jar
/home/travis/.ivy2/local/org.scala-lang.modules/scala-xml_2.11/1.0.6-SNAPSHOT/docs/scala-xml_2.11-javadoc.jar.md5
/home/travis/.ivy2/local/org.scala-lang.modules/scala-xml_2.11/1.0.6-SNAPSHOT/docs/scala-xml_2.11-javadoc.jar.sha1
/home/travis/.ivy2/local/org.scala-lang.modules/scala-xml_2.11/1.0.6-SNAPSHOT/ivys/ivy.xml
/home/travis/.ivy2/local/org.scala-lang.modules/scala-xml_2.11/1.0.6-SNAPSHOT/ivys/ivy.xml.md5
/home/travis/.ivy2/local/org.scala-lang.modules/scala-xml_2.11/1.0.6-SNAPSHOT/ivys/ivy.xml.sha1
/home/travis/.ivy2/local/org.scala-lang.modules/scala-xm

I'll fix...

@SethTisue
Copy link
Member Author

SethTisue commented Feb 4, 2017

all good now, I think

screen shot 2017-02-04 at 9 57 03 am 👍

looks like this cuts the build time from almost 4 minutes to about 2 minutes.

@SethTisue
Copy link
Member Author

SethTisue commented Feb 7, 2017

I'll take the thumbs-ups and the lack of other feedback as sufficient goahead to merge.

@SethTisue SethTisue merged commit 9cb7579 into scala:master Feb 7, 2017
@SethTisue SethTisue deleted the travis-caching branch February 7, 2017 21:58
@ashawley
Copy link
Member

ashawley commented Feb 7, 2017

Indeed. I rebased a bunch of PRs, and it's a lot faster.

https://travis-ci.org/scala/scala-xml/pull_requests

@gourlaysama
Copy link

Awesome! 👍
I shamelessly did the same over at scala/scala-parser-combinators#100.

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.

3 participants