Re: [libvirt PATCH v2 1/1] virt-ssh-helper: Add manual page

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jan 07, 2022 at 04:30:33PM +0100, Michal Prívozník wrote:
> On 12/13/21 14:44, Andrea Bolognani wrote:
> > We don't usually provide manual pages for internal tools,
> > but in the case of virt-ssh-helper the command is installed
> > inside the default $PATH and so it's likely that the user
> > will stumble upon it by using the shell's completion feature
> > when invoking another virt-* command, which makes it a good
> > idea to provide at least a minimal manual page.
> >
> > Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx>
> > ---
> >  docs/manpages/meson.build         |  1 +
> >  docs/manpages/virt-ssh-helper.rst | 95 +++++++++++++++++++++++++++++++
> >  libvirt.spec.in                   |  1 +
> >  3 files changed, 97 insertions(+)
> >  create mode 100644 docs/manpages/virt-ssh-helper.rst
>
> Reviewed-by: Michal Privoznik <mprivozn@xxxxxxxxxx>

I just realized that there is a small bug in the patch:
virt-ssh-helper is only built and installed when the daemon is, and
obviously the same should happen to its manual page. I have squashed
in the diff below before pushing.

Thanks for the review! :)


diff --git a/docs/manpages/meson.build b/docs/manpages/meson.build
index bf6fc730e0..6763d19af8 100644
--- a/docs/manpages/meson.build
+++ b/docs/manpages/meson.build
@@ -19,7 +19,7 @@ docs_man_files = [
   { 'name': 'virt-pki-query-dn', 'section': '1', 'install': true },
   { 'name': 'virt-pki-validate', 'section': '1', 'install': true },
   { 'name': 'virt-qemu-run', 'section': '1', 'install':
conf.has('WITH_QEMU') },
-  { 'name': 'virt-ssh-helper', 'section': '1', 'install': true },
+  { 'name': 'virt-ssh-helper', 'section': '1', 'install':
conf.has('WITH_LIBVIRTD') },
   { 'name': 'virt-xml-validate', 'section': '1', 'install': true },

   { 'name': 'libvirtd', 'section': '8', 'install': conf.has('WITH_LIBVIRTD') },
-- 
Andrea Bolognani / Red Hat / Virtualization





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux