Review Request: Build-infra update

[Fredrik Öhrström]

Nice!  Just fix the spelling of "annotaion".

//Fredrik

4 jun 2012 kl. 16:15 skrev Erik Joelsson:

> I have created a (hopefully temporary) hack to run javah manually for these 5 classes. This webrev is just against the build-infra repo. Unless anybody objects to this temporary solution, I will publish a new full webrev against the jdk8/build forest tomorrow.
> 
> http://cr.openjdk.java.net/~erikj/build-infra-m1.1/jdk-base-headers/ <http://cr.openjdk.java.net/%7Eerikj/build-infra-m1.1/jdk-base-headers/>
> 
> /Erik
> 
> On 2012-05-23 19:05, Alan Bateman wrote:
>> On 23/05/2012 17:50, Kelly O'Hair wrote:
>>> Has anyone tried creating an unused native method declaration in the class to trigger the native header
>>> generation?   public native void GenerateNativeHeader();   ???  Is that too ugly?
>>> 
>>> -kto
>> As Jon put it, for the classes in the base module, "we are delicate bootstrapping ground" so if you add something like "private static native void dummy()" rather than the annotation to the classes in the base module then it could be a solution for the short term. This means the following sources in the webrev:
>> 
>> src/share/classes/java/lang/Integer.java
>> src/share/classes/java/lang/Long.java
>> src/share/classes/java/net/SocketOptions.java
>> src/windows/classes/sun/nio/ch/PollArrayWrapper.java
>> 
>> Also it would be good to remove it from src/share/classes/sun/nio/ch/IOStatus.java too (it got the annotation in the first round).
>> 
>> -Alan.
>>