<div dir="ltr"><div><div><div>The patch does not regress the mingw-based build (basically that used in <a href="http://cairographics.org/end_to_end_build_for_win32/">http://cairographics.org/end_to_end_build_for_win32/</a> )<br></div>I did not manage to get the build working on the cmd.exe shell. Do you have any instructions (or possibly indications about what is the correct environment) for that?<br></div><div><br>AFAICT the change still relies on a Unix-like make command.<br></div>A bigger (but possibly even more interesting) change would be an NMAKE-based build, so that the command-line tools available in Visual Studio are sufficient for building the whole project.<br><br></div>Andrea<br><div><div><div><div><br>On Mon, Jun 1, 2015 at 8:44 AM, Andrea Canciani <span dir="ltr"><<a href="mailto:ranma42@gmail.com" target="_blank">ranma42@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><p dir="ltr">The ping was good and the changes make sense. Currently I am replying from the phone and I will not have access to my computer until the 3rd, but I am flagging this mail so that I will test and review the patch asap.</p>
<p dir="ltr">Sorry for the delay<span class=""><font color="#888888"><br>
Andrea</font></span></p><div class=""><div class="h5">
<div class="gmail_quote">On Jun 1, 2015 8:09 AM, "Siarhei Siamashka" <<a href="mailto:siarhei.siamashka@gmail.com" target="_blank">siarhei.siamashka@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Wed, 22 Apr 2015 04:01:31 +0200<br>
Simon Richter <<a href="mailto:Simon.Richter@hogyros.de" target="_blank">Simon.Richter@hogyros.de</a>> wrote:<br>
<br>
> This finds out whether the standard shell for make is cmd.exe, and adjusts<br>
> the build process accordingly.<br>
> ---<br>
>  Makefile.win32.common | 14 ++++++++++++--<br>
>  1 file changed, 12 insertions(+), 2 deletions(-)<br>
<br>
Thanks for the patch.<br>
<br>
But it is a bit difficult for non-Windows folks to see if<br>
it makes any sense or not. So it would be best if some other<br>
Windows user could review and confirm that it works.<br>
<br>
I have added Andrea Canciani (the author of the win32 makefiles)<br>
to CC.<br>
<br>
> diff --git a/Makefile.win32.common b/Makefile.win32.common<br>
> index 777f94c..335886f 100644<br>
> --- a/Makefile.win32.common<br>
> +++ b/Makefile.win32.common<br>
> @@ -1,5 +1,15 @@<br>
>  LIBRARY = pixman-1<br>
><br>
> +ifeq ($(shell echo ""),)<br>
> +# POSIX style shell<br>
> +mkdir_p = mkdir -p $1<br>
> +rm = $(RM) $1<br>
> +else<br>
> +# DOS/Windows style shell<br>
> +mkdir_p = if not exist $(subst /,\,$1) md $(subst /,\,$1)<br>
> +rm = del $(subst /,\,$1)<br>
> +endif<br>
> +<br>
>  CC = cl<br>
>  LD = link<br>
>  AR = lib<br>
> @@ -47,10 +57,10 @@ endif<br>
><br>
><br>
>  $(CFG_VAR)/%.obj: %.c $(libpixman_headers)<br>
> -     @mkdir -p $(CFG_VAR)<br>
> +     @$(call mkdir_p,$(@D))<br>
>       @$(CC) -c $(PIXMAN_CFLAGS) -Fo"$@" $<<br>
><br>
>  clean: inform<br>
> -     @$(RM) $(CFG_VAR)/*.{exe,ilk,lib,obj,pdb} $(BUILT_SOURCES) || exit 0<br>
> +     @-$(call rm,$(CFG_VAR)/*.exe $(CFG_VAR)/*.ilk $(CFG_VAR)/*.lib $(CFG_VAR)/*.obj $(CFG_VAR)/*.pdb} $(BUILT_SOURCES))<br>
><br>
>  .PHONY: inform clean<br>
<br>
<br>
<br>
--<br>
Best regards,<br>
Siarhei Siamashka<br>
</blockquote></div>
</div></div></blockquote></div><br></div></div></div></div></div></div>