Review request: 7127874 Add handling of macosx env variables to ProcessBuilder regression test (original) (raw)
Jason Uh jason.uh at oracle.com
Wed Jan 11 17:56:23 PST 2012
- Previous message: Review request: 7127874 Add handling of macosx env variables to ProcessBuilder regression test
- Next message: Review request: 7127874 Add handling of macosx env variables to ProcessBuilder regression test
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Sorry, please see instead: http://cr.openjdk.java.net/~juh/7127874/webrev.01/
I've changed
+ private static final boolean is = osName.startsWith("Mac OS X");to
+ private static final boolean is = osName.startsWith("Mac OS");Thanks, Jason
On 1/11/12 2:34 PM, Jason Uh wrote:
Hi,
Could I please get the following change reviewed? http://cr.openjdk.java.net/~juh/7127874/webrev.00/ <http://cr.openjdk.java.net/%7Ejuh/7127874/webrev.00/> This change to the test of ProcessBuilder adds handling of Mac OS and the environment _variables that it adds -- namely, CFUSERTEXTENCODING and JAVAMAINCLASS. Thanks, Jason
- Previous message: Review request: 7127874 Add handling of macosx env variables to ProcessBuilder regression test
- Next message: Review request: 7127874 Add handling of macosx env variables to ProcessBuilder regression test
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]