RFC: Minor build correctness patch
martinrb at google.com
Tue Jul 7 11:48:27 PDT 2009
On Tue, Jul 7, 2009 at 11:21, Andrew John Hughes
<gnu_andrew at member.fsf.org>wrote:
> 2009/7/7 Lillian Angel <langel at redhat.com>:
> > jon.vanalten at redhat.com wrote:
> >> Hi,
> >> The attached patch corrects a minor issue noted in
> >> http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=289
> >> Include statements for the .c files affected are missing, resulting in
> >> compiler warnings. While this wasn't affecting the build or correct
> >> performance, it is good practice to include headers where appropriate.
> >> patch, adapted from that provided by the original bug reporter above,
> >> the include statements.
> >> Comments welcome.
> > As per Andrew Hughes comments in the bug, I would like to see this filed
> > upstream to Sun. I have CC'ed Matthew Flaschen.
> > For now, it is fine to commit.
> > Cheers,
> > Lillian
> I'd much prefer we didn't commit this to IcedTea, as it is just
> another patch we have to manage and maintain. The arguments Jon makes
> for including this (it doesn't affect performance or the build) can
> equally be used as very good reasons for not including it in IcedTea.
> The majority of the developers on this list, who have been hacking on
> IcedTea for a while, will be aware of how much of a pain it is to have
> to test the build with different patches applied and to have to
> recreate them when a new build drop appears upstream. The current aim
> (at least of those at Red Hat) is to try and reduce this burden by
> getting the majority of patches upstream.
> Jon, as you're still new to the project, you're probably unaware of a
> lot of this pain so you'll have to take what we say on trust for now
> :) Believe me, I think your efforts would be better spent applying
> this to the appropriate JDK7 tree and creating a webrev for it. I
> can't see an appropriate tree, but presumably those on the net-dev
> mailing list can provide appropriate hints.
The tl forest is the right tree.
You can use me ("martin") as a "reviewer" when pushing this upstream.
> Lillian: there already is a bug,
> http://bugs.sun.com/view_bug.do?bug_id=6562614 (referenced in our
> IcedTea bug) so it should be a simple matter of just getting the patch
> approved (the bug has been) and someone pushing it.
> BTW, I'm aware I said 'I'd commit this' on the bug, but take the fact
> that I didn't as a reconsideration on my part... ;)
> 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
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the distro-pkg-dev