Bug 970825

I can reproduce the bug

Version:
virt-viewer-0.5.6-1.el7

Steps:
1.# man remote-viewer
2.# remote-viewer --help

Expected results:
1. Remote-viewer now has ability to launch connection from a init file, we should add something to introduce this in man page and help info.
2. Should attach an init file as an example into man page.
3. Should list all options supported by init file in man page or at least provide a link to tell user how to find them and supported values for them.
4. Should add something to introduce spice_proxy and how to set it into man page

Actual results:
1. Nothing about launching connection from a init file and spice_proxy
 
Additional info:

I sent a patch to improve man:
https://www.redhat.com/archives/virt-tools-list/2013-August/msg00031.html

Reproduced the bug:
# rpm -qa virt-viewer 
virt-viewer-0.5.7-4.el7.x86_64

Steps:
1.# man remote-viewer
2.# remote-viewer --help

Expected results:
1. Remote-viewer now has ability to launch connection from a init file, we should add something to introduce this in man page and help info.
2. Should attach an init file as an example into man page.
3. Should list all options supported by init file in man page or at least provide a link to tell user how to find them and supported values for them.
4. Should add something to introduce spice_proxy and how to set it into man page

Actual results:
1. Nothing about launching connection from a init file and spice_proxy

Recheck the modification on the fixed version: 
# rpm -qa virt-viewer
virt-viewer-0.5.7-6.el7.x86_64

Steps:
1.# man remote-viewer
2.# remote-viewer --help

Actual results:
1. Add the introduction about this function, the init file example, supported options and corresponding values in the man page. 
And there's a typo in Key list as below, should be "silently" not "silentely"
"delete-this-file" (boolean)
           Set to 1 for the client to remove this connection file (if it can't, it will fail silentely)

2. There's no modification about launching remote-viewer by the init file in the help list, is it possible to improve the help? 

Due to the above test result, I assign this bug.

Hi Marcandre,
As comment5, would you please fix the problem(at least fix the typo) or consider to remove this bug from RHEL7.0 errata?

Regards,
Lei

Created attachment 873374 [details]
Document virt-viewer --full-screen=auto-conf in virt-viewer man page

--full-screen=auto-conf is gone upstream so this patch would be RHEL-only. The typo fix is https://www.redhat.com/archives/virt-tools-list/2014-March/msg00052.html

To get this into RHEL7.0, we'd need blocker+ though.

Moved to 7.1. Not sure why I put the '--fullscreen=autoconf' patch in there, probably wrong bug.

Moving back to NEW as "2. There's no modification about launching remote-viewer by the init file in the help list, is it possible to improve the help? " is still missing.

(In reply to Christophe Fergeau from comment #12)
> Moving back to NEW as "2. There's no modification about launching
> remote-viewer by the init file in the help list, is it possible to improve
> the help? " is still missing.

see commit 657d490e93764274a140794742dae96060e7d944

I can reproduce with package:virt-viewer-0.5.7-7.el7.x86_64
and try to verify with new build: virt-viewer-0.6.0-1.el7.x86_64

steps:
1. check remote-viewer help info:

# remote-viewer --help
Usage:
  remote-viewer [OPTION...] URI|VV-FILE - Remote viewer client
...
or
# remote-viewer -h
Usage:
  remote-viewer [OPTION...] URI|VV-FILE - Remote viewer client
...
we can see VV-FILE added, but the output format before(virt-viewer-0.5.7-7.el7.x86_64) is:
# remote-viewer --help 
Usage:
  remote-viewer [OPTION...] -- URI - Remote viewer client
                            ^now we can see "--" is missed in new version.

2. check remote-viewer manual page:
# man remote-viewer

NAME
       remote-viewer - a simple remote desktop client

SYNOPSIS
       remote-viewer [OPTIONS] -- [URI]
...

Q1: shall we update SYNOPSIS consistent with "URI|VV-FILE" with help info.

Q2: Issue on Comment 5 still exist.
...
"delete-this-file" (boolean)
           Set to 1 for the client to remove this connection file (if it can't, it will fail silentely)
             ^ wrong typo 

so Marc-Andre Lureau, please help check above issue, does it influence we verify this bug, thanks.

(In reply to zhoujunqin from comment #17)
> # remote-viewer --help 
> Usage:
>   remote-viewer [OPTION...] -- URI - Remote viewer client
>                             ^now we can see "--" is missed in new version.

The -- was removed by the following commit. It is not required (it used to be there to show that URI are better placed as last argument)

commit 657d490e93764274a140794742dae96060e7d944
Author: Jonathon Jongsma <jjongsma>
Date:   Tue Jul 1 15:10:48 2014 -0500

> 2. check remote-viewer manual page:
> # man remote-viewer
> 
> NAME
>        remote-viewer - a simple remote desktop client
> 
> SYNOPSIS
>        remote-viewer [OPTIONS] -- [URI]
> ...
> 
> Q1: shall we update SYNOPSIS consistent with "URI|VV-FILE" with help info.

Well, it's also a URI. Man page and --help don't have to be in sync, even if it is nicer.
 
> Q2: Issue on Comment 5 still exist.
> ...
> "delete-this-file" (boolean)
>            Set to 1 for the client to remove this connection file (if it
> can't, it will fail silentely)
>              ^ wrong typo 
> 
> so Marc-Andre Lureau, please help check above issue, does it influence we
> verify this bug, thanks.

right we missed cherry-pick e0e25be56dd9e2d21f55d94ba3e45890b314333d

Verified with package: 
virt-viewer-0.6.0-3.el7.x86_64

Steps:
1. check remote-viewer manual page:
# man remote-viewer
...
"delete-this-file" (boolean)
           Set to 1 for the client to remove this connection file (if it can't, it will fail silently)
...
we see the "silentely" has been changed to silently.

and according to Comment #18, move this bug from ON_QA to VERIFIED.

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHBA-2015-0295.html