<AWT Dev> Review Request: 8004151: build-infra: Generating X11 wrapper offset file is not cross compilable (AWT folks look here!)

David Holmes david.holmes at oracle.com
Mon Jan 21 17:57:30 PST 2013

Hi Erik,

I need this pushed to jdk8/build yesterday! :) Seriously!

This is now a blocker for the Profiles integration work. I thought I was 
utilizing 8005855 as a workaround but it never got pushed - instead I 
still have my hack of deleting the X_LIBS and X_CFLAGS from the sizer 
build. I can't have that hack in the repo that will go to PIT this week 
for the Profiles work, but I need something to get cross-compilation 


On 19/01/2013 12:05 AM, Erik Joelsson wrote:
> Here is a new webrev with my sorts incorporated. This has been baking
> for quite a while in build-infra now and seems to be working.
> http://cr.openjdk.java.net/~erikj/8004151/webrev.jdk.04/
> /Erik
> On 2012-12-07 11:05, Erik Joelsson wrote:
>> (resending to full recepients list)
>> I just hit another issue. I tried using a jdk8 boot-jdk and the order
>> of the output was different in the generated sizes file compared to
>> the one in the repo. Sorting both resulted in a clean diff.
>> /Erik
>> On 2012-12-05 14:20, Fredrik Öhrström wrote:
>>> 2012-11-29 15:54, Fredrik Öhrström skrev:
>>>> 2012-11-29 15:36, Erik Joelsson skrev:
>>>>> I just submitted a patch to build-infra for the dual generation on
>>>>> all platforms since it breaks comparisons between old and new
>>>>> build. In general, we can't change behavior in new build without
>>>>> also changing the old before the old is removed.
>>>>> Removing sizes.64-solaris-i386 breaks the old build on solaris-x86,
>>>>> so I have readded it to build-infra.
>>>> Thanks Erik! The new and updated webrev is here:
>>>> http://cr.openjdk.java.net/~ohrstrom/webrev-8004151-gensrcX11wrapper-v2/
>>>> <http://cr.openjdk.java.net/%7Eohrstrom/webrev-8004151-gensrcX11wrapper-v2/>
>>> Ok, new webrev incorporating a few more Erik changes. Any comments
>>> from the AWT experts?
>>> http://cr.openjdk.java.net/~ohrstrom/webrev-8004151-gensrcX11wrapper-v3/
>>> //Fredrik

More information about the awt-dev mailing list