Dan Kenigsberg has posted comments on this change.

Change subject: Adding policycoreutils-python-utils dependency for fedora 23 
and above
......................................................................


Patch Set 4:

(3 comments)

https://gerrit.ovirt.org/#/c/44019/4//COMMIT_MSG
Commit Message:

Line 5: CommitDate: 2015-10-29 05:49:00 -0400
Line 6: 
Line 7: Adding policycoreutils-python-utils dependency for fedora 23 and above
Line 8: 
Line 9: The utilities of policycoreutils-python splited and now contains also
the semanage utility has moved to policycoreutils-python-utils. Require the new 
package in Fedora 23.
Line 10: policycoreutils-python-utils in fedora 23 which vdsm should require
Line 11: 
Line 12: Change-Id: I4d0b9fbfd77419cbf885c7663d799f19213592f5
Line 13: Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1246626


https://gerrit.ovirt.org/#/c/44019/4/vdsm.spec.in
File vdsm.spec.in:

Line 85: BuildRequires: libvirt-python
Line 86: BuildRequires: m2crypto
Line 87: BuildRequires: mom >= 0.4.5
Line 88: BuildRequires: openssl
Line 89: %if 0%{?fedora} >= 23
this is not needed in BuildRequires
Line 90: BuildRequires: policycoreutils-python-utils
Line 91: %endif # fedora 23
Line 92: BuildRequires: policycoreutils-python
Line 93: BuildRequires: psmisc


Line 209: Requires: e2fsprogs >= 1.41.14
Line 210: %if 0%{?fedora} >= 23
Line 211: Requires: policycoreutils-python-utils
Line 212: %else
Line 213: Requires: policycoreutils-python
we need both in Fedora 23
Line 214: %endif # fedora 23
Line 215: Requires: sed >= 4.2.1-10
Line 216: Requires: ed
Line 217: Requires: lvm2 >= 2.02.107


-- 
To view, visit https://gerrit.ovirt.org/44019
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4d0b9fbfd77419cbf885c7663d799f19213592f5
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to