Review please: 6367077 : Purge LD_LIBRARY_PATH from the Unix launchers

View: New views
3 Messages — Rating Filter:   Alert me  

Review please: 6367077 : Purge LD_LIBRARY_PATH from the Unix launchers

by kumar.srinivasan :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hello All,

This is a long standing issue we have had with the java launcher, please
review.
http://cr.openjdk.java.net/~ksrini/6367077/webrev.00/


Thanks
Kumar

Re: Review please: 6367077 : Purge LD_LIBRARY_PATH from the Unix launchers

by martinrb :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Thanks for doing this.  10-minute review looks good.

One minor comment:

You probably want to die if "sun.arch.data.model" is not either "32" or "64".

+    static final boolean is64Bit =
+            System.getProperty("sun.arch.data.model", "32").equals("64");
+    static final boolean is32Bit = !is64Bit;

Martin

On Tue, Nov 10, 2009 at 14:55, Kumar Srinivasan
<Kumar.Srinivasan@...> wrote:

> Hello All,
>
> This is a long standing issue we have had with the java launcher, please
> review.
> http://cr.openjdk.java.net/~ksrini/6367077/webrev.00/
>
>
> Thanks
> Kumar
>

Re: Review please: 6367077 : Purge LD_LIBRARY_PATH from the Unix launchers

by kumar.srinivasan :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Thanks Martin, I will take care of that.

Kumar

> Thanks for doing this.  10-minute review looks good.
>
> One minor comment:
>
> You probably want to die if "sun.arch.data.model" is not either "32" or "64".
>
> +    static final boolean is64Bit =
> +            System.getProperty("sun.arch.data.model", "32").equals("64");
> +    static final boolean is32Bit = !is64Bit;
>
> Martin
>
> On Tue, Nov 10, 2009 at 14:55, Kumar Srinivasan
> <Kumar.Srinivasan@...> wrote:
>  
>> Hello All,
>>
>> This is a long standing issue we have had with the java launcher, please
>> review.
>> http://cr.openjdk.java.net/~ksrini/6367077/webrev.00/
>>
>>
>> Thanks
>> Kumar
>>
>>