On Wed, Oct 26, 2011 at 7:18 PM, Wenyi Gao <wenyi@xxxxxxxxxxxxxxxxxx> wrote: > On Mon, 2011-10-24 at 20:44 +0900, Ryota Ozaki wrote: >> If vol-create-from is failed due to 'input volume not found', >> virsh outputs like this: >> $ sudo virsh vol-create-from testpool test-vol.xml test.img >> error: failed to get vol 'test.img', specifying --pool might help >> error: Storage volume not found: no storage vol with matching path >> However, '--pool' is incorrect because it is already specified as >> second argument ('testpool' in this case). It should be "--inputpool". >> >> The patch fixes this by using pooloptname, which will be "inputpool" >> in this case and "pool" in other cases, as error message. >> --- >> tools/virsh.c | 4 ++-- >> 1 files changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/tools/virsh.c b/tools/virsh.c >> index 72344f0..5544a41 100644 >> --- a/tools/virsh.c >> +++ b/tools/virsh.c >> @@ -15216,8 +15216,8 @@ vshCommandOptVolBy(vshControl *ctl, const vshCmd *cmd, >> if (pool) >> vshError(ctl, _("failed to get vol '%s'"), n); >> else >> - vshError(ctl, _("failed to get vol '%s', specifying --pool " >> - "might help"), n); >> + vshError(ctl, _("failed to get vol '%s', specifying --%s " >> + "might help"), n, pooloptname); >> } >> >> if (pool) > > > Tested-by Wenyi Gao <wenyi@xxxxxxxxxxxxxxxxxx> > > > Wenyi Gao > > Thanks! ozaki-r -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list