murisfurder / vim

Automatically exported from code.google.com/p/vim
0 stars 0 forks source link

Patch for /src/Make_mvc.mak #224

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Use _MSC_VER to achieve cl.exe version. 

Original issue reported on code.google.com by nobu.nak...@gmail.com on 13 May 2014 at 4:29

Attachments:

GoogleCodeExporter commented 9 years ago
Hi Nobu.

It's a very good idea to use cl.exe version directly instead of nmake.exe 
version.
(I already use almost same method in my project.)
Unfortunately, your patch doesn't work, so I have updated it.
* I don't think findstr is needed. I think it's enough to use /EP option 
instead of /P.
* _MSC_VER is different from Visual C++ version.
  E.g. _MSC_VER is 1600 for VC10.
* Update some messages and comments.
* Support of batch compilation is depends on nmake version, not cl.exe version.

I tested this patch with VC6, VC8 and VC10.

Original comment by ktakata6...@gmail.com on 13 May 2014 at 1:52

Attachments: