Re: [ovirt-devel] Moving forward our frontend stack

2014-09-01 Thread Sven Kieske
Well I really think this is a very bad idea.
I'm pretty sure I don't need to introduce
the concept of checksums to anybody on this list
and why this is done.

You must decide whether you want the correct code
or not.
If you decide to disable it on one repo, what is the
justification to not disable it on another/all repos/projects?

just my 2 cents :)

Am 29.08.2014 16:54, schrieb Vojtech Szocs:
 Well, I meant to avoid disabling checksum checking globally.
 If we can selectively disable checksum checking for specific
 Maven artifacts like gwtp etc. then we could do that..

-- 
Mit freundlichen Grüßen / Regards

Sven Kieske

Systemadministrator
Mittwald CM Service GmbH  Co. KG
Königsberger Straße 6
32339 Espelkamp
T: +49-5772-293-100
F: +49-5772-293-333
https://www.mittwald.de
Geschäftsführer: Robert Meyer
St.Nr.: 331/5721/1033, USt-IdNr.: DE814773217, HRA 6640, AG Bad Oeynhausen
Komplementärin: Robert Meyer Verwaltungs GmbH, HRB 13260, AG Bad Oeynhausen
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] Moving forward our frontend stack

2014-08-29 Thread Martin Betak
- Original Message -
 From: Vojtech Szocs vsz...@redhat.com
 To: Martin Betak mbe...@redhat.com
 Cc: devel@ovirt.org, Einav Cohen eco...@redhat.com, Alexander Wels 
 aw...@redhat.com, Greg Sheremeta
 gsher...@redhat.com, Tomas Jelinek tjeli...@redhat.com, Lior Vernia 
 lver...@redhat.com, Daniel Erez
 de...@redhat.com
 Sent: Thursday, August 28, 2014 5:38:04 PM
 Subject: Re: Moving forward our frontend stack
 
 Hey Martin!
 
 I've just reviewed your patch, looks good overall.
 
 Here are my thoughts:
 
 * Jenkins CI fails on patch due to missing dependencies,
   we need to provide following new dependencies in order
   to proceed with the upgrade:
  
   org.aspectj:aspectjweaver:1.8.2
   org.aspectj:aspectjrt:1.8.2
   com.google.gwt:gwt-user:2.6.1
   com.google.gwt:gwt-dev:2.6.1
   com.google.gwt:gwt-servlet:2.6.1
   com.google.gwt:2.6.1
   org.codehaus.mojo:gwt-maven-plugin:2.6.1
   com.gwtplatform:gwtp-processors:1.3.1
   com.gwtplatform:gwtp-mvp-client:1.3.1
   com.google.gwt.inject:gin:2.1.2
 
   Who will take charge of that?

I already spoke to David Caro about the jenkins failure of 
patch http://gerrit.ovirt.org/#/c/32135/. It seems the files
for appropriate versions are there but the files are corrupted.

Let us hope this can be resolved easier than Node.js :-)

Best regards,

