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

scala doesn't take -D properties on WIN7 #4615

Closed
scabug opened this issue May 19, 2011 · 7 comments
Closed

scala doesn't take -D properties on WIN7 #4615

scabug opened this issue May 19, 2011 · 7 comments
Assignees
Milestone

Comments

@scabug
Copy link

@scabug scabug commented May 19, 2011

scala -Dprop=value -e "println(System.getProperty("prop"))"

output

null

this problem does not appear on Linux or Mac

This pull request scala/scala#1957 based on the 2nd attachment to this bug did not address all the issues. From the pull request discussion:

  • While this parses the -D and -J options, they also get passed as arguments to the class that is run in the end (and I expect that trying to fix this would make the other problems much worse)
  • Quoted -J options are silently dropped

  • Single-quoted -D options are silently dropped

  • Double-quoted -D options are either silently dropped or result in an error

@scabug
Copy link
Author

@scabug scabug commented May 19, 2011

Imported From: https://issues.scala-lang.org/browse/SI-4615?orig=1
Reporter: centaur
Attachments:

@scabug
Copy link
Author

@scabug scabug commented May 23, 2011

huynhjl said:
My attempt at fixing #4614 and #4615.

@scabug
Copy link
Author

@scabug scabug commented May 23, 2011

huynhjl said:
I just tested my patch with the -e "println(System.getProperty("prop"))" argument and my patch does not work on that... I guess I'll give it another try later.

@scabug
Copy link
Author

@scabug scabug commented May 24, 2011

huynhjl said:
Second take on the fix.

@scabug
Copy link
Author

@scabug scabug commented Jan 23, 2013

@scabug
Copy link
Author

@scabug scabug commented Jan 30, 2013

@JamesIry said (edited on Jan 30, 2013 1:12:18 AM UTC):
Turns out fix wasn't complete. See discussion on the pull. I'm kicking the fix version and assigning to community since I don't have a working Windows environment for development.

@scabug
Copy link
Author

@scabug scabug commented Jul 26, 2013

@gourlaysama said:
A new tentative fix for this: scala/scala#2767.

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

Successfully merging a pull request may close this issue.

None yet
2 participants