Re: [Xen-devel] [PATCH 3 of 3 v2] libxl: Warn that /usr/bin/pygrub is deprecated

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Tue, 2012-05-15 at 11:33 +0100, George Dunlap wrote:
> v2: Use strcmp rather than strncmp.
> 
> Signed-off-by: George Dunlap <george.dunlap@xxxxxxxxxxxxx>

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

> 
> diff -r 4503705206f3 -r 01804c550dd1 tools/libxl/libxl_bootloader.c
> --- a/tools/libxl/libxl_bootloader.c  Tue May 15 11:20:55 2012 +0100
> +++ b/tools/libxl/libxl_bootloader.c  Tue May 15 11:28:12 2012 +0100
> @@ -15,6 +15,7 @@
>  #include "libxl_osdeps.h" /* must come before any other headers */
>  
>  #include <termios.h>
> +#include <string.h>
>  
>  #include "libxl_internal.h"
>  
> @@ -401,6 +402,11 @@ int libxl_run_bootloader(libxl_ctx *ctx,
>      LIBXL__LOG(ctx, LIBXL__LOG_DEBUG, "Config bootloader path: %s",
>                 info->u.pv.bootloader);
>  
> +    if ( !strcmp(info->u.pv.bootloader, "/usr/bin/pygrub") )
> +        LIBXL__LOG(ctx, LIBXL__LOG_WARNING,
> +                   "WARNING: bootloader='/usr/bin/pygrub' "             \
> +                   "is deprecated; use bootloader='pygrub' instead");
> +    
>      bootloader = libxl__abs_path(gc, info->u.pv.bootloader,
>                                   libxl__libexec_path());
>      if ( bootloader == NULL ) {
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxx
> http://lists.xen.org/xen-devel



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