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

Fix deprecation warnings in windows WixHelper #726

Closed
muuki88 opened this issue Jan 15, 2016 · 7 comments
Closed

Fix deprecation warnings in windows WixHelper #726

muuki88 opened this issue Jan 15, 2016 · 7 comments
Labels

Comments

@muuki88
Copy link
Contributor

muuki88 commented Jan 15, 2016

Referring to scala/scala-dist#158 the WixHelper has deprecation warnings since more than two years. We should either fix this or remove the deprecated annotation.

cc: @metasim @nazoking @bjuric

@metasim
Copy link
Member

metasim commented Jan 20, 2016

IMO, the WiX support overall could be considered deprecated in favor of JDKPackager (which allows you to use WiX if you want).

@metasim
Copy link
Member

metasim commented Jan 20, 2016

@muuki88 I added the deprecation description to remove another warning asking that deprecations have descriptions and dates. I think I guessed at the reason for the deprecation, and never asked @jsuereth. My suggestion, since the plugin is obviously working for some people, is to just remove the deprecation warning, converting it to an inline "TODO".

@SethTisue
Copy link
Member

yes please

@muuki88
Copy link
Contributor Author

muuki88 commented Nov 10, 2016

@SethTisue happy to merge a pull request 😊

@SethTisue
Copy link
Member

happy to merge a pull request

a PR that does "remove the deprecation warning, converting it to an inline "TODO""? I can do that

@SethTisue
Copy link
Member

PR: #909

@muuki88
Copy link
Contributor Author

muuki88 commented Nov 12, 2016

Will release 1.1.5 and a 1.2.0 Milestone today.

littleRoundaer added a commit to littleRoundaer/sbt-native-packager that referenced this issue Jul 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants