Bug 949373

(In reply to comment #0)
> Description
> Issues about virsh with incorrect argument
> 
> Version:
> libvirt-1.0.3-1.el7.x86_64
> 
> How reproducible:
> 100%
> 
> Steps to Reproduce:
> 1.Run virsh with incorrect argument.(Different output with rhel6.4,see
> Additional info)
> # virsh --debu
> error: option '-d' requires an argument

'--debu' is unambiguously short for '--debug', and since '--debug' and '-d' are synonyms, the output is technically correct.  But yes, it would be nice if the error message included the name of the option as the user typed it instead of canonicalizing it to the short option variant.

> 
> # virsh ---debug
> error: unsupported option '-

This is correct - there is no option that starts with three dashes.  But again, it would be nice if the error message included the entire option as the user typed it, instead of just the first character (the third dash) that caused the problem.  Furthermore, the output is unbalanced (missing a close '), so that should be fixed, even if we don't figure out how to replay what the user typed.

> 
> # virsh --debugg
> error: unsupported option '-

Again, there is no long option named --debugg, but again, displaying the full text of what the user typed would be better.

> 
> 2.Run virsh with --,no handle for it.(Same with rhel6.4,not sure whether it
> is designed by this)
> # virsh -- list --all

This is not a bug, but behavior required by POSIX.  The '--' marker must be silently ignored, and states that all remaining arguments are taken as arguments instead of options even if they start with '-'.  There is nothing to fix here.

> # virsh --
> Welcome to virsh, the virtualization interactive terminal.

Again, nothing to fix.

The fix is now pushed upstream:
commit 53531e16bf7cfe4fe20a4fe132f8199dcd474245
Author:     Ján Tomko <jtomko>
AuthorDate: 2013-04-29 19:12:17 +0200
Commit:     Ján Tomko <jtomko>
CommitDate: 2013-04-30 10:23:44 +0200

    virsh: fix incorrect argument errors for long options
    
    For long options, print:
    * the option as specified by the user if it's unknown
    * the canonical long option if its argument is not
      a number (and should be)
    
    And for missing arguments, print both the short and
    the long option name.
    (Doing only one of those would require either parsing
    argv ourselves or let getopt print the errors, since
    we can't tell long and short options apart by optopt
    or longindex)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=949373
    
    Unsupported long option:
    
    $ virsh --pm
    Before:
    error: unsupported option '-
    After:
    error: unsupported option '--pm'. See --help.
    
    Missing parameter:
    
    $ virsh --deb
    Before:
    error: option '-d' requires an argument
    After:
    error: option '-d'/'--debug' requires an argument
    
    $ virsh -rd
    Before:
    error: option '-d' requires an argument
    After:
    error: option '-d'/'--debug' requires an argument
    
    Non-numeric parameter:
    
    $ virsh --deb duck
    Before:
    error: option -d takes a numeric argument
    After:
    error: option --debug takes a numeric argument

can reporduce with libvirt-1.0.3-1.el7.x86_64
verify with build:
libvirt-1.0.5-1.el7.x86_64

step:
1.Run virsh with incorrect argument.(Different output with rhel6.4,see Additional info)
#virsh --debu
error: option '-d'/'--debug' requires an argument

#virsh ---debug
error: unsupported option '---debug'. See --help.

#virsh --debugg
error: unsupported option '--debugg'. See --help.

2.Run virsh with --,no handle for it.
#virsh -- list --all
Id Name State
----------------------------------------------------
24 test running
29 bb running
- test1 shut off
- win-mig shut off
- win7 shut off

#virsh --
Welcome to virsh, the virtualization interactive terminal.

Type: 'help' for help with commands
'quit' to quit

virsh#

#virsh --pm
error: unsupported option '--pm'. See --help.

#virsh --deb
error: option '-d'/'--debug' requires an argument

#virsh -rd
error: option '-d'/'--debug' requires an argument

#virsh --deb duck
error: option --debug takes a numeric argument

with wrong argument,it will have the error info to be correct and clear.verification passed

This request was resolved in Red Hat Enterprise Linux 7.0.

Contact your manager or support representative in case you have further questions about the request.