[libvirt] [PATCH] spec: Fix requirement for "tc" on new distros

Jiri Denemark posted 1 patch 5 years, 10 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/fca281189dc6664404752b50027324e95a65ff63.1527855267.git.jdenemar@redhat.com
Test syntax-check passed
libvirt.spec.in | 6 ++++++
1 file changed, 6 insertions(+)
[libvirt] [PATCH] spec: Fix requirement for "tc" on new distros
Posted by Jiri Denemark 5 years, 10 months ago
At least since Fedora 26 (maybe earlier, but we don't support older
Fedora releases), the "tc" tool is provided by a separate iproute-tc
package.

Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
---
 libvirt.spec.in | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/libvirt.spec.in b/libvirt.spec.in
index 17d5d4dff7..55057e353c 100644
--- a/libvirt.spec.in
+++ b/libvirt.spec.in
@@ -482,8 +482,14 @@ Requires: %{name}-libs = %{version}-%{release}
 
 # for modprobe of pci devices
 Requires: module-init-tools
+
 # for /sbin/ip & /sbin/tc
 Requires: iproute
+# tc is provided by iproute-tc since at least Fedora 26
+%if 0%{?fedora} || 0%{?rhel} > 7
+Requires: iproute-tc
+%endif
+
 Requires: avahi-libs
 %if 0%{?fedora} || 0%{?rhel} >= 7
 Requires: polkit >= 0.112
-- 
2.17.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] spec: Fix requirement for "tc" on new distros
Posted by Peter Krempa 5 years, 10 months ago
On Fri, Jun 01, 2018 at 14:14:27 +0200, Jiri Denemark wrote:
> At least since Fedora 26 (maybe earlier, but we don't support older
> Fedora releases), the "tc" tool is provided by a separate iproute-tc
> package.
> 
> Signed-off-by: Jiri Denemark <jdenemar@redhat.com>
> ---
>  libvirt.spec.in | 6 ++++++
>  1 file changed, 6 insertions(+)

ACK && freeze-worthy
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list