Bug 592180

Summary: Spelling error in `virsh nodedev-dettach`
Product: Red Hat Enterprise Linux 6 Reporter: zhanghaiyan <yoyzhang>
Component: libvirtAssignee: Daniel Veillard <veillard>
Status: CLOSED CANTFIX QA Contact: Virtualization Bugs <virt-bugs>
Severity: low Docs Contact:
Priority: low    
Version: 6.0CC: berrange, clalance, llim, notting, tyan, xen-maint
Target Milestone: rc   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2010-05-14 09:54:43 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description zhanghaiyan 2010-05-14 07:21:33 UTC
Description of problem:
`virsh nodedev-dettach` has spelling error - detach should has  1 't' instead of 2 't'

Version-Release number of selected component (if applicable):
- libvirt-0.8.1-3.el6.x86_64

How reproducible:
Always

Steps to Reproduce:
1. # virsh help nodedev-dettach
  NAME
    nodedev-dettach - dettach node device from its device driver

  SYNOPSIS
    nodedev-dettach <device>

  DESCRIPTION
    Dettach node device from its device driver before assigning to a domain.

  OPTIONS
    <device>         device key


Actual results:
it uses 'dettach', obviously spelling error

Expected results:
Replcae 'dettach' with 'detach'

Additional info:

Comment 2 RHEL Program Management 2010-05-14 08:37:34 UTC
This request was evaluated by Red Hat Product Management for inclusion in a Red
Hat Enterprise Linux major release.  Product Management has requested further
review of this request by Red Hat Engineering, for potential inclusion in a Red
Hat Enterprise Linux Major release.  This request is not yet committed for
inclusion.

Comment 3 Daniel Berrangé 2010-05-14 09:54:43 UTC
THis has been known about for a long time, but it cannot be fixed because renaming the command will break any scripts currently using this functionality.

Comment 4 Chris Lalancette 2010-05-14 13:38:55 UTC
However, we could add a shadow command that used exactly the same implementation, but was named correctly.  Then we would have both "virsh nodedev-dettach" and "virsh nodedev-detach", but newer users could use the correct spelling.

Chris Lalancette

Comment 5 Daniel Berrangé 2010-05-14 13:58:53 UTC
That will cause more trouble than it is worth, because then people will write scripts / docs using 'nodedev-detach' and they'll not be compatible with any existing deployment.

Comment 6 Bill Nottingham 2010-05-14 14:07:40 UTC
Well, we could certainly backport the right spelling to any production we ship, couldn't we? (I realize you can't *force* people to deploy errata, but...)