Martin

 
 * patch itself looks quite harmless (not too risky)
 
 * consolidating Java source  target version across
   frontend and backend is nice!
 
   - this means we could use Java 7 features
  also on the frontend (Java/GWT) side
 
 * TODO-GWT tags [1] proved to be helpful [2]
 
   - reminder to all UI maintainers to use TODO-GWT
  tags whenever we have some GWT(P) workaround,
  so that the future upgrade will be safer w.r.t.
  existing code
 
 [1] https://www.mail-archive.com/devel@ovirt.org/msg00761.html
 [2]
 http://gerrit.ovirt.org/#/c/32135/1/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/SearchPanelView.java
 
 Regards,
 Vojtech
 
 PS: I went through GWT-Platform release notes,
 found an interesting new feature in recent GWTP
 release, worth investigating:
 
   https://github.com/ArcBees/GWTP/wiki/Release-Notes
   #346 : Map more than one name token to presenter
 
 
 - Original Message -
  From: Martin Betak mbe...@redhat.com
  To: devel@ovirt.org
  Cc: Vojtech Szocs vsz...@redhat.com, Einav Cohen eco...@redhat.com,
  Alexander Wels aw...@redhat.com,
  Greg Sheremeta gsher...@redhat.com, Tomas Jelinek
  tjeli...@redhat.com, Lior Vernia lver...@redhat.com,
  Daniel Erez de...@redhat.com
  Sent: Thursday, August 28, 2014 4:11:10 PM
  Subject: Moving forward our frontend stack
  
  Hello oVirt developers!
  
  I have prepared patch [1] that upgrades our frontend stack
  to use GWT version 2.6.1 (from previous 2.5.1).
  
  This patch also updates GIN to version 2.1.2 and GWT-P to 1.3.1
  
  Since GWT 2.6 features support for Java 7 it was possible to increment
  language levels of all projects stuck at Java 6 (common, compat,
  searchbackend
  and entire of frontend).
  
  To facilitate emitting bytecode compatible with Java 7 also upgrade of
  AspectJ was
  necessary. This patch upgrades it to AspectJ 1.8 that features even support
  for
  Java 8 which will save effort when upgrading to GWT 2.7/3.0 in the future.
  
  Most of the changes in the patch are due to upgrade of GWT-P - i.e.
  changing
  packages of TokenFormatter and PlaceRequest.
  
  Overall this patch is *MUCH* simpler than the previous
  http://gerrit.ovirt.org/#/c/16739/
  which facilitated upgrade from 2.3 to 2.5.1, and hopefully much less risky.
  
  I have tested draft-compile, debug-mode and also tried to use the resulting
  application
  manually for some time. So far everything worked (surprisingly well!) and I
  have not
  detected any defects. Of course I invite anyone to test this patch on his
  own
  since
  it is and upgrade of our core infrastructure.
  
  That having said I think is comparatively simple and the benefits outweigh
  the risks
  if this upgrade is done at the beginning of ovirt-3.6 development cycle.
  
  Reviews, comments and testing are very welcome :-)
  
  Best regards,
  
  Martin
  
  [1] http://gerrit.ovirt.org/#/c/32135/
  
 
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel


Re: [ovirt-devel] Moving forward our frontend stack

2014-08-29 Thread David Caro
On 08/29, Martin Betak wrote:
 - Original Message -
  From: Vojtech Szocs vsz...@redhat.com
  To: Martin Betak mbe...@redhat.com
  Cc: devel@ovirt.org, Einav Cohen eco...@redhat.com, Alexander Wels 
  aw...@redhat.com, Greg Sheremeta
  gsher...@redhat.com, Tomas Jelinek tjeli...@redhat.com, Lior Vernia 
  lver...@redhat.com, Daniel Erez
  de...@redhat.com
  Sent: Thursday, August 28, 2014 5:38:04 PM
  Subject: Re: Moving forward our frontend stack
  
  Hey Martin!
  
  I've just reviewed your patch, looks good overall.
  
  Here are my thoughts:
  
  * Jenkins CI fails on patch due to missing dependencies,
we need to provide following new dependencies in order
to proceed with the upgrade:
   
org.aspectj:aspectjweaver:1.8.2
org.aspectj:aspectjrt:1.8.2
com.google.gwt:gwt-user:2.6.1
com.google.gwt:gwt-dev:2.6.1
com.google.gwt:gwt-servlet:2.6.1
com.google.gwt:2.6.1
org.codehaus.mojo:gwt-maven-plugin:2.6.1
com.gwtplatform:gwtp-processors:1.3.1
com.gwtplatform:gwtp-mvp-client:1.3.1
com.google.gwt.inject:gin:2.1.2
  
Who will take charge of that?
 
 I already spoke to David Caro about the jenkins failure of 
 patch http://gerrit.ovirt.org/#/c/32135/. It seems the files
 for appropriate versions are there but the files are corrupted.
 
 Let us hope this can be resolved easier than Node.js :-)

Ok job passed:

  
http://jenkins.ovirt.org/job/ovirt_engine_master_compile_checkstyle_gerrit/32071


 
 Best regards,
 
 Martin
 
  
  * patch itself looks quite harmless (not too risky)
  
  * consolidating Java source  target version across
frontend and backend is nice!
  
- this means we could use Java 7 features
   also on the frontend (Java/GWT) side
  
  * TODO-GWT tags [1] proved to be helpful [2]
  
- reminder to all UI maintainers to use TODO-GWT
   tags whenever we have some GWT(P) workaround,
   so that the future upgrade will be safer w.r.t.
   existing code
  
  [1] https://www.mail-archive.com/devel@ovirt.org/msg00761.html
  [2]
  http://gerrit.ovirt.org/#/c/32135/1/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/SearchPanelView.java
  
  Regards,
  Vojtech
  
  PS: I went through GWT-Platform release notes,
  found an interesting new feature in recent GWTP
  release, worth investigating:
  
https://github.com/ArcBees/GWTP/wiki/Release-Notes
#346 : Map more than one name token to presenter
  
  
  - Original Message -
   From: Martin Betak mbe...@redhat.com
   To: devel@ovirt.org
   Cc: Vojtech Szocs vsz...@redhat.com, Einav Cohen 
   eco...@redhat.com,
   Alexander Wels aw...@redhat.com,
   Greg Sheremeta gsher...@redhat.com, Tomas Jelinek
   tjeli...@redhat.com, Lior Vernia lver...@redhat.com,
   Daniel Erez de...@redhat.com
   Sent: Thursday, August 28, 2014 4:11:10 PM
   Subject: Moving forward our frontend stack
   
   Hello oVirt developers!
   
   I have prepared patch [1] that upgrades our frontend stack
   to use GWT version 2.6.1 (from previous 2.5.1).
   
   This patch also updates GIN to version 2.1.2 and GWT-P to 1.3.1
   
   Since GWT 2.6 features support for Java 7 it was possible to increment
   language levels of all projects stuck at Java 6 (common, compat,
   searchbackend
   and entire of frontend).
   
   To facilitate emitting bytecode compatible with Java 7 also upgrade of
   AspectJ was
   necessary. This patch upgrades it to AspectJ 1.8 that features even 
   support
   for
   Java 8 which will save effort when upgrading to GWT 2.7/3.0 in the future.
   
   Most of the changes in the patch are due to upgrade of GWT-P - i.e.
   changing
   packages of TokenFormatter and PlaceRequest.
   
   Overall this patch is *MUCH* simpler than the previous
   http://gerrit.ovirt.org/#/c/16739/
   which facilitated upgrade from 2.3 to 2.5.1, and hopefully much less 
   risky.
   
   I have tested draft-compile, debug-mode and also tried to use the 
   resulting
   application
   manually for some time. So far everything worked (surprisingly well!) and 
   I
   have not
   detected any defects. Of course I invite anyone to test this patch on his
   own
   since
   it is and upgrade of our core infrastructure.
   
   That having said I think is comparatively simple and the benefits outweigh
   the risks
   if this upgrade is done at the beginning of ovirt-3.6 development cycle.
   
   Reviews, comments and testing are very welcome :-)
   
   Best regards,
   
   Martin
   
   [1] http://gerrit.ovirt.org/#/c/32135/
   
  

-- 
David Caro

Red Hat S.L.
Continuous Integration Engineer - EMEA ENG Virtualization RD

Tel.: +420 532 294 605
Email: dc...@redhat.com
Web: www.redhat.com
RHT Global #: 82-62605


pgpH_TAfNE2mw.pgp
Description: PGP signature
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Re: [ovirt-devel] Moving forward our frontend stack

2014-08-29 Thread David Caro
On 08/29, Vojtech Szocs wrote:
 
 
 - Original Message -
  From: David Caro dcaro...@redhat.com
  To: Martin Betak mbe...@redhat.com
  Cc: Vojtech Szocs vsz...@redhat.com, devel@ovirt.org, Einav Cohen 
  eco...@redhat.com, Alexander Wels
  aw...@redhat.com, Greg Sheremeta gsher...@redhat.com, Tomas Jelinek 
  tjeli...@redhat.com, Lior Vernia
  lver...@redhat.com, Daniel Erez de...@redhat.com
  Sent: Friday, August 29, 2014 11:24:21 AM
  Subject: Re: Moving forward our frontend stack
  
  On 08/29, Martin Betak wrote:
   - Original Message -
From: Vojtech Szocs vsz...@redhat.com
To: Martin Betak mbe...@redhat.com
Cc: devel@ovirt.org, Einav Cohen eco...@redhat.com, Alexander Wels
aw...@redhat.com, Greg Sheremeta
gsher...@redhat.com, Tomas Jelinek tjeli...@redhat.com, Lior
Vernia lver...@redhat.com, Daniel Erez
de...@redhat.com
Sent: Thursday, August 28, 2014 5:38:04 PM
Subject: Re: Moving forward our frontend stack

Hey Martin!

I've just reviewed your patch, looks good overall.

Here are my thoughts:

* Jenkins CI fails on patch due to missing dependencies,
  we need to provide following new dependencies in order
  to proceed with the upgrade:
 
  org.aspectj:aspectjweaver:1.8.2
  org.aspectj:aspectjrt:1.8.2
  com.google.gwt:gwt-user:2.6.1
  com.google.gwt:gwt-dev:2.6.1
  com.google.gwt:gwt-servlet:2.6.1
  com.google.gwt:2.6.1
  org.codehaus.mojo:gwt-maven-plugin:2.6.1
  com.gwtplatform:gwtp-processors:1.3.1
  com.gwtplatform:gwtp-mvp-client:1.3.1
  com.google.gwt.inject:gin:2.1.2

  Who will take charge of that?
   
   I already spoke to David Caro about the jenkins failure of
   patch http://gerrit.ovirt.org/#/c/32135/. It seems the files
   for appropriate versions are there but the files are corrupted.
   
   Let us hope this can be resolved easier than Node.js :-)
  
  It seems that the pom on jboss repository has a checksum that does not
  match the ones there.
  

  http://repository.jboss.org/nexus/content/groups/public-jboss/com/gwtplatform/gwtp/1.3.1/
  
  If you get the gwtp-1.3.1.pom file, you get the hashed:
  
MD5: 9acbb4e5088825d31e2c8307ae679243
SHA1: eb6e92c012926fb265cf4f359c3d5adca8c5c0f9
  
  While the ones there are:
  
MD5: c33a1cd1ffa88aa4cec42e03364b0e0c
SHA1: ca6c0155356765fc23f0a557373193f013adfd6c
  
  Any of you have any contact with the guys that maintain that repo (or
  know about who can know about?)
  
  I seem to be unable to find any contant in the repo itself...
  
  Anyhow, I've disabled the checksum checking and just pass through, so
  it should not complain about unable to download it (it might fail due
  to the checksum, it should at least xd).
 
 Sorry, I've got no idea who to contact about that repo :(
 
 I'd rather avoid disabling checksum checking for our builds,
 bad checksums should be fixed in that repo.

I've opened a ticket to servicedesk, maybe they know.

In the meantime, do you prefer failing all the jobs that use that
artfact?

 
  
  
  
   
   Best regards,
   
   Martin
   

* patch itself looks quite harmless (not too risky)

* consolidating Java source  target version across
  frontend and backend is nice!

  - this means we could use Java 7 features
 also on the frontend (Java/GWT) side

* TODO-GWT tags [1] proved to be helpful [2]

  - reminder to all UI maintainers to use TODO-GWT
 tags whenever we have some GWT(P) workaround,
 so that the future upgrade will be safer w.r.t.
 existing code

[1] https://www.mail-archive.com/devel@ovirt.org/msg00761.html
[2]
http://gerrit.ovirt.org/#/c/32135/1/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/SearchPanelView.java

Regards,
Vojtech

PS: I went through GWT-Platform release notes,
found an interesting new feature in recent GWTP
release, worth investigating:

  https://github.com/ArcBees/GWTP/wiki/Release-Notes
  #346 : Map more than one name token to presenter


