Review request: 7127874 Add handling of macosx env variables to ProcessBuilder regression test

[Scott Kovatch]

On Jan 17, 2012, at 3:12 AM, Michael McMahon wrote:

> On 11/01/12 22:34, 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, __CF_USER_TEXT_ENCODING and JAVA_MAIN_CLASS_<pid>.
>> 
>> Thanks,
>> Jason
> 
> Jason,
> 
> This change is fine, so long as it remains the case that these environment variables will be set
> by the runtime on Mac OS. I understand the JAVA_MAIN_CLASS_<pid> variable is used by AWT.
> 
> Can any AWT experts comment on whether that will continue to be the case?

Yes. This is a backstop for providing a default value for the name of the application to be used in the application menu.

It could be removed if there another (better?) way to find out the name of the class whose main(String args[]) has been executed to start off the JVM.

-- Scott