New project: getting rid of IcedTea local patches
Andrew John Hughes
gnu_andrew at member.fsf.org
Thu Apr 9 11:01:18 PDT 2009
2009/4/9 Andrew John Hughes <gnu_andrew at member.fsf.org>:
> 2009/4/9 Alan Bateman <Alan.Bateman at sun.com>:
>> Andrew Haley wrote:
>>> The page is at http://icedtea.classpath.org/wiki/IcedTea_JDK6_Patches
>>> These are the patches in icedtea/patches and icedtea/patches/hotspot.
>>> * Every patch should be assigned to someone.
>>> * Every patch should be discussed.
>>> * If we can, we will push a patch upstream. If we can't the discussion
>>> should explain why we can't.
>>> Here we go...
>> I'm not very familiar with IcedTea but I cloned the repository to see what
>> these patches are about. Here are a few comments:
>> icedtea-6728542-epoll.patch seems to be a change-set I pushed to jdk7 last
>> year. It seems very reasonable to push this into jdk6/jdk6/jdk. Go for it!
> Yes, I've referenced the bug on the wiki now. Joe, can we commit this to 6?
>> icedtea-bytebuffer-compact.patch - this looks to be a partial fix to
>> 6593946. If so, 6593946 is already in jdk6/jdk6/jdk
> IcedTea6 is still based on the last build drop of jdk6, not the
> Mercurial repository. So this 6 week old patch won't be in the 25th
> of November b14 tarball.
> Interestingly, it also still exists in IcedTea7 as there were cases
> covered by the patch which didn't make it into the commit.
> I've attached that version. It was originally added by Keith Seitz
> while he was working on OpenJDK at Red Hat (according to the
>> icedtea-io_util-overflow.patch - this looks like the initial fix for
>> 6788196. Martin has already pushed this to jdk6/jdk7/jdk
> Same case as the epoll patch, not in b14.
>> icedtea-jvmtiEnv.patch - I thought JvmtiEnv::GetLineNumberTable already
>> handled this case so I'm curious why this one is needed.
> Added by Andrew Haley as part of work on Shark; perhaps he can explain.
>> icedtea-network-unreachable.patch - I'm pretty sure net-dev would be open to
>> this because it looks to be cases where the specific exceptions aren't
>> thrown in the timed read case.
> From our intern, Omair so this is covered by the SCA. Omair, maybe
> you could file a bug report for this (or link to one here if there is
>> icedtea-sunsrc.patch - I'm not sure what this is about but it looks like it
>> removes the service configuration files. Does JDI still work?
> Don't know about this one, all I can say is it was added by Joshua
> Sumali in late 2007. Perhaps someone at Red Hat then can shed some
> light on it?
>> icedtea-testenv.patch - the changes to
>> test/sun/net/InetAddress/nameservice/dns/cname.sh seems reasonable (pilot
>> error is why this one was missed when the other tests were updated).
> Recent patch from Mark Wielaard. Mark?
>> icedtea-jtreg-hatrun.patch - I suspect this is 6545295. We should just fix
>> this test (serviceability-dev).
> Again, this one is from Omair, so he should be able to provide more detail.
>> PS: I'm not on distro-pkg-dev so this mail might need the moderator to
> Thanks again for the conf. call, very useful!
> Andrew :-)
> Free Java Software Engineer
> Red Hat, Inc. (http://www.redhat.com)
> Support Free Java!
> Contribute to GNU Classpath and the OpenJDK
> PGP Key: 94EFD9D8 (http://subkeys.pgp.net)
> Fingerprint: F8EF F1EA 401E 2E60 15FA 7927 142C 2591 94EF D9D8
And then forgot the patch!
Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)
Support Free Java!
Contribute to GNU Classpath and the OpenJDK
PGP Key: 94EFD9D8 (http://subkeys.pgp.net)
Fingerprint: F8EF F1EA 401E 2E60 15FA 7927 142C 2591 94EF D9D8
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 1333 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/jdk6-dev/attachments/20090409/5bbf5b8b/attachment.obj
More information about the jdk6-dev