- Original Message -
 From: Martin Betak mbe...@redhat.com
 To: devel@ovirt.org
 Cc: Vojtech Szocs vsz...@redhat.com, Einav Cohen
 eco...@redhat.com,
 Alexander Wels aw...@redhat.com,
 Greg Sheremeta gsher...@redhat.com, Tomas Jelinek
 tjeli...@redhat.com, Lior Vernia lver...@redhat.com,
 Daniel Erez de...@redhat.com
 Sent: Thursday, August 28, 2014 4:11:10 PM
 Subject: Moving forward our frontend stack
 
 Hello oVirt developers!
 
 I have prepared patch [1] that upgrades our frontend stack
 to use GWT version 2.6.1 (from previous 2.5.1).
 
 This patch also updates GIN to version 2.1.2 and GWT-P to 1.3.1
 
 Since GWT 2.6 features support for Java 7 it was possible to increment
 language levels of all projects 

Re: [ovirt-devel] Moving forward our frontend stack

2014-08-29 Thread Vojtech Szocs


- Original Message -
 From: David Caro dcaro...@redhat.com
 To: Vojtech Szocs vsz...@redhat.com
 Cc: Martin Betak mbe...@redhat.com, devel@ovirt.org, Einav Cohen 
 eco...@redhat.com, Alexander Wels
 aw...@redhat.com, Greg Sheremeta gsher...@redhat.com, Tomas Jelinek 
 tjeli...@redhat.com, Lior Vernia
 lver...@redhat.com, Daniel Erez de...@redhat.com
 Sent: Friday, August 29, 2014 4:32:03 PM
 Subject: Re: Moving forward our frontend stack
 
 On 08/29, Vojtech Szocs wrote:
  
  
  - Original Message -
   From: David Caro dcaro...@redhat.com
   To: Martin Betak mbe...@redhat.com
   Cc: Vojtech Szocs vsz...@redhat.com, devel@ovirt.org, Einav Cohen
   eco...@redhat.com, Alexander Wels
   aw...@redhat.com, Greg Sheremeta gsher...@redhat.com, Tomas
   Jelinek tjeli...@redhat.com, Lior Vernia
   lver...@redhat.com, Daniel Erez de...@redhat.com
   Sent: Friday, August 29, 2014 11:24:21 AM
   Subject: Re: Moving forward our frontend stack
   
   On 08/29, Martin Betak wrote:
- Original Message -
 From: Vojtech Szocs vsz...@redhat.com
 To: Martin Betak mbe...@redhat.com
 Cc: devel@ovirt.org, Einav Cohen eco...@redhat.com, Alexander
 Wels
 aw...@redhat.com, Greg Sheremeta
 gsher...@redhat.com, Tomas Jelinek tjeli...@redhat.com, Lior
 Vernia lver...@redhat.com, Daniel Erez
 de...@redhat.com
 Sent: Thursday, August 28, 2014 5:38:04 PM
 Subject: Re: Moving forward our frontend stack
 
 Hey Martin!
 
 I've just reviewed your patch, looks good overall.
 
 Here are my thoughts:
 
 * Jenkins CI fails on patch due to missing dependencies,
   we need to provide following new dependencies in order
   to proceed with the upgrade:
  
   org.aspectj:aspectjweaver:1.8.2
   org.aspectj:aspectjrt:1.8.2
   com.google.gwt:gwt-user:2.6.1
   com.google.gwt:gwt-dev:2.6.1
   com.google.gwt:gwt-servlet:2.6.1
   com.google.gwt:2.6.1
   org.codehaus.mojo:gwt-maven-plugin:2.6.1
   com.gwtplatform:gwtp-processors:1.3.1
   com.gwtplatform:gwtp-mvp-client:1.3.1
   com.google.gwt.inject:gin:2.1.2
 
   Who will take charge of that?

I already spoke to David Caro about the jenkins failure of
patch http://gerrit.ovirt.org/#/c/32135/. It seems the files
for appropriate versions are there but the files are corrupted.

Let us hope this can be resolved easier than Node.js :-)
   
   It seems that the pom on jboss repository has a checksum that does not
   match the ones there.
   
 
   http://repository.jboss.org/nexus/content/groups/public-jboss/com/gwtplatform/gwtp/1.3.1/
   
   If you get the gwtp-1.3.1.pom file, you get the hashed:
   
 MD5: 9acbb4e5088825d31e2c8307ae679243
 SHA1: eb6e92c012926fb265cf4f359c3d5adca8c5c0f9
   
   While the ones there are:
   
 MD5: c33a1cd1ffa88aa4cec42e03364b0e0c
 SHA1: ca6c0155356765fc23f0a557373193f013adfd6c
   
   Any of you have any contact with the guys that maintain that repo (or
   know about who can know about?)
   
   I seem to be unable to find any contant in the repo itself...
   
   Anyhow, I've disabled the checksum checking and just pass through, so
   it should not complain about unable to download it (it might fail due
   to the checksum, it should at least xd).
  
  Sorry, I've got no idea who to contact about that repo :(
  
  I'd rather avoid disabling checksum checking for our builds,
  bad checksums should be fixed in that repo.
 
 I've opened a ticket to servicedesk, maybe they know.

Thanks!

 
 In the meantime, do you prefer failing all the jobs that use that
 artfact?

Well, I meant to avoid disabling checksum checking globally.
If we can selectively disable checksum checking for specific
Maven artifacts like gwtp etc. then we could do that..

Anyway, I feel like Martin's patch needs some more testing
before it's merged, so this is not super-urgent I guess :)

 
  
   
   
   

Best regards,

Martin

 
 * patch itself looks quite harmless (not too risky)
 
 * consolidating Java source  target version across
   frontend and backend is nice!
 
   - this means we could use Java 7 features
  also on the frontend (Java/GWT) side
 
 * TODO-GWT tags [1] proved to be helpful [2]
 
   - reminder to all UI maintainers to use TODO-GWT
  tags whenever we have some GWT(P) workaround,
  so that the future upgrade will be safer w.r.t.
  existing code
 
 [1] https://www.mail-archive.com/devel@ovirt.org/msg00761.html
 [2]
 http://gerrit.ovirt.org/#/c/32135/1/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/SearchPanelView.java
 
 Regards,
 Vojtech
 
 PS: I went through GWT-Platform release notes,
 found an interesting new feature in recent GWTP
 release, worth investigating:
 
   

[ovirt-devel] Moving forward our frontend stack

2014-08-28 Thread Martin Betak
Hello oVirt developers!

I have prepared patch [1] that upgrades our frontend stack
to use GWT version 2.6.1 (from previous 2.5.1).

This patch also updates GIN to version 2.1.2 and GWT-P to 1.3.1

Since GWT 2.6 features support for Java 7 it was possible to increment
language levels of all projects stuck at Java 6 (common, compat, searchbackend 
and entire of frontend).

To facilitate emitting bytecode compatible with Java 7 also upgrade of AspectJ 
was 
necessary. This patch upgrades it to AspectJ 1.8 that features even support for 
Java 8 which will save effort when upgrading to GWT 2.7/3.0 in the future.

Most of the changes in the patch are due to upgrade of GWT-P - i.e. changing 
packages of TokenFormatter and PlaceRequest.

Overall this patch is *MUCH* simpler than the previous 
http://gerrit.ovirt.org/#/c/16739/
which facilitated upgrade from 2.3 to 2.5.1, and hopefully much less risky.

I have tested draft-compile, debug-mode and also tried to use the resulting 
application
manually for some time. So far everything worked (surprisingly well!) and I 
have not
detected any defects. Of course I invite anyone to test this patch on his own 
since 
it is and upgrade of our core infrastructure.

That having said I think is comparatively simple and the benefits outweigh the 
risks
if this upgrade is done at the beginning of ovirt-3.6 development cycle.

Reviews, comments and testing are very welcome :-)

Best regards,

Martin

[1] http://gerrit.ovirt.org/#/c/32135/
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel


Re: [ovirt-devel] Moving forward our frontend stack

2014-08-28 Thread Vojtech Szocs
Hey Martin!

I've just reviewed your patch, looks good overall.

Here are my thoughts:

* Jenkins CI fails on patch due to missing dependencies,
  we need to provide following new dependencies in order
  to proceed with the upgrade:
 
  org.aspectj:aspectjweaver:1.8.2
  org.aspectj:aspectjrt:1.8.2
  com.google.gwt:gwt-user:2.6.1
  com.google.gwt:gwt-dev:2.6.1
  com.google.gwt:gwt-servlet:2.6.1
  com.google.gwt:2.6.1
  org.codehaus.mojo:gwt-maven-plugin:2.6.1
  com.gwtplatform:gwtp-processors:1.3.1
  com.gwtplatform:gwtp-mvp-client:1.3.1
  com.google.gwt.inject:gin:2.1.2

  Who will take charge of that?

* patch itself looks quite harmless (not too risky)

* consolidating Java source  target version across
  frontend and backend is nice!

  - this means we could use Java 7 features
 also on the frontend (Java/GWT) side

* TODO-GWT tags [1] proved to be helpful [2]

  - reminder to all UI maintainers to use TODO-GWT
 tags whenever we have some GWT(P) workaround,
 so that the future upgrade will be safer w.r.t.
 existing code

[1] https://www.mail-archive.com/devel@ovirt.org/msg00761.html
[2] 
http://gerrit.ovirt.org/#/c/32135/1/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/SearchPanelView.java

Regards,
Vojtech

PS: I went through GWT-Platform release notes,
found an interesting new feature in recent GWTP
release, worth investigating:

  https://github.com/ArcBees/GWTP/wiki/Release-Notes
  #346 : Map more than one name token to presenter


- Original Message -
 From: Martin Betak mbe...@redhat.com
 To: devel@ovirt.org
 Cc: Vojtech Szocs vsz...@redhat.com, Einav Cohen eco...@redhat.com, 
 Alexander Wels aw...@redhat.com,
 Greg Sheremeta gsher...@redhat.com, Tomas Jelinek 
 tjeli...@redhat.com, Lior Vernia lver...@redhat.com,
 Daniel Erez de...@redhat.com
 Sent: Thursday, August 28, 2014 4:11:10 PM
 Subject: Moving forward our frontend stack
 
 Hello oVirt developers!
 
 I have prepared patch [1] that upgrades our frontend stack
 to use GWT version 2.6.1 (from previous 2.5.1).
 
 This patch also updates GIN to version 2.1.2 and GWT-P to 1.3.1
 
 Since GWT 2.6 features support for Java 7 it was possible to increment
 language levels of all projects stuck at Java 6 (common, compat,
 searchbackend
 and entire of frontend).
 
 To facilitate emitting bytecode compatible with Java 7 also upgrade of
 AspectJ was
 necessary. This patch upgrades it to AspectJ 1.8 that features even support
 for
 Java 8 which will save effort when upgrading to GWT 2.7/3.0 in the future.
 
 Most of the changes in the patch are due to upgrade of GWT-P - i.e. changing
 packages of TokenFormatter and PlaceRequest.
 
 Overall this patch is *MUCH* simpler than the previous
 http://gerrit.ovirt.org/#/c/16739/
 which facilitated upgrade from 2.3 to 2.5.1, and hopefully much less risky.
 
 I have tested draft-compile, debug-mode and also tried to use the resulting
 application
 manually for some time. So far everything worked (surprisingly well!) and I
 have not
 detected any defects. Of course I invite anyone to test this patch on his own
 since
 it is and upgrade of our core infrastructure.
 
 That having said I think is comparatively simple and the benefits outweigh
 the risks
 if this upgrade is done at the beginning of ovirt-3.6 development cycle.
 
 Reviews, comments and testing are very welcome :-)
 
 Best regards,
 
 Martin
 
 [1] http://gerrit.ovirt.org/#/c/32135/
 
___
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel