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

target Maven 3.1 #155

Merged
merged 2 commits into from
Dec 16, 2013
Merged

target Maven 3.1 #155

merged 2 commits into from
Dec 16, 2013

Conversation

xcolwell
Copy link
Contributor

Breaking change that is not compatible with Maven < 3.1 .

spoon-maven-plugin: use org.eclipse.aether instead of org.sonatype.aether .
see http://maven.apache.org/docs/3.1.0/release-notes.html

spoon: use android-maven-plugin 3.8.0 which is the lowest version compatible
with Maven 3.1 .

Fix issue: #152

Breaking change that is not compatible with Maven < 3.1 .

spoon-maven-plugin: use org.eclipse.aether instead of org.sonatype.aether .
see http://maven.apache.org/docs/3.1.0/release-notes.html

spoon: use android-maven-plugin 3.8.0 which is the lowest version compatible
with Maven 3.1 .
@@ -173,7 +173,7 @@
<plugin>
<groupId>com.jayway.maven.plugins.android.generation2</groupId>
<artifactId>android-maven-plugin</artifactId>
<version>3.6.0</version>
<version>3.8.0</version>
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not just 3.8.1?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It should be. 3.8.1 just came out. I'll make that change.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ahh, didn't notice this was opened a while ago.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

made the change in 66b1a7e . This pull request should fix Spoon for Maven users

@JakeWharton
Copy link
Collaborator

Can you sign our CLA

@xcolwell
Copy link
Contributor Author

Just signed the CLA

JakeWharton added a commit that referenced this pull request Dec 16, 2013
@JakeWharton JakeWharton merged commit ece50f1 into square:master Dec 16, 2013
@JakeWharton
Copy link
Collaborator

Thanks!

@xcolwell
Copy link
Contributor Author

Thanks for merging this. Spoon has been awesome so far. Thanks!

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