Re: [Engine-devel] Java 7, revisited again
After a week of positive reviews in gerrit, and no objections voiced here, patch was
merged.
piped catch clauses, here we come.
----- Original Message -----
From: "Allon Mureinik" <amureini(a)redhat.com>
To: "Kanagaraj" <kmayilsa(a)redhat.com>
Cc: "engine-devel" <engine-devel(a)ovirt.org>
Sent: Wednesday, April 3, 2013 10:06:34 AM
Subject: Re: [Engine-devel] Java 7, revisited again
Good point, thanks!
Fixed up the patch and uploaded a new version.
----- Original Message -----
> From: "Kanagaraj" <kmayilsa(a)redhat.com>
> To: "Allon Mureinik" <amureini(a)redhat.com>
> Cc: "engine-devel" <engine-devel(a)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(a)ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/engine-devel
>
>
_______________________________________________
Engine-devel mailing list
Engine-devel(a)ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-devel