Re: [ft-devel] Adding new platform to installer

[Werner LEMBERG]

Sorry for the late reply.

> The error appears to be coming from the substitution of \ for / in
> these lines:
> 
> ifneq ($(findstring $(PLATFORM),dos win32 win16 os2),)
>    OPEN_MODULE   := @echo$(space)
>    CLOSE_MODULE  :=  >> $(subst /,\,$(FTMODULE_H))
>    REMOVE_MODULE := @-$(DELETE) $(subst /,\,$(FTMODULE_H))
> else
>    OPEN_MODULE   := @echo "
>    CLOSE_MODULE  := " >> $(FTMODULE_H)
>    REMOVE_MODULE := @-$(DELETE) $(FTMODULE_H)
> endif
> 
> in builds/modules.mk. (The make that I am using uses Unix
> conventions, so it wants / in paths.)  If I declare that my platform
> is not one in the list, then it doesn't do the substitution of \ for
> / and things complete.  So builds are working for me now with my
> custom .mk file.
> 
> Just a suggestion: since there's an explicit declaration of the SEP
> character, shouldn't these lines make use of that, rather than
> assuming it is \ on win32, etc?

Yes.  I've done that in all affected files.  Thanks for the report,
and please test!


    Werner


PS: Would you like to contribute your platform to the FreeType build
    system?