Re: [PATCH] virsh: fix a typos issue cause error not right

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 05/20/2015 09:32 PM, Luyao Huang wrote:
> Signed-off-by: Luyao Huang <lhuang@xxxxxxxxxx>
> ---
>  tools/virsh-host.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

While correct, this would also be fixed by patches on list for "more
common" virsh error message processing currently by Andrea Bolognani, see:

http://www.redhat.com/archives/libvir-list/2015-May/msg00686.html

Let's see where that series goes first...

John
> diff --git a/tools/virsh-host.c b/tools/virsh-host.c
> index a72fd05..e939dac 100644
> --- a/tools/virsh-host.c
> +++ b/tools/virsh-host.c
> @@ -500,7 +500,7 @@ cmdAllocpages(vshControl *ctl, const vshCmd *cmd)
>      if (vshCommandOptScaledInt(cmd, "pagesize", &tmp, 1024, UINT_MAX) < 0) {
>          vshError(ctl,
>                   _("Numeric value for <%s> option is malformed or out of range"),
> -                 "cellno");
> +                 "pagesize");
>          return false;
>      }
>      pageSizes[0] = VIR_DIV_UP(tmp, 1024);
> 

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




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]