Review Request: 8005635: build-infra: Support building install	in jprt
    David Holmes 
    david.holmes at oracle.com
       
    Thu Jan  3 11:59:57 UTC 2013
    
    
  
spec.gmk.in:
Not sure about this one. Why do the image directories need to be 
configure time variables placed in spec.gmk? For profiles I dynamically 
change JRE_IMAGE_DIR based on the profile being built. I don't think 
this change breaks that because I set it as a make variable on the 
submake invocation. But the fact that you need to do this indicates 
there is something that may conflict with what I am doing.
David
-----
On 3/01/2013 9:36 PM, Erik Joelsson wrote:
> One more variable being converted for jprt.
>
> http://cr.openjdk.java.net/~erikj/8005635/webrev.root.03/
>
> /Erik
>
> On 2013-01-02 19:34, Erik Joelsson wrote:
>> Missed the changes to common/makefiles/Jprt.gmk. New webrev:
>>
>> http://cr.openjdk.java.net/~erikj/8005635/webrev.root.02/
>>
>> /Erik
>>
>> On 2013-01-02 17:14, Erik Joelsson wrote:
>>> This is the open part of this review. The changes are mostly
>>> concerning compare logic and introducing a couple of new variables in
>>> spec for images directories.
>>>
>>> Contributed by Tim Bell and myself.
>>>
>>> http://cr.openjdk.java.net/~erikj/8005635/webrev.root.01/
>>> http://cr.openjdk.java.net/~erikj/8005635/webrev.jdk.01/
>>>
>>> /Erik
    
    
More information about the build-dev
mailing list