Re: [Xen-devel] [PATCH] include: fix build without C++ compiler installed

[Julien Grall <julien.grall@xxxxxxx>]

Hi Jan,

On 12/05/17 07:52, Jan Beulich wrote:
> The rule for headers++.chk wants to move headers++.chk.new to the
> designated target, which means we have to create that file in the first
> place.
>
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx>
Release-acked-by: Julien Grall <julien.grall@xxxxxxx>

Cheers,

> --- a/xen/include/Makefile
> +++ b/xen/include/Makefile
> @@ -120,7 +120,10 @@ headers99.chk: $(PUBLIC_C99_HEADERS) Mak
>
>  headers++.chk: $(PUBLIC_HEADERS) Makefile
>         rm -f $@.new
> -       $(CXX) -v >/dev/null 2>&1 || exit 0;                                  \
> +       if ! $(CXX) -v >/dev/null 2>&1; then                                  \
> +           touch $@.new;                                                     \
> +           exit 0;                                                           \
> +       fi;                                                                   \
>         $(foreach i, $(filter %.h,$^),                                        \
>             echo "#include "\"$(i)\"                                          \
>             | $(CXX) -x c++ -std=gnu++98 -Wall -Werror -D__XEN_TOOLS__        \
--
Julien Grall

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