RFR - 7163471: Licensee source bundle failed around 7u4

Nils Loodin nils.loodin at oracle.com
Tue Jun 5 01:53:12 PDT 2012


Definitely. Do you think I should send out a new review after that, or fix it and then push?

Regards,
Nils Loodin

On Jun 5, 2012, at 10:27 , Bengt Rutisson wrote:

> 
> Nils,
> 
> This looks good to me. One minor thing if you have time:
> 
> Could we move the JFR defintion up to between JFR_SRCDIRS_EXIST and UCRYPTO? That way we would get a more natural order in the file: JFR_SRCDIRS_EXIST, JFR, UCRYPTO, SUBDIRS.
> 
> Ship it!
> Bengt
> 
> On 2012-06-04 13:11, Nils Loodin wrote:
>> In the scenario which the bugs talk about, i.e. licensee builds.
>> They don´t have, and don't want, a define in the makefiles.
>> Their only mechanism is to remove certain files based on a  blacklist.
>> 
>> This is currently causing a build failure from the licensee sources.
>> 
>> /Nisse
>> 
>> 
>> On Jun 4, 2012, at 12:01 , Erik Joelsson wrote:
>> 
>>> In what scenario would the directory not exist when OPENJDK is not defined?
>>> 
>>> /Erik
>>> 
>>> On 2012-06-04 11:10, Nils Loodin wrote:
>>>> http://cr.openjdk.java.net/~nloodin/71663471/webrev.00/<http://cr.openjdk.java.net/%7Enloodin/71663471/webrev.00/>
>>>> 
>>>> Simple fix, check if the directory exist or not, and if not don't build that subdirectory.
>>>> 
>>>> Modelled the mechanic after another makefile in the jdk.
>>>> 
>>>> 
> 



More information about the hotspot-runtime-dev mailing list