Change in vdsm[master]: spec: bump policycoreutils-python version to 2.0.83-19.47.

mtayer at redhat.com mtayer at redhat.com
Wed Nov 5 10:10:07 UTC 2014


mooli tayer has posted comments on this change.

Change subject: spec: bump policycoreutils-python version to 2.0.83-19.47.
......................................................................


Patch Set 7:

(1 comment)

ps I don't understand why we have the BuildRequires on policycoreutils-python.

http://gerrit.ovirt.org/#/c/33771/7/vdsm.spec.in
File vdsm.spec.in:

Line 80: BuildRequires: libselinux-python
Line 81: BuildRequires: libvirt-python
Line 82: BuildRequires: m2crypto
Line 83: BuildRequires: openssl
Line 84: BuildRequires: policycoreutils-python
Dan: we have it here.

I'm pretty sure this logic works: always policycoreutils-python, lf rhel6 also policycoreutils-python > 2.0.83-19.47. But it could be changed to if rhel6 else.

Do we care?
Line 85: BuildRequires: psmisc
Line 86: BuildRequires: python-cpopen >= 1.3
Line 87: BuildRequires: python-inotify
Line 88: BuildRequires: python-ioprocess >= 0.14


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Idc1c30a157d6763674874113e62add82833bd070
Gerrit-PatchSet: 7
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: mooli tayer <mtayer at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: David Caro <dcaroest at redhat.com>
Gerrit-Reviewer: Eyal Edri <eedri at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Ohad Basan <obasan at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: mooli tayer <mtayer at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list