[Engine-devel] Java 7, revisited again

Allon Mureinik amureini at redhat.com
Wed Apr 3 07:06:34 UTC 2013


Good point, thanks!

Fixed up the patch and uploaded a new version.

----- Original Message -----
> From: "Kanagaraj" <kmayilsa at redhat.com>
> To: "Allon Mureinik" <amureini at redhat.com>
> Cc: "engine-devel" <engine-devel at ovirt.org>
> Sent: Wednesday, April 3, 2013 7:23:16 AM
> Subject: Re: [Engine-devel] Java 7, revisited again
> 
> On 04/02/2013 07:03 PM, Allon Mureinik wrote:
> > Hola,
> >
> > A couple of weeks (months?) ago we discussed upgrading to Java 7.[1]
> > Generally, there was a positive vibe about moving forward with technology,
> > although we noted that this should not be done for common, compat and the
> > GWT modules,
> 
> searchbackend is also used by GWT, this needs to be excluded as well.
> 
> Thanks,
> Kanagaraj
> 
> > under the limitation of the current GWT version we're using.
> >
> > Here's an initial suggestion for this upgrade:
> > http://gerrit.ovirt.org/#/c/13519/
> >
> > Areas handled and checked:
> > 1. compilation (including GWT compilation)[2]
> > 2. animal-sniffer validations for to makes sure that modules that aren't
> > supposed to use JDK7 features really don't[2]
> > 3. Running and making sure the UI still works :-)
> > (checkstyle issues which Laslo pointed out in the original discussion were
> > already solved in a different patch merged long ago)
> >
> >
> > Sincerely,
> > Allon and Alissa
> >
> > [1] http://lists.ovirt.org/pipermail/engine-devel/2012-December/003139.html
> > [2] This includes, of course, purposely introducing Java 7 syntax and JDK 7
> > features to modules that aren't supposed to have them,
> >      and making sure that the build fails early, as expected, without
> >      having to add the -Pgwt-admin or -Pgwt-user profiles.
> > _______________________________________________
> > Engine-devel mailing list
> > Engine-devel at ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/engine-devel
> 
> 



More information about the Devel mailing list