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

Play backward compatibility issue #403

Merged
merged 1 commit into from
Nov 6, 2014
Merged

Conversation

huntc
Copy link
Contributor

@huntc huntc commented Nov 6, 2014

I've just tried using 1.0.0-M1 with Play and unfortunately it is not possible.

The first thing I've come across is that some APIs have become private e.g.:

https://github.com/sbt/sbt-native-packager/blob/master/src/main/scala/com/typesafe/sbt/packager/archetypes/JavaApp.scala#L111

Play actually uses the makeJarName function (there may be others, but I've not been able to progress beyond this issue so far).

I'm just wondering if we should re-visit what was made private - how concerned are we re. backward compatibility from an API perspective?

Signatures restored so that the existing Play 2.3.x release remains compatible.
@huntc
Copy link
Contributor Author

huntc commented Nov 6, 2014

I've turned this issue into a PR.

I've also released sbt-native-packager as 1.0.0-d818400131a351f0b3b89680b688920f178344fc and tested it against Play 2.4.0-M1. All appears well.

@muuki88 muuki88 added the bug label Nov 6, 2014
@muuki88
Copy link
Contributor

muuki88 commented Nov 6, 2014

Thanks @huntc ! Well, I tried to make it as compatible as poosible. We shouldn't hurt our users, too much :/

muuki88 added a commit that referenced this pull request Nov 6, 2014
Play backward compatibility issue
@muuki88 muuki88 merged commit 017b2c6 into sbt:master Nov 6, 2014
@huntc
Copy link
Contributor Author

huntc commented Nov 6, 2014

Hey you've done a magnificent job re compatibility! This was the only api that I could find Play depending on. It may be worth thinking about some of those other methods that became private though; just in case other projects are using them. WDYT?

@muuki88
Copy link
Contributor

muuki88 commented Nov 6, 2014

Thanks :) Happy it worked out this good.

And yes, we could revisit the methods turning private. However I think there aren't a lot more. The deploymentSettings are currently re added in #389 and the rest got deprecated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants