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

CODENVY-88 Setup new jvm defaults #528

Merged
merged 1 commit into from
Feb 26, 2016
Merged

CODENVY-88 Setup new jvm defaults #528

merged 1 commit into from
Feb 26, 2016

Conversation

skabashnyuk
Copy link
Contributor

Setup jvm defaults and all java options in che.sh.
Used a non-blocking entropy source

Connected
eclipse-che/che-lib#1
#498

### Cannot add this in setenv.sh.
### We do the port mapping here, and this gets inserted into server.xml when tomcat boots
export JAVA_OPTS="${JAVA_OPTS} -Dport.http=${CHE_PORT}"
export JAVA_OPTS="${JAVA_OPTS} -Dport.http=${CHE_PORT} -Dche.home=${CHE_HOME}"
export SERVER_PORT=${CHE_PORT}

# Launch the Che application server, passing in command line parameters

Choose a reason for hiding this comment

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

you changed file permissions of that file. please fix that

Copy link
Contributor

Choose a reason for hiding this comment

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

but 100644 → 100755 it is correct permission for .sh

Choose a reason for hiding this comment

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

yeap, didn't noticed. thank you @riuvshin

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will change

Choose a reason for hiding this comment

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

@skabashnyuk it was OK

<destName>setenv.sh</destName>
<fileMode>755</fileMode>
</file>
<file>

Choose a reason for hiding this comment

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

Am I correct that after that change setenv.sh from che-lib will be used?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes

Choose a reason for hiding this comment

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

ok

@skabashnyuk skabashnyuk force-pushed the CODENVY-88 branch 2 times, most recently from 83da708 to 191b920 Compare February 26, 2016 07:56
skabashnyuk added a commit that referenced this pull request Feb 26, 2016
CODENVY-88 Setup new jvm defaults
@skabashnyuk skabashnyuk merged commit b19edf5 into master Feb 26, 2016
@skabashnyuk skabashnyuk deleted the CODENVY-88 branch February 26, 2016 09:00
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.

4 participants