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

[ Thread Index | Date Index | More lists.liballeg.org/allegro-developers Archives ]


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




Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/