Alon Bar-Lev has posted comments on this change.
Change subject: build: pass directories and utilities from spec to Makefile
......................................................................
Patch Set 5: (11 inline comments)
Thanks for review!
I still think the postgresql jdbc should go to spec :)))
....................................................
File Makefile
Line 27: PACKAGE_NAME=ovirt-engine
OK.
Line 32: DATA_DIR=$(DATAROOT_DIR)/$(PACKAGE_NAME)
Done
Line 347: install -dm 755
$(DESTDIR)$(DATA_DIR)/modules/org/postgresql/main
Done
Line 349: ln -s $(JAVA_DIR)/postgresql-jdbc.jar
$(DESTDIR)$(DATA_DIR)/modules/org/postgresql/main/.
Done
....................................................
File packaging/fedora/spec/ovirt-engine.spec.in
Line 15: %global engine_lock %{_localstatedir}/lock/%{engine_name}
this is the standard name for var...
Line 602: %{_mavenpomdir}/JPP.%{engine_name}-utils.pom
ok.
Line 632: %{_mavenpomdir}/JPP.%{engine_name}-vdsbroker.pom
Done
Line 728: %{_mavenpomdir}/JPP.%{engine_name}-restapi-types.pom
Done
Line 741: %{_mavenpomdir}/JPP.%{engine_name}-genericapi.pom
Done
Line 763: %{_mavenpomdir}/JPP.%{engine_name}-engine-tools-common.pom
Done
Line 809: %{_mavenpomdir}/JPP.%{engine_name}-engine-notifier.pom
Done
--
To view, visit http://gerrit.ovirt.org/6252
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I78708f7bdc4cbcdfc736210821a05cfd1800f234
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernan...@redhat.com>
Gerrit-Reviewer: Moran Goldboim <mgold...@redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschr...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
