Review Request: Build-infra update

[Magnus Ihse Bursie]

I had to ask Erik about it, too. :-) Since we are now a small crowd not knowing about it, I think that warrants a comment describing the purpose. Erik, can you fix that?

/Magnus

4 jun 2012 kl. 22:22 skrev Tim Bell <tim.bell at oracle.com>:

> Looks good to me.   Line 314  of CompileJavaClasses.gmk is an 'order-only' prerequisite - I had to look it up as I have only seen it used one other time.
> 
> http://www.gnu.org/software/make/manual/make.html#Prerequisite-Types
> 
> Tim
> 
> 
> 
> 
> On 06/04/12 07:15, Erik Joelsson wrote:
>> 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.
>>> 
>