mooli tayer has posted comments on this change. Change subject: spec: bump policycoreutils-python version to 2.0.83-19.47 ......................................................................
Patch Set 1: (3 comments) http://gerrit.ovirt.org/#/c/33987/1/vdsm.spec.in File vdsm.spec.in: Line 184: Requires: e2fsprogs Line 185: Requires: fence-agents-all Line 186: Requires: lvm2 Line 187: Requires: python Line 188: Requires: policycoreutils-python > Sandro, these are good questions, that are better to be asked on the master on rhel 7 no need: # cat /etc/redhat-release Red Hat Enterprise Linux Server release 7.0 (Maipo) # rpm -qv policycoreutils-python policycoreutils-python-2.2.5-11.el7.x86_64 # python >>> import seobject >>> seobject.booleanRecords().modify('virt_use_samba','on') >>> Line 189: Requires: selinux-policy-targeted Line 190: %else Line 191: Requires: python Line 192: Requires: device-mapper-multipath >= 0.4.9-52 Line 208: Requires: python-ethtool >= 0.8-1 Line 209: Requires: initscripts >= 9.42.2-1 Line 210: Requires: device-mapper-multipath >= 0.4.9-18 Line 211: Requires: e2fsprogs >= 1.41.14 Line 212: Requires: policycoreutils-python on fedora no need: $ sudo python ... >>> import seobject; seobject.booleanRecords().modify('virt_use_samba','on') >>> $ cat /etc/red redhat-lsb/ redhat-release $ cat /etc/redhat-release Fedora release 20 (Heisenbug) $ rpm -qv policycoreutils-python policycoreutils-python-2.2.5-4.fc20.x86_64 Line 213: Requires: sed >= 4.2.1-10 Line 214: Requires: ed Line 215: Requires: lvm2 >= 2.02.98-15 Line 216: Requires: selinux-policy-targeted >= 3.12.1-71 Line 263: %else Line 264: Requires(post): /bin/hostname Line 265: %endif Line 266: Line 267: Requires(post): policycoreutils-python > Any specific version to be required here? Yes, I think I should devide to too sections: rhel < 7 and others define one with version and one with out Dan? Line 268: Requires(preun): policycoreutils-python Line 269: Line 270: %description Line 271: The VDSM service is required by a Virtualization Manager to manage the -- To view, visit http://gerrit.ovirt.org/33987 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Idc1c30a157d6763674874113e62add82833bd070 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5.0 Gerrit-Owner: mooli tayer <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: Sandro Bonazzola <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: mooli tayer <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
