Re: [AD] MinGW32 make install

[hstokset@xxxxxxxxxx]

"Eric Botcazou" <ebotcazou@xxxxxxxxxx> wrote:

>> With the attached patch, make install for MinGW32 works better if
>> 'include' but not 'include/allegro' exist (it doesn't try to create an
>> existing directory). I don't know why it only creates 'include' if
>> UNIX_TOOLS is defined, but I kept it that way. OK?
>
>[A cygwin-ism (ask Henrik for more detailed informations)]

if i remember your patch correctly it replaced:

$(MINGDIR_U)/include/allegro:
	mkdir $(MINGDIR_U)/include
	mkdir $(MINGDIR_U)/include/allegro 

with:

$(MINGDIR_U)/include/allegro : $(MINGDIR_U)/include
	mkdir $(MINGDIR_U)/include/allegro

$(MINGDIR_U)/include:
	mkdir $(MINGDIR_U)/include

if UNIX_TOOLS is defined which is OK. when UNIX_TOOLS is not defined the
makefile currently does:

$(MINGDIR_U)/include/allegro:
	md $(MINGDIR_D)\include\allegro 

which should for correctness sake be replaced with:

$(MINGDIR_U)/include:
	md $(MINGDIR_D)\include

$(MINGDIR_U)/include/allegro : $(MINGDIR_U)/include
	md $(MINGDIR_D)\include\allegro 

HTH. i didn't have time to test with cygwin as i'm busy with job and truck-license
among other more programming related things. :o)

-henrik