[Engine-patches] Change in ovirt-engine[master]: core: Remove ovirtengine.war

2012-06-15 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: core: Remove ovirtengine.war .. Patch Set 5: Rebased. -- To view, visit http://gerrit.ovirt.org/4968 To unsubscribe, visit http://gerrit.ovirt.org/settings Ge

[Engine-patches] Change in ovirt-engine[master]: core: Remove ovirtengine.war

2012-06-15 Thread vszocs
Vojtech Szocs has posted comments on this change. Change subject: core: Remove ovirtengine.war .. Patch Set 5: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/4968 To unsubscribe, visit http://ger

[Engine-patches] Change in ovirt-engine[master]: core: Remove ovirtengine.war

2012-06-15 Thread vszocs
Vojtech Szocs has submitted this change and it was merged. Change subject: core: Remove ovirtengine.war .. core: Remove ovirtengine.war Despite the apparently important name this is not used at all, so it can be safely removed.

[Engine-patches] Change in ovirt-engine[master]: core: Remove ovirtengine.war

2012-06-15 Thread vszocs
Vojtech Szocs has posted comments on this change. Change subject: core: Remove ovirtengine.war .. Patch Set 4: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/4968 To unsubscribe, visit

[Engine-patches] Change in ovirt-engine[master]: core: Remove ovirtengine.war

2012-06-15 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: core: Remove ovirtengine.war .. Patch Set 4: I verified this again building the packages, installing, running engine-setup and adding a host. -- To view, visi

[Engine-patches] Change in ovirt-engine[master]: core: Remove ovirtengine.war

2012-06-15 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: core: Remove ovirtengine.war .. Patch Set 4: Verified; Looks good to me, but someone else must approve Looks *very* good to me. -- To view, visit http://gerrit

[Engine-patches] Change in ovirt-engine[master]: core: Remove ovirtengine.war

2012-06-15 Thread ecohen
Einav Cohen has posted comments on this change. Change subject: core: Remove ovirtengine.war .. Patch Set 4: Verified; Looks good to me, but someone else must approve rebased, verified the bahavior on devel env. -- To view, vi

[Engine-patches] Change in ovirt-engine[master]: core: Remove ovirtengine.war

2012-06-04 Thread vszocs
Vojtech Szocs has posted comments on this change. Change subject: core: Remove ovirtengine.war .. Patch Set 3: Looks good to me, but someone else must approve Juan, you can remove the legacy UserPortal reference in ear/pom.xml.

[Engine-patches] Change in ovirt-engine[master]: core: Remove ovirtengine.war

2012-06-04 Thread juan . hernandez
Juan Hernandez has posted comments on this change. Change subject: core: Remove ovirtengine.war .. Patch Set 3: Vojtech, I remember that you didn't want to remove yet references from the legacy user portal from the POMs. Is it

[Engine-patches] Change in ovirt-engine[master]: core: Remove ovirtengine.war

2012-06-03 Thread iheim
Itamar Heim has posted comments on this change. Change subject: core: Remove ovirtengine.war .. Patch Set 1: just a btw - unless strictly related, additional cleanups can be done in additional patches (submitters decree), but

[Engine-patches] Change in ovirt-engine[master]: core: Remove ovirtengine.war

2012-06-03 Thread dfediuck
Doron Fediuck has posted comments on this change. Change subject: core: Remove ovirtengine.war .. Patch Set 1: (1 inline comment) Hey, looks like there are a few more bits you can remove (while at it). See inside. ...

[Engine-patches] Change in ovirt-engine[master]: core: Remove ovirtengine.war

2012-06-01 Thread juan . hernandez
Juan Hernandez has uploaded a new change for review. Change subject: core: Remove ovirtengine.war .. core: Remove ovirtengine.war Despite the apparently important name this is not used at all, so it can be safely removed. Chang