[PATCH FOR REVIEW]: Support the alpha architecture

Joe Darcy joe.darcy at oracle.com
Fri Jun 4 15:46:34 UTC 2010


Andrew John Hughes wrote:
> On 4 June 2010 01:24, Joe Darcy <joe.darcy at oracle.com> wrote:
>   
>> Andrew John Hughes wrote:
>>     
>>> On 3 June 2010 21:12, Alan Bateman <Alan.Bateman at oracle.com> wrote:
>>>
>>>       
>>>> Andrew John Hughes wrote:
>>>>
>>>>         
>>>>> :
>>>>> Thanks Kelly.  Your change does make more sense, especially given we
>>>>> don't yet use the _alpha_ define anyway.  I've pushed the revised
>>>>> version:
>>>>>
>>>>> http://hg.openjdk.java.net/jdk7/build/corba/rev/2657ee0d2d14
>>>>> http://hg.openjdk.java.net/jdk7/build/jdk/rev/fb56f86642d6
>>>>>
>>>>> Thanks,
>>>>>
>>>>>
>>>>>           
>>>> I realize I'm late to this discussion but I assume the changes to the
>>>> corba
>>>> repo aren't really necessary as it doesn't have any native code any more.
>>>>
>>>>         
>>> Hmmm, I see this is the case with OpenJDK7 but OpenJDK6 still has
>>> src/share/native/com/sun/corba/se/internal/io/ioser.c.  The patch
>>> originated on 6, where presumably the fix was needed.
>>>
>>>       
>> Yes, I would be open to a backport of Alan's
>>
>> 6939646: Remove obsolete com.sun.corba.se.internal.io package
>> 6955873: CORBA resources bundles and javax.activity missing from b94
>>
>> to OpenJDK 6.
>>
>>     
>
> Backported and pushed both of these.
>
> Are you okay with the JDK portion of this alpha patch being backported
> to OpenJDK6?
>
> i.e. just http://hg.openjdk.java.net/jdk7/build/jdk/rev/fb56f86642d6
>
>   

Yes, that looks fine to backport.

Cheers,

-Jeo



More information about the build-dev mailing list