Skip to content
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

Use maven-3.5.0 in travis builds #492

Merged
merged 2 commits into from
Jun 19, 2017
Merged

Use maven-3.5.0 in travis builds #492

merged 2 commits into from
Jun 19, 2017

Conversation

merlimat
Copy link
Contributor

Motivation

Use latest version of maven in Travis builds, which along with multiple fixes, introduces colored output for builds.

@merlimat merlimat added this to the 1.19 milestone Jun 19, 2017
@merlimat merlimat requested review from rdhabalia, jai1 and saandrews June 19, 2017 17:04
# Upgrade to maven 3.5.0
before_install:
- export M2_HOME=$HOME/apache-maven-3.5.0
- if [ ! -d $M2_HOME/bin ]; then curl https://archive.apache.org/dist/maven/maven-3/3.5.0/binaries/apache-maven-3.5.0-bin.tar.gz | tar zxf - -C $HOME; fi
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tar zxf - -C $HOME extra -?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I copied the command, but I believe the - is to tell tar to read from stdin

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see.

Copy link
Contributor

@rdhabalia rdhabalia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@merlimat merlimat force-pushed the master branch 2 times, most recently from 58583fa to abbdb82 Compare June 19, 2017 17:35
@merlimat merlimat merged commit e1a703f into apache:master Jun 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants