Yaniv Dary has posted comments on this change.

Change subject: packaging: added override file for application context files
......................................................................


Patch Set 5:

> 1. In the setup you copy the files from the server-customization to the 
> WEB-INF directory. Won't two files exist, your override and the original one?

I should overwrite every time, so we can fix bugs. We also update the SSO 
related stuff every time.

> 2. You once changed the applicationContext-security-web.xml file, and now the 
> applicationContext.xml file. Is that intentionally?

I don't edit either of them anymore only add one file.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1f9e637dd385dfd6928a511ec801759616746782
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-reports
Gerrit-Branch: master
Gerrit-Owner: Yaniv Dary <yd...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: Yaniv Dary <yd...@redhat.com>
Gerrit-Reviewer: mooli tayer <mta...@redhat.com>
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to