Need final check

[Kelly O'Hair]

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.

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
>>