Re: [libvirt] [PATCH] docs: introduce libvirt-dbus binding

On Fri, Mar 23, 2018 at 12:26:56PM +0100, Pavel Hrdina wrote: > On Fri, Mar 23, 2018 at 10:15:58AM +0000, Daniel P. Berrangé wrote: > > On Fri, Mar 23, 2018 at 08:35:34AM +0100, Pavel Hrdina wrote: > > > libvirt-dbus is a new binding that wraps libvirt API into D-Bus calls. > > > > > > Signed-off-by: Pavel Hrdina <phrdina(a)redhat.com&gt; > > > --- > > > docs/bindings.html.in | 4 ++ > > > docs/dbus.html.in | 102 +++++++++++++++++++++++++++++++++++++++++++++++++ > > > docs/docs.html.in | 1 + > > > docs/downloads.html.in | 15 ++++++++ > > > 4 files changed, 122 insertions(+) > > > create mode 100644 docs/dbus.html.in > > > > > > diff --git a/docs/downloads.html.in b/docs/downloads.html.in > > > index 7c28e2ba9b..c2bc7af3e9 100644 > > > --- a/docs/downloads.html.in > > > +++ b/docs/downloads.html.in > > > @@ -60,6 +60,21 @@ > > > </td> > > > <td></td> > > > </tr> > > > + <tr> > > > + <td>D-Bus</td> > > > + <td> > > > + <a href="ftp://libvirt.org/libvirt/dbus/">ftp</a> > > > + <a href="https://libvirt.org/sources/dbus/">https</a> > > > + </td> > > > + <td> > > > + <a href="https://libvirt.org/git/?p=libvirt-dbus.git;a=summary">... > > > + </td> > > > + <td> > > > + <a href="https://gitlab.com/libvirt/libvirt-dbus">gitlab</a> > > > + <a href="https://github.com/libvirt/libvirt-dbus">github</a> > > > + </td> > > > + <td></td> > > > + </tr> > > > > This should go in the "Integration modules" section, rather than the > > "language bindings" section of the table. > > In that case I should also remove it from all places that list bindings. > I'll put a link to the dbus.html page into "Resources" column on the > download page since there is no better place where to mention it. Assume you're referring to the entry on the 'docs.html.in' page ? We could just change the title from 'language bindings' to 'language bindings and API modules' and leave it there.