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

handle unexpected output while setting JAVA_TOOL_OPTIONS enviroment variable #268

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@adyliu
Contributor

adyliu commented Aug 8, 2014

I set the variable of JAVA_TOOL_OPTIONS on OS X, so the bash failed with the unexpected output:
Picked up JAVA_TOOL_OPTIONS: -Dfile.encoding=UTF-8

@ypujante

This comment has been minimized.

Member

ypujante commented Aug 8, 2014

Can you give an example of a command line that make it fail?

@adyliu

This comment has been minimized.

Contributor

adyliu commented Aug 12, 2014

When I start the tutorial.sh

➜ ~/Downloads/glu $ bin/tutorial.sh start
bin/tutorial.sh: line 178: [: Picked up JAVA_TOOL_OPTIONS: -Dfile.encoding=UTF-8: integer expression expected

I set an enviroment variable in the bash profile:

➜ ~/Downloads/glu $ grep JAVA_TOOL_OPTIONS /etc/bashrc
export JAVA_TOOL_OPTIONS=-Dfile.encoding=UTF-8
@ypujante

This comment has been minimized.

Member

ypujante commented Aug 12, 2014

Thank you. I will try to reproduce.

@ypujante

This comment has been minimized.

Member

ypujante commented Sep 8, 2014

I see what is happening. I guess on my system it does not fail (I do not get the integer expression expected error message), but I can see that the sed command returns the wrong value.

ypujante added a commit that referenced this pull request Sep 8, 2014

@ypujante

This comment has been minimized.

Member

ypujante commented Sep 11, 2014

Applied in 5.5.2. Thanks!

@ypujante ypujante closed this Sep 11, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment