Review for 7153977

[Andrew Hughes]

----- Original Message -----
> On 3/20/12 2:16 AM, Andrew Hughes wrote:
> > Is this patch in OpenJDK8?  My understanding was that patches had
> > to go there first.
> 
> It's possible for a change to go here first. It takes a bit more
> work, from my POV,
> since what is often a simple backport from 8 to 7u resulting in the
> identical change
> with often a quick approval by a maintainer then turns into a change
> that needs its own
> separate review thread for 7u (since the identical change is not in 8
> already), which
> may take a couple of days on its own and, of course, the obligatory
> maintainer approval.
> 

Yes, using 8 makes more sense to me too, as that's what the individual groups
are focusing on.

> I'd also add that some changes can be so 7u specific, that it makes
> no sense for them
> to go into 8 first or at all.
> 

Yes, I'm expecting to have some of those myself once 8 changes its build system...

> But in general, yeah - fixes should (and mostly do, afaict) go into 8
> first.
> 
> cheers,
> dalibor topic
> --
> Oracle <http://www.oracle.com>
> Dalibor Topic | Principal Product Manager
> Phone: +494089091214 <tel:+494089091214> | Mobile: +491737185961
> <tel:+491737185961>
> Oracle Java Platform Group
> 
> ORACLE Deutschland B.V. & Co. KG | Nagelsweg 55 | 20097 Hamburg
> 
> ORACLE Deutschland B.V. & Co. KG
> Hauptverwaltung: Riesstr. 25, D-80992 München
> Registergericht: Amtsgericht München, HRA 95603
> Geschäftsführer: Jürgen Kunz
> 
> Komplementärin: ORACLE Deutschland Verwaltung B.V.
> Hertogswetering 163/167, 3543 AS Utrecht, Niederlande
> Handelsregister der Handelskammer Midden-Niederlande, Nr. 30143697
> Geschäftsführer: Alexander van der Ven, Astrid Kepper, Val Maher
> 
> Green Oracle <http://www.oracle.com/commitment> Oracle is committed
> to developing practices and products that help protect the
> environment
> 

-- 
Andrew :)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

PGP Key: 248BDC07 (https://keys.indymedia.org/)
Fingerprint = EC5A 1F5E C0AD 1D15 8F1F  8F91 3B96 A578 248B DC07