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

Cross build paradox for sbt 0.13 and 1.0 #147

Merged
merged 2 commits into from
Sep 19, 2017

Conversation

bantonsson
Copy link
Contributor

No description provided.

@raboof
Copy link
Contributor

raboof commented Sep 15, 2017

References #134

Copy link
Member

@pvlugter pvlugter left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@ktoso ktoso left a comment

Choose a reason for hiding this comment

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

Lookin great, can we merge/release?

}},
crossSbtVersions := Seq("0.13.16", "1.0.0"),
// fixed in https://github.com/sbt/sbt/pull/3397 (for sbt 0.13.17)
sbtBinaryVersion in update := (sbtBinaryVersion in pluginCrossBuild).value,
Copy link
Contributor

@raboof raboof Sep 19, 2017

Choose a reason for hiding this comment

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

Thanks for finding this, I've ran into this as well 👍 :)

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, thanks a lot indeed. Used for scalacenter/scalajs-bundler#179

Copy link
Contributor Author

Choose a reason for hiding this comment

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

All credit for that should go to @pvlugter who found it and did the workaround for sbt/sbt-javaagent#9

I can only aspire to reach the same level of sbt-fu!

@pvlugter pvlugter merged commit 9e0a65e into lightbend:master Sep 19, 2017
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.

5 participants