Request for review: 6833444 version 2

Dmitri Trembovetski Dmitri.Trembovetski at Sun.COM
Mon Apr 27 15:56:38 UTC 2009



Anthony Petrov wrote:
> Jon, Kelly, Xiomara,
> 
> Thank you very much for your feedback. I decided to revise the fix to:
> 1. Leave the current c:\jdk default path for those who still uses it.
> 2. Issue a warning message on MS Windows platform if the BOOTDIR is 
> located on the J: drive (that is assumed to be mapped over network).
> 
> Here's the updated webrev:
> 
> http://cr.openjdk.java.net/~anthony/webrev-6833444.1/
> 
> Please review.
> 
> Some comments:
> 
> On 04/23/2009 11:53 PM, Jonathan Gibbons wrote:
>> On pretty much any system you can tell if a drive is local or not, 
>> albeit in a system specific way.
> That would be cool. However I don't see much justification around 
> verifying for sure if a path is on the network. After all it's just a 
> warning message, not a error.

   It's very easy to check on windows (in shell, adjust appropriately for makefile):
    net use j: > /dev/null 2>&1
    if [ $? -ne 0 ]; then
      # local drive
    else
      # remote drive
    fi

    If you want to be fancy you could grep the output of net use for "Remote 
Resource" or something..

   Thanks,
     Dmitri



More information about the build-dev mailing list