[Qemu-devel] [Bug 1614609] [NEW] alphabetical order of monitor options

[Qemu-devel] [Bug 1614609] [NEW] alphabetical order of monitor options

[Kai Poeritz]

Public bug reported:

Looking for the 'continue'/'resume' option I found this order that was not quite 'alphabetical'.
It had me overlook the 'cont' option at glance. Which is just a little impractical.

...
boot_set bootdevice -- define new values for the boot device list
change device filename [format [read-only-mode]] -- change a removable medium, optional format
chardev-add args -- add chardev
chardev-remove id -- remove chardev
client_migrate_info protocol hostname port tls-port cert-subject -- set migration information for remote display
closefd closefd name -- close a file descriptor previously passed via SCM rights
commit device|all -- commit changes to the disk images (if -snapshot is used) or backing files
cpu index -- set the default CPU
cpu-add id -- add cpu
c|cont  -- resume emulation
delvm tag|id -- delete a VM snapshot from its tag or id
...

I tested this list with 'sort' just to make sure and make a point:

$ cat Desktop/order-orig.txt 
boot_set
change
chardev-add
chardev-remove
client_migrate_info
closefd
commit
cpu
cpu-add
c|cont
delvm
$ cat Desktop/order-orig.txt | sort
boot_set
c|cont
change
chardev-add
chardev-remove
client_migrate_info
closefd
commit
cpu
cpu-add
delvm
$

** Affects: qemu
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1614609

Title:
  alphabetical order of monitor options

Status in QEMU:
  New

Bug description:
  Looking for the 'continue'/'resume' option I found this order that was not quite 'alphabetical'.
  It had me overlook the 'cont' option at glance. Which is just a little impractical.

  ...
  boot_set bootdevice -- define new values for the boot device list
  change device filename [format [read-only-mode]] -- change a removable medium, optional format
  chardev-add args -- add chardev
  chardev-remove id -- remove chardev
  client_migrate_info protocol hostname port tls-port cert-subject -- set migration information for remote display
  closefd closefd name -- close a file descriptor previously passed via SCM rights
  commit device|all -- commit changes to the disk images (if -snapshot is used) or backing files
  cpu index -- set the default CPU
  cpu-add id -- add cpu
  c|cont  -- resume emulation
  delvm tag|id -- delete a VM snapshot from its tag or id
  ...

  I tested this list with 'sort' just to make sure and make a point:

  $ cat Desktop/order-orig.txt 
  boot_set
  change
  chardev-add
  chardev-remove
  client_migrate_info
  closefd
  commit
  cpu
  cpu-add
  c|cont
  delvm
  $ cat Desktop/order-orig.txt | sort
  boot_set
  c|cont
  change
  chardev-add
  chardev-remove
  client_migrate_info
  closefd
  commit
  cpu
  cpu-add
  delvm
  $

To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1614609/+subscriptions

[Bug 1614609] Re: alphabetical order of monitor options

[Peter Maydell]

Should be fixed by this patch:
https://patchew.org/QEMU/20201121151711.20783-1-peter.maydell@linaro.org/

** Changed in: qemu
       Status: New => In Progress

-- 
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1614609

Title:
  alphabetical order of monitor options

Status in QEMU:
  In Progress

Bug description:
  Looking for the 'continue'/'resume' option I found this order that was not quite 'alphabetical'.
  It had me overlook the 'cont' option at glance. Which is just a little impractical.

  ...
  boot_set bootdevice -- define new values for the boot device list
  change device filename [format [read-only-mode]] -- change a removable medium, optional format
  chardev-add args -- add chardev
  chardev-remove id -- remove chardev
  client_migrate_info protocol hostname port tls-port cert-subject -- set migration information for remote display
  closefd closefd name -- close a file descriptor previously passed via SCM rights
  commit device|all -- commit changes to the disk images (if -snapshot is used) or backing files
  cpu index -- set the default CPU
  cpu-add id -- add cpu
  c|cont  -- resume emulation
  delvm tag|id -- delete a VM snapshot from its tag or id
  ...

  I tested this list with 'sort' just to make sure and make a point:

  $ cat Desktop/order-orig.txt 
  boot_set
  change
  chardev-add
  chardev-remove
  client_migrate_info
  closefd
  commit
  cpu
  cpu-add
  c|cont
  delvm
  $ cat Desktop/order-orig.txt | sort
  boot_set
  c|cont
  change
  chardev-add
  chardev-remove
  client_migrate_info
  closefd
  commit
  cpu
  cpu-add
  delvm
  $

