Alon Bar-Lev has posted comments on this change.

Change subject: aaa: Introducing extensions module
......................................................................


Patch Set 6:

(2 comments)

http://gerrit.ovirt.org/#/c/25192/6/ovirt-engine.spec.in
File ovirt-engine.spec.in:

Line 583: 
%{engine_jboss_modules}/org/ovirt/engine/core/searchbackend/main/searchbackend.jar
Line 584: %{engine_jboss_modules}/org/ovirt/engine/core/tools/main/tools.jar
Line 585: %{engine_jboss_modules}/org/ovirt/engine/core/utils/main/utils.jar
Line 586: %{engine_restapi_war}/WEB-INF/lib/interface-common-jaxrs.jar
Line 587: 
%{engine_jboss_modules}/org/ovirt/engine/api/ovirt-engine-extensions-api/main/ovirt-engine-extensions-api.jar
please sort()
Line 588: %{engine_restapi_war}/WEB-INF/lib/restapi-definition.jar
Line 589: %{engine_restapi_war}/WEB-INF/lib/restapi-jaxrs.jar
Line 590: %{engine_restapi_war}/WEB-INF/lib/restapi-types.jar
Line 591: __EOF__


Line 784: %{engine_java}/branding.jar
Line 785: %{engine_java}/common.jar
Line 786: %{engine_java}/compat.jar
Line 787: %{engine_java}/extension-manager.jar
Line 788: %{engine_java}ovirt-engine-extensions-api.jar
I already commented this... please add '/' and sort()
Line 789: %{engine_java}/utils.jar
Line 790: 
Line 791: #
Line 792: # ATTENTION:


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

Gerrit-MessageType: comment
Gerrit-Change-Id: If373ef883588fdf363f5800fa8fc74c7836838cd
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
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