Jim Meyering wrote:
> * src/virsh.c (cmdPoolStart, cmdStart): Change hard-coded
> vshCommandOptDomainBy string argument to match just-changed
> option name.  Cole Robinson reported that "virsh start" was
> broken and provided that part of the fix.
> Bug introduced by yesterday's "virsh.c: tweak options to produce
> more accurate help".
> * tests/start: New file.  Test for the above fix.
> * tests/Makefile.am (test_scripts): Add start.
> ---
> 
> Without this change, "virsh start" and "virsh pool-start"
> would always fail.
> 
> I'm fixing this in two steps: first is this patch: make the tiny
> string-changing fixes (along with the new test).
> Second will be a more invasive change to make it so the duplicate
> strings are removed altogether, so they'll never get out of sync again.
> 
> 

ACK.

- Cole

--
Libvir-list mailing list
Libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to