diff mbox

[ovs-dev,v2,1/2] rhel: Add hostname as dependency

Message ID 20170223210617.14032-2-russell@ovn.org
State Accepted
Delegated to: Russell Bryant
Headers show

Commit Message

Russell Bryant Feb. 23, 2017, 9:06 p.m. UTC
From: Jakub Libosvar <libosvar@redhat.com>

ovs-ctl script uses hostname. This patch adds dependency for
rhel systems.

Signed-off-by: Jakub Libosvar <libosvar@redhat.com>
Signed-off-by: Russell Bryant <russell@ovn.org>
---
 AUTHORS.rst                     | 1 +
 rhel/openvswitch-fedora.spec.in | 2 +-
 rhel/openvswitch.spec.in        | 2 +-
 3 files changed, 3 insertions(+), 2 deletions(-)

Comments

Russell Bryant Feb. 23, 2017, 9:11 p.m. UTC | #1
On Thu, Feb 23, 2017 at 4:06 PM, Russell Bryant <russell@ovn.org> wrote:

> From: Jakub Libosvar <libosvar@redhat.com>
>
> ovs-ctl script uses hostname. This patch adds dependency for
> rhel systems.
>
> Signed-off-by: Jakub Libosvar <libosvar@redhat.com>
> Signed-off-by: Russell Bryant <russell@ovn.org>
> ---
>  AUTHORS.rst                     | 1 +
>  rhel/openvswitch-fedora.spec.in | 2 +-
>  rhel/openvswitch.spec.in        | 2 +-
>  3 files changed, 3 insertions(+), 2 deletions(-)
>

I applied this patch to master and branch-2.7.  Thanks!

As I mentioned in the cover letter for this v2, I split out the debian
change into another patch and would prefer that someone familiar with
debian packaging review and apply that one.
diff mbox

Patch

diff --git a/AUTHORS.rst b/AUTHORS.rst
index 4dde860..39a5f40 100644
--- a/AUTHORS.rst
+++ b/AUTHORS.rst
@@ -148,6 +148,7 @@  Ian Campbell                    Ian.Campbell@citrix.com
 Ian Stokes                      ian.stokes@intel.com
 Ilya Maximets                   i.maximets@samsung.com
 Isaku Yamahata                  yamahata@valinux.co.jp
+Jakub Libosvar                  libosvar@redhat.com
 James P.                        roampune@gmail.com
 James Page                      james.page@ubuntu.com
 Jamie Lennox                    jamielennox@gmail.com
diff --git a/rhel/openvswitch-fedora.spec.in b/rhel/openvswitch-fedora.spec.in
index a1bf1c7..d100f49 100644
--- a/rhel/openvswitch-fedora.spec.in
+++ b/rhel/openvswitch-fedora.spec.in
@@ -72,7 +72,7 @@  BuildRequires: dpdk-devel >= 16.11
 Provides: %{name}-dpdk = %{version}-%{release}
 %endif
 
-Requires: openssl iproute module-init-tools
+Requires: openssl hostname iproute module-init-tools
 #Upstream kernel commit 4f647e0a3c37b8d5086214128614a136064110c3
 #Requires: kernel >= 3.15.0-0
 
diff --git a/rhel/openvswitch.spec.in b/rhel/openvswitch.spec.in
index 6aa23de..31a535c 100644
--- a/rhel/openvswitch.spec.in
+++ b/rhel/openvswitch.spec.in
@@ -32,7 +32,7 @@  License: ASL 2.0
 Release: 1
 Source: openvswitch-%{version}.tar.gz
 Buildroot: /tmp/openvswitch-rpm
-Requires: logrotate, python >= 2.7, python-six
+Requires: logrotate, hostname, python >= 2.7, python-six
 BuildRequires: python-six
 BuildRequires: openssl-devel
 BuildRequires: checkpolicy, selinux-policy-devel