Andrew John Hughes wrote:
On 4 June 2010 01:24, Joe Darcy <joe.da...@oracle.com> wrote:
Andrew John Hughes wrote:
On 3 June 2010 21:12, Alan Bateman <alan.bate...@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