To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1614609/+subscriptions

[Bug 1614609] Re: alphabetical order of monitor options

[Thomas Huth]

Fix has been included here:
https://gitlab.com/qemu-project/qemu/-/commit/ff688cd2c7c3a677b71e

** Changed in: qemu
       Status: In Progress => Fix Committed

-- 
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1614609

Title:
  alphabetical order of monitor options

Status in QEMU:
  Fix Committed

Bug description:
  Looking for the 'continue'/'resume' option I found this order that was not quite 'alphabetical'.
  It had me overlook the 'cont' option at glance. Which is just a little impractical.

  ...
  boot_set bootdevice -- define new values for the boot device list
  change device filename [format [read-only-mode]] -- change a removable medium, optional format
  chardev-add args -- add chardev
  chardev-remove id -- remove chardev
  client_migrate_info protocol hostname port tls-port cert-subject -- set migration information for remote display
  closefd closefd name -- close a file descriptor previously passed via SCM rights
  commit device|all -- commit changes to the disk images (if -snapshot is used) or backing files
  cpu index -- set the default CPU
  cpu-add id -- add cpu
  c|cont  -- resume emulation
  delvm tag|id -- delete a VM snapshot from its tag or id
  ...

  I tested this list with 'sort' just to make sure and make a point:

  $ cat Desktop/order-orig.txt 
  boot_set
  change
  chardev-add
  chardev-remove
  client_migrate_info
  closefd
  commit
  cpu
  cpu-add
  c|cont
  delvm
  $ cat Desktop/order-orig.txt | sort
  boot_set
  c|cont
  change
  chardev-add
  chardev-remove
  client_migrate_info
  closefd
  commit
  cpu
  cpu-add
  delvm
  $

To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1614609/+subscriptions

[Bug 1614609] Re: alphabetical order of monitor options

[Thomas Huth]

** Changed in: qemu
       Status: Fix Committed => Fix Released

-- 
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1614609

Title:
  alphabetical order of monitor options

Status in QEMU:
  Fix Released

Bug description:
  Looking for the 'continue'/'resume' option I found this order that was not quite 'alphabetical'.
  It had me overlook the 'cont' option at glance. Which is just a little impractical.

  ...
  boot_set bootdevice -- define new values for the boot device list
  change device filename [format [read-only-mode]] -- change a removable medium, optional format
  chardev-add args -- add chardev
  chardev-remove id -- remove chardev
  client_migrate_info protocol hostname port tls-port cert-subject -- set migration information for remote display
  closefd closefd name -- close a file descriptor previously passed via SCM rights
  commit device|all -- commit changes to the disk images (if -snapshot is used) or backing files
  cpu index -- set the default CPU
  cpu-add id -- add cpu
  c|cont  -- resume emulation
  delvm tag|id -- delete a VM snapshot from its tag or id
  ...

  I tested this list with 'sort' just to make sure and make a point:

  $ cat Desktop/order-orig.txt 
  boot_set
  change
  chardev-add
  chardev-remove
  client_migrate_info
  closefd
  commit
  cpu
  cpu-add
  c|cont
  delvm
  $ cat Desktop/order-orig.txt | sort
  boot_set
  c|cont
  change
  chardev-add
  chardev-remove
  client_migrate_info
  closefd
  commit
  cpu
  cpu-add
  delvm
  $

To manage notifications about this bug go to:
https://bugs.launchpad.net/qemu/+bug/1614609/+subscriptions