Re: [AD] Patch for makefile issue when compiling MSVC with Cygwin

[Michal Molhanec <michal@xxxxxxxxxx>]

Evert Glebbeek napsal(a):
> Now for some comments. Warning: pedantic language checking below!

Thanks! As the update was already commited can you please update it 
directly in SVN?

> > +   If you are using MSVC 7 you can specify "<tt>msvc7</tt>" instead of 
> "<tt>msvc</tt>" to enable
> > +   better optimizations. Likewise, you can specify "<tt>icl</tt>" to use 
> the Intel
> > +   commandline compiler. If you are using MSVC 8 you should specify 
> "<tt>msvc8</tt>"
> > +   otherwise you will get a lot of deprecation warnings. For MSVC 6 and 
> older
> > +   you can use also "<tt>msvc6</tt>".
>
> I would phrase this differently, along the following lines:
> You can also configure for a specific version of MSVC to enable specific 
> optimisations or switches. For example, you  can use <tt>msvc7</tt> for 
> MSVC 7 or <tt>msvc8</tt> for MSVC 8. For MSVC8, it is recommended that you 
> use msvc8, otherwise the compiler will generate a lot of deprecation 
> warnings.
> Likewise, you can pass "<tt>icl</tt>" to use the Intel commandline 
> compiler.
>
> I'm also not sure about the "MSVC 6 and older" part. Not that anyone uses 
> "older" any more, but best not to mention it at all I think.

IMHO is MSVC 6 still the best compiler for allegro DLL, I'm using it for 
my devpaks. Note that "fix msvc6" is faster than "fix msvc" (doesn't try 
to convert path) and it doesn't display warning message.

I would personally not mension "msvc" at all, as it's in fact deprecated 
and replace those two paragraphs with simple table, e.g.:

====
To configure Allegro for your compiler please run in the command prompt:
<textblock>
      fix.bat msvc6  - for MSVC 6 and older
      fix.bat msvc7  - for MSVC 7 (.NET) and 7.1 (.NET 2003)
      fix.bat msvc8  - for MSVC 8 (.NET 2005)
      fix.bat icl    - for Intel compiler
<endblock>
====

 >> run from command prompt
> > +   "<tt>misc\embedman.bat all</tt>" ("<tt>./misc/embedman.bat all</tt>" 
> under Cygwin).
>
> run the command "<tt>misc\embedman.bat all</tt>" from the command prompt 
> (Cygwin users run "<tt>misc/embedman.bat all</tt>")

this must be <tt>./misc/embedman.bat all</tt>, you cannot strip the ./

--
Regards,
    Michal

ICQ# 175762750