Re: [PATCH virt-viewer] Ensure windres & icotool are present on Win32 builds

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

 



ack

----- Mensaje original -----
> From: "Daniel P. Berrange" <berrange@xxxxxxxxxx>
> 
> Builds are failing with an obscure error message
> 
> make[3]: Entering directory
> `/var/lib/builder/source-root/virt-viewer/build/icons'
>   GEN    virt-viewer.ico
> /bin/sh: -c: command not found
> make[3]: *** [virt-viewer.ico] Error 127
> 
> This is because configure.ac does not enforce that icotool
> is present on Win32.
> 
> * configure.ac: Mandate windres & icotool on Win32
> ---
>  configure.ac |    7 +++++++
>  1 files changed, 7 insertions(+), 0 deletions(-)
> 
> diff --git a/configure.ac b/configure.ac
> index d35e887..c8e7429 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -41,6 +41,13 @@ AC_PROG_LIBTOOL
>  AS_IF([test "x$os_win32" = "xyes"], [
>       AC_CHECK_TOOL(WINDRES, [windres])
>       AC_CHECK_PROGS(ICOTOOL, [icotool])
> +
> +     if test -z "$WINDRES" ; then
> +       AC_MSG_ERROR("windres is required to compile virt-viewer on
> this platform")
> +     fi
> +     if test -z "$ICOTOOL" ; then
> +       AC_MSG_ERROR("icotool is required to compile virt-viewer on
> this platform")
> +     fi
>  ])
>  
>  AC_CONFIG_LIBOBJ_DIR([src])
> --
> 1.7.7.6
> 
> _______________________________________________
> virt-tools-list mailing list
> virt-tools-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/virt-tools-list
> 


[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux