RFR: 8017047: Can't use --with-java-devtools and --with-devkit	at the same time
    Tim Bell 
    tim.bell at oracle.com
       
    Wed Jun 26 23:12:25 UTC 2013
    
    
  
Hi Erik:
>> In the new basics.m4:
>>
>> 362     TOOLS_DIR=$TOOLS_DIR:$with_devkit/bin
>>
>> If TOOLS_DIR was empty, this will add an extra :
>>
>> Probably harmless, but I'd prefer not to see it happen.  If you feel 
>> it is harmless, go ahead.
>>
>> Code deletion looks good.
>>
> Yes, there would be an extra :, but it didn't seem to matter. We 
> should try to keep things correct though, so I added a convenience 
> macro for appending to a path without adding : if it's empty.
>
> http://cr.openjdk.java.net/~erikj/8017047/webrev.root.02/
>
> I also added calls to BASIC_FIXUP_PATH to make the parameters accept 
> relative paths.
>
> /Erik
Yes, this is better.  Looks good.  Thanks.
Tim
    
    
More information about the build-dev
mailing list