Re: [Xen-devel] [PATCH v3 4/6] xl: Return an error on failed cd-insert

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

On Tue, 2015-12-01 at 11:53 +0000, George Dunlap wrote:
> This makes xl more useful in scripts.
> 
> The strange thing about this is that the internal cd_insert function
> *already* returned something appropriate, and cd-eject was using it,
> but cd-insert wasnt.

Missing an apostrophe in wasnt.

> Signed-off-by: George Dunlap <george.dunlap@xxxxxxxxxxxxx>

Acked-by: Ian Campbell <Ian.campbell@xxxxxxxxxx>

> ---
> CC: Ian Campbell <ian.campbell@xxxxxxxxxx>
> CC: Ian Jackson <ian.jackson@xxxxxxxxxx>
> CC: Wei Liu <wei.liu2@xxxxxxxxxx>
> ---
> Âtools/libxl/xl_cmdimpl.c | 3 +--
> Â1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c
> index 4455d73..72ece2e 100644
> --- a/tools/libxl/xl_cmdimpl.c
> +++ b/tools/libxl/xl_cmdimpl.c
> @@ -3405,8 +3405,7 @@ int main_cd_insert(int argc, char **argv)
> ÂÂÂÂÂvirtdev = argv[optind + 1];
> ÂÂÂÂÂfile = argv[optind + 2];
> Â
> -ÂÂÂÂcd_insert(domid, virtdev, file);
> -ÂÂÂÂreturn 0;
> +ÂÂÂÂreturn cd_insert(domid, virtdev, file);
> Â}
> Â
> Âint main_console(int argc, char **argv)

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel