Review Request: 8005850: build-infra: Make --enable-openjdk-only really disable custom

[Tim Bell]

On 01/10/13 03:09, Erik Joelsson wrote:
> Fixed the spelling error
>
> http://cr.openjdk.java.net/~erikj/8005850/webrev.root.02/


Looks good to me.

Tim

>
> /Erik
>
> On 2013-01-09 03:44, David Holmes wrote:
>> On 8/01/2013 10:56 PM, Erik Joelsson wrote:
>>> The open part of this review. Move handling of --enable-openjdk-only to
>>> before the first custom hook call.
>>>
>>> http://cr.openjdk.java.net/~erikj/8005850/webrev.root.01/
>>
>> Minor nit: SUPPRESS has two P's :)
>>
>> One concern I have with this is that we keep conflating custom and 
>> closed. The OpenJDK build system provides a facility whereby the user 
>> can augment it with custom build files - both at the configure level 
>> and makefile level. The customizations that Oracle builds include are 
>> what we refer to as "closed" and generally involve additional 
>> "closed" repositories in the forest.
>>
>> In that regard to me --enable-openjdk-only was actually a closed 
>> addition only needed for Oracle closed builds, as a convenience to 
>> say "yes I have a full forest of open and closed parts but ignore the 
>> closed parts and do me an openjdk build".
>>
>> Magnus argued that any build with customizations is not an OpenJDK 
>> build. But if you follow that line then --enable-openjdk-only should 
>> disable all use of customizations.
>>
>> What we have now seems to switch back and forth between custom and 
>> closed.
>>
>> Something to clean up at a later time.
>>
>> Thanks,
>> David