diff mbox series

[ovs-dev] Documentation: Document optional RHEL7 repositories

Message ID 1516224261-5152-1-git-send-email-gvrose8192@gmail.com
State Superseded
Headers show
Series [ovs-dev] Documentation: Document optional RHEL7 repositories | expand

Commit Message

Gregory Rose Jan. 17, 2018, 9:24 p.m. UTC
On minimal install RHEL 7 servers (and perhaps other types of installs)
you need to enable a couple of optional repositories for the yum-builddep
utility to work correctly.  This patch documents those two optional
repositories.

Signed-off-by: Greg Rose <gvrose8192@gmail.com>
---
 Documentation/intro/install/fedora.rst | 5 +++++
 1 file changed, 5 insertions(+)

Comments

Flavio Leitner Jan. 18, 2018, 12:15 a.m. UTC | #1
On Wed, Jan 17, 2018 at 01:24:21PM -0800, Greg Rose wrote:
> On minimal install RHEL 7 servers (and perhaps other types of installs)
> you need to enable a couple of optional repositories for the yum-builddep
> utility to work correctly.  This patch documents those two optional
> repositories.


> 
> Signed-off-by: Greg Rose <gvrose8192@gmail.com>
> ---
>  Documentation/intro/install/fedora.rst | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/Documentation/intro/install/fedora.rst b/Documentation/intro/install/fedora.rst
> index 8d10338..fd010f1 100644
> --- a/Documentation/intro/install/fedora.rst
> +++ b/Documentation/intro/install/fedora.rst
> @@ -63,6 +63,11 @@ The command below will create a temporary SPEC file::
>        > /tmp/ovs.spec
>  
>  And to install specific dependencies, use the corresponding tool below.
> +For some of the dependencies you may need to add two additional
> +repositories to help yum-builddep.

For some of the dependencies on RHEL you may need...

Because it doesn't apply to Fedora, otherwise it looks good.

fbl

> +
> +subscription-manager repos --enable=rhel-7-server-extras-rpms
> +subscription-manager repos --enable=rhel-7-server-optional-rpms
>  
>  DNF::
>  
> -- 
> 1.8.3.1
> 
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
Flavio Leitner Jan. 18, 2018, 12:16 a.m. UTC | #2
On Wed, Jan 17, 2018 at 10:15:31PM -0200, Flavio Leitner wrote:
> On Wed, Jan 17, 2018 at 01:24:21PM -0800, Greg Rose wrote:
> > On minimal install RHEL 7 servers (and perhaps other types of installs)
> > you need to enable a couple of optional repositories for the yum-builddep
> > utility to work correctly.  This patch documents those two optional
> > repositories.
> 
> 
> > 
> > Signed-off-by: Greg Rose <gvrose8192@gmail.com>
> > ---
> >  Documentation/intro/install/fedora.rst | 5 +++++
> >  1 file changed, 5 insertions(+)
> > 
> > diff --git a/Documentation/intro/install/fedora.rst b/Documentation/intro/install/fedora.rst
> > index 8d10338..fd010f1 100644
> > --- a/Documentation/intro/install/fedora.rst
> > +++ b/Documentation/intro/install/fedora.rst
> > @@ -63,6 +63,11 @@ The command below will create a temporary SPEC file::
> >        > /tmp/ovs.spec
> >  
> >  And to install specific dependencies, use the corresponding tool below.
> > +For some of the dependencies you may need to add two additional
> > +repositories to help yum-builddep.
> 
> For some of the dependencies on RHEL you may need...
> 
> Because it doesn't apply to Fedora, otherwise it looks good.

and thanks a lot for improving the documentation! :)
Gregory Rose Jan. 18, 2018, 12:17 a.m. UTC | #3
On 1/17/2018 4:15 PM, Flavio Leitner wrote:
> On Wed, Jan 17, 2018 at 01:24:21PM -0800, Greg Rose wrote:
>> On minimal install RHEL 7 servers (and perhaps other types of installs)
>> you need to enable a couple of optional repositories for the yum-builddep
>> utility to work correctly.  This patch documents those two optional
>> repositories.
>
>> Signed-off-by: Greg Rose <gvrose8192@gmail.com>
>> ---
>>   Documentation/intro/install/fedora.rst | 5 +++++
>>   1 file changed, 5 insertions(+)
>>
>> diff --git a/Documentation/intro/install/fedora.rst b/Documentation/intro/install/fedora.rst
>> index 8d10338..fd010f1 100644
>> --- a/Documentation/intro/install/fedora.rst
>> +++ b/Documentation/intro/install/fedora.rst
>> @@ -63,6 +63,11 @@ The command below will create a temporary SPEC file::
>>         > /tmp/ovs.spec
>>   
>>   And to install specific dependencies, use the corresponding tool below.
>> +For some of the dependencies you may need to add two additional
>> +repositories to help yum-builddep.
> For some of the dependencies on RHEL you may need...
>
> Because it doesn't apply to Fedora, otherwise it looks good.
>
> fbl

Thanks for the review Flavio.  I'll send a V2 with your suggested change 
after the patch
has a little more time for others to look at it.

- Greg
>
>> +
>> +subscription-manager repos --enable=rhel-7-server-extras-rpms
>> +subscription-manager repos --enable=rhel-7-server-optional-rpms
>>   
>>   DNF::
>>   
>> -- 
>> 1.8.3.1
>>
>> _______________________________________________
>> dev mailing list
>> dev@openvswitch.org
>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
diff mbox series

Patch

diff --git a/Documentation/intro/install/fedora.rst b/Documentation/intro/install/fedora.rst
index 8d10338..fd010f1 100644
--- a/Documentation/intro/install/fedora.rst
+++ b/Documentation/intro/install/fedora.rst
@@ -63,6 +63,11 @@  The command below will create a temporary SPEC file::
       > /tmp/ovs.spec
 
 And to install specific dependencies, use the corresponding tool below.
+For some of the dependencies you may need to add two additional
+repositories to help yum-builddep.
+
+subscription-manager repos --enable=rhel-7-server-extras-rpms
+subscription-manager repos --enable=rhel-7-server-optional-rpms
 
 DNF::