Adam Litke has posted comments on this change. Change subject: Change versioning schema and fix tarball ......................................................................
Patch Set 7: Code-Review-1 (4 comments) Good overall, just some unresolved questions and minor suggestions. .................................................... File AUTHORS Line 1: The following people have contributed to this project. Line 2: Line 3: * Kiril Nesenko <[email protected]> If you're going to create this file, please add the historical authors as well. You can get the list pretty easily: git log --format="%an <%ae>" | sort | uniq This will produce a list that only needs a couple of obvious cleanups. .................................................... File configure.ac Line 6: define([VERSION_MINOR], [3]) Line 7: define([VERSION_FIX], [2]) Line 8: define([VERSION_NUMBER], VERSION_MAJOR[.]VERSION_MINOR[.]VERSION_FIX) Line 9: define([VERSION_SUFFIX], [_master]) Line 10: So now when building packages, we must manually edit this file first? Line 11: AC_INIT([mom], VERSION_NUMBER[]VERSION_SUFFIX, [[email protected]]) Line 12: PACKAGE_RPM_VERSION="VERSION_NUMBER" Line 13: PACKAGE_RPM_RELEASE="3.20.$(echo VERSION_SUFFIX | sed 's/^_//')" Line 14: PACKAGE_NAME="mom" Line 9: define([VERSION_SUFFIX], [_master]) Line 10: Line 11: AC_INIT([mom], VERSION_NUMBER[]VERSION_SUFFIX, [[email protected]]) Line 12: PACKAGE_RPM_VERSION="VERSION_NUMBER" Line 13: PACKAGE_RPM_RELEASE="3.20.$(echo VERSION_SUFFIX | sed 's/^_//')" Where does '20' come from in the version above? Line 14: PACKAGE_NAME="mom" Line 15: AC_SUBST([PACKAGE_NAME]) Line 16: AC_SUBST([PACKAGE_RPM_VERSION]) Line 17: AC_SUBST([PACKAGE_RPM_RELEASE]) .................................................... File mom.spec.in Line 62: "%{buildroot}"/%{_sysconfdir}/momd.conf Line 63: Line 64: # Correct the installed location of documentation files Line 65: mv "%{buildroot}"%{_defaultdocdir}/%{package_name} \ Line 66: "%{buildroot}"%{_defaultdocdir}/%{package_name}-%{version} See https://bugzilla.redhat.com/show_bug.cgi?id=993977 Line 67: Line 68: %check Line 69: nosetests tests/*.py Line 70: -- To view, visit http://gerrit.ovirt.org/22874 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I4337150f58cd7d9b3a04a12a3afa9acd8a826a22 Gerrit-PatchSet: 7 Gerrit-Project: mom Gerrit-Branch: master Gerrit-Owner: Kiril Nesenko <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Doron Fediuck <[email protected]> Gerrit-Reviewer: Eyal Edri <[email protected]> Gerrit-Reviewer: Kiril Nesenko <[email protected]> Gerrit-Reviewer: Martin Sivák <[email protected]> Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
