Need final check

[David Holmes]

On 19/09/2012 2:18 AM, Kelly O'Hair wrote:
>
> On Sep 18, 2012, at 6:23 AM, Erik Joelsson wrote:
>
>> While hunting down a build break on mac that always happened on our hudson system, but not when I built manually on the same machines, I discovered that jdk7u4 can't be used as bootjdk.
>>
>> When I fixed differences in the mac build, I removed these checks in jdk/makefiles/GensrcIcons.gmk:
>>
>> # TODO FIX FIX FIX
>> ifneq ($(OPENJDK_TARGET_OS),macosx)
>>         $(CAT) $$<  | $(TOOL_TOBIN)>>  $$@.tmp
>> endif
>>
>> It worked fine for me. It didn't work in hudson though as we were using jdk7u4 there and I was accidentally using 7u7 when building manually. The error looks like this:
>>
>> Exception in thread "main" java.lang.Error: Could not instantiate Graphics Environment: sun.java2d.HeadlessGraphicsEnvironment
>>     at java.awt.GraphicsEnvironment.createGE(GraphicsEnvironment.java:118)
>>     at java.awt.GraphicsEnvironment.getLocalGraphicsEnvironment(GraphicsEnvironment.java:81)
>>     at java.awt.image.BufferedImage.createGraphics(BufferedImage.java:1152)
>>     at java.awt.image.BufferedImage.getGraphics(BufferedImage.java:1142)
>>     at sun.awt.X11.ToBin.main(ToBin.java:41)
>>
>> As far as I can tell, official builds are still happening with plain 7. What is currently used in JPRT? I will update our hudson config to something that works.
>
> No, JPRT is using 7u4, and RE is supposed to have switched to 7u4.

As discussed in other emails 7u4 on MAC was a developer preview build 
and is not stable enough to use as a bootjdk. It should be updated to 7u6.

David
-----


> I think the trick is that the user doing the build needs to be logged in via the GUI to the system during the build.
> JPRT does this, as does RE.
> This is what makes it fairly impossible to share Macs and allow people to just ssh in and build. :^(
>
> Building with 7u7 might seem like a solution, but it means we are different from JPRT and RE.
>
> -kto
>
>>
>> /Erik
>>
>> On 2012-09-18 04:26, Kelly O'Hair wrote:
>>> I pulled out all the changes to the old makefiles and changes that are being pursued under separate review/integration:
>>>
>>> 7197849: Update new build-infra makefiles
>>>
>>> http://cr.openjdk.java.net/~ohair/openjdk8/build-infra4/repo-./webrev/
>>> http://cr.openjdk.java.net/~ohair/openjdk8/build-infra4/repo-corba/webrev/
>>> http://cr.openjdk.java.net/~ohair/openjdk8/build-infra4/repo-jaxp/webrev/
>>> http://cr.openjdk.java.net/~ohair/openjdk8/build-infra4/repo-jaxws/webrev/
>>> http://cr.openjdk.java.net/~ohair/openjdk8/build-infra4/repo-langtools/webrev/
>>> http://cr.openjdk.java.net/~ohair/openjdk8/build-infra4/repo-jdk/webrev/
>>>
>>> If there is anything wrong here, please fix it in build-infra and let me know.
>>>
>>> -kto
>>>
>