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 #82 adding the ability to specify jvm options via sbt #510

Merged
merged 1 commit into from
Mar 28, 2015

Conversation

muuki88
Copy link
Contributor

@muuki88 muuki88 commented Mar 1, 2015

Adding the ability to specify javaOptions via build.sbt.

javaOptions in Linux ++= Seq(
  "-J-Xmx64m", "-J-Xms64m", "-Dproperty=true",  
)  

@muuki88
Copy link
Contributor Author

muuki88 commented Mar 2, 2015

@kardapoltsev can you have a look at this if the tests all succeed?

@kardapoltsev
Copy link
Member

I'll try a bit later.

@kardapoltsev
Copy link
Member

At the first look seems good.

@muuki88
Copy link
Contributor Author

muuki88 commented Mar 22, 2015

This one is a huge one. I added all things discussed in #473

@dhardy92 , @kardapoltsev , @knshiro can you take a look, if you have time.

@muuki88 muuki88 force-pushed the wip/jvmopts-in-build branch 2 times, most recently from 478d519 to 6c4c030 Compare March 22, 2015 18:06
Java App
~~~~~~~~

Create a file ``src/universal/conf/jvmopts`` (gets automatically added to the package mappings)
Copy link

Choose a reason for hiding this comment

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

Thejvmopts file is a bit confusing since you changed it to application.ini.

@muuki88 muuki88 added the universal Zip, tar.gz, tgz and bash issues label Mar 23, 2015
muuki88 added a commit that referenced this pull request Mar 28, 2015
FIX #82 adding the ability to specify jvm options via sbt
@muuki88 muuki88 merged commit 48b0c19 into master Mar 28, 2015
@muuki88 muuki88 deleted the wip/jvmopts-in-build branch March 28, 2015 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
debian rpm universal Zip, tar.gz, tgz and bash issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants