Re: [AD] Build problems with MSVC

[Eric Botcazou <ebotcazou@xxxxxxxxxx>]

> Build documentation is not explicit about CVS, so I failed to make
> depend and fixdll.bat the lot.

That needs to be corrected.  But I think fixdll.sh should be used instead of 
fixdll.bat under Cygwin.

> Also, current version requires g++ to build the dependencies.

Because of the new C++ test? I overlooked that. Does it work to use

$(GCC) -x c $(DEPEND_PARAMS) ....

at the end of makefile.vc instead?

> Cygwin users might need to chmod 775 the bat file to execute it.

Is that needed for fixdll.sh too?

> Order of mkdir/cp during install is wrong, so it can fail if there's no
> include directory:
>
> [ghankiewicz@xxxxxxxxxx] [~/project/allegro/allegro]$ make install
> DEBUGMODE=1 cp include/allegro.h /include
> mkdir /include/allegro
> mkdir: cannot create directory `/include/allegro': No such file or
> directory make: *** [/include/allegro] Error 1
>
> Some missing checks?

(1) MSVCDIR is not set.
(2) A few missing dependencies in makefile.vc. I'm going to post a patch to 
factor out all the install dependencies in makefile.all.

> [ghankiewicz@xxxxxxxxxx] [~/project/allegro/allegro]$ make DEBUGMODE=1
> Your MSVCDIR environment variable is not set!
> See the docs/build/msvc.txt file!
> [ghankiewicz@xxxxxxxxxx] [~/project/allegro/allegro]$ make DEBUGMODE=1
> install The debugging MSVC library has been installed.
>
> I must be lucky to have built the debug version without MSVCDIR!

It is only used when installing.

-- 
Eric Botcazou