[libvirt] [PATCH] docs: index.html.in: Deduplicate the link of QEMU and KVM

Han Han posted 1 patch 5 years ago
Test syntax-check passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20190416022813.4645-1-hhan@redhat.com
docs/index.html.in | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
[libvirt] [PATCH] docs: index.html.in: Deduplicate the link of QEMU and KVM
Posted by Han Han 5 years ago
Because the page of QEMU and KVM use the same doc drvqemu.html, merge the into
one and name it as QEMU/KVM.

Signed-off-by: Han Han <hhan@redhat.com>
---
 docs/index.html.in | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/docs/index.html.in b/docs/index.html.in
index f593445d06..97a4e9eef7 100644
--- a/docs/index.html.in
+++ b/docs/index.html.in
@@ -31,8 +31,8 @@
         <li>is a toolkit to manage <a href="platforms.html">virtualization platforms</a></li>
         <li>is accessible from C, Python, Perl, Java and more</li>
         <li>is licensed under open source licenses</li>
-        <li>supports <a href="drvqemu.html">KVM</a>,
-          <a href="drvqemu.html">QEMU</a>, <a href="drvxen.html">Xen</a>,
+        <li>supports <a href="drvqemu.html">QEMU/KVM</a>,
+          <a href="drvxen.html">Xen</a>,
           <a href="drvvirtuozzo.html">Virtuozzo</a>,
           <a href="drvesx.html">VMWare ESX</a>,
           <a href="drvlxc.html">LXC</a>,
-- 
2.20.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] docs: index.html.in: Deduplicate the link of QEMU and KVM
Posted by Cole Robinson 5 years ago
On 4/15/19 10:28 PM, Han Han wrote:
> Because the page of QEMU and KVM use the same doc drvqemu.html, merge the into
> one and name it as QEMU/KVM.
> 
> Signed-off-by: Han Han <hhan@redhat.com>
> ---
>  docs/index.html.in | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/docs/index.html.in b/docs/index.html.in
> index f593445d06..97a4e9eef7 100644
> --- a/docs/index.html.in
> +++ b/docs/index.html.in
> @@ -31,8 +31,8 @@
>          <li>is a toolkit to manage <a href="platforms.html">virtualization platforms</a></li>
>          <li>is accessible from C, Python, Perl, Java and more</li>
>          <li>is licensed under open source licenses</li>
> -        <li>supports <a href="drvqemu.html">KVM</a>,
> -          <a href="drvqemu.html">QEMU</a>, <a href="drvxen.html">Xen</a>,
> +        <li>supports <a href="drvqemu.html">QEMU/KVM</a>,
> +          <a href="drvxen.html">Xen</a>,
>            <a href="drvvirtuozzo.html">Virtuozzo</a>,
>            <a href="drvesx.html">VMWare ESX</a>,
>            <a href="drvlxc.html">LXC</a>,
> 

This was added with the frontpage rewrite in:

commit b17a97f223ad2feb587ee677246cec8ec968e7df
Author: Daniel P. Berrange <berrange@redhat.com>
Date:   Wed Oct 26 10:37:06 2016 +0200

    docs: rewrite content on front page to be more useful

The previous content had joined QEMU/KVM link text, but that commit
explicitly separated them. Maybe for marketing reasons. Dan?

- Cole

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] docs: index.html.in: Deduplicate the link of QEMU and KVM
Posted by Daniel P. Berrangé 5 years ago
On Tue, Apr 16, 2019 at 01:55:42PM -0400, Cole Robinson wrote:
> On 4/15/19 10:28 PM, Han Han wrote:
> > Because the page of QEMU and KVM use the same doc drvqemu.html, merge the into
> > one and name it as QEMU/KVM.
> > 
> > Signed-off-by: Han Han <hhan@redhat.com>
> > ---
> >  docs/index.html.in | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/docs/index.html.in b/docs/index.html.in
> > index f593445d06..97a4e9eef7 100644
> > --- a/docs/index.html.in
> > +++ b/docs/index.html.in
> > @@ -31,8 +31,8 @@
> >          <li>is a toolkit to manage <a href="platforms.html">virtualization platforms</a></li>
> >          <li>is accessible from C, Python, Perl, Java and more</li>
> >          <li>is licensed under open source licenses</li>
> > -        <li>supports <a href="drvqemu.html">KVM</a>,
> > -          <a href="drvqemu.html">QEMU</a>, <a href="drvxen.html">Xen</a>,
> > +        <li>supports <a href="drvqemu.html">QEMU/KVM</a>,
> > +          <a href="drvxen.html">Xen</a>,
> >            <a href="drvvirtuozzo.html">Virtuozzo</a>,
> >            <a href="drvesx.html">VMWare ESX</a>,
> >            <a href="drvlxc.html">LXC</a>,
> > 
> 
> This was added with the frontpage rewrite in:
> 
> commit b17a97f223ad2feb587ee677246cec8ec968e7df
> Author: Daniel P. Berrange <berrange@redhat.com>
> Date:   Wed Oct 26 10:37:06 2016 +0200
> 
>     docs: rewrite content on front page to be more useful
> 
> The previous content had joined QEMU/KVM link text, but that commit
> explicitly separated them. Maybe for marketing reasons. Dan?

Yes, exactly, we want to call out KVM as an explicit think on its own
for clarity. I don't see any problem with having a duplicated link.


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list