Cole Robinson <crobinso(a)redhat.com> wrote:
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.
Thanks for the quick review.
I've committed that.