Greg Sheremeta has posted comments on this change.

Change subject: engine: root.war cleanup
......................................................................


Patch Set 13: Code-Review+1

(1 comment)

+1, very nice patch.

....................................................
File backend/manager/modules/root/src/main/webapp/WEB-INF/web.xml
Line 95:   </servlet>
Line 96:   <servlet-mapping>
Line 97:     <servlet-name>docsmissing</servlet-name>
Line 98:     <url-pattern>/ovirt-engine/docsmissing</url-pattern>
Line 99:   </servlet-mapping>
+1. nice, never seen this usage of a JSP before.
Line 100:   <!-- Files: -->
Line 101:   <servlet>
Line 102:     <servlet-name>files</servlet-name>
Line 103:     <servlet-class>org.ovirt.engine.core.FileServlet</servlet-class>


-- 
To view, visit http://gerrit.ovirt.org/19848
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iceae0ebf671efc951522c464db1a5b2b95dd5637
Gerrit-PatchSet: 13
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alexander Wels <aw...@redhat.com>
Gerrit-Reviewer: Alexander Wels <aw...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Einav Cohen <eco...@redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gsher...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to