Clang cmake fix for lib naming http://llvm.org/bugs/show_bug.cgi?id=13909

[martell]

Summary

Mingw and mingw64 builds compile to liblibclang.dll

and this diff will change that back to libclang.dll

mingw is like linux in that it prepends the lib word :)

http://llvm.org/bugs/show_bug.cgi?id=13909

This patch is over 2 years old :D
but it was never merged
adding reviewers to see if we can squash this bug

[martell]

Hi chandler,

This is my first ever diff patch pull request.
I hope I have done it correctly.

Looking forward to your input :)

[yaron.keren]

On my mingw 4.9.1 builds (without this patch) I get "libclang.dll" and not "liblibclang.dll".
Do you still get "liblibclang.dll" with current SVN (without this patch) ?

[martell]

In D601#202124, @yaron.keren wrote:

On my mingw 4.9.1 builds (without this patch) I get "libclang.dll" and not "liblibclang.dll".
Do you still get "liblibclang.dll" with current SVN (without this patch) ?

Still not quite sure how to reply to a comment. I'm a phabricator noob :)
I only noticed this patch when I logged in
Ahh yes I just checked you are correct it is now fixed on svn

not sure when but this should probably be closed
http://llvm.org/bugs/show_bug.cgi?id=13909

tools/libclang/CMakeLists.txt

   target_link_libraries(libclang ${LIBRARIES})
   add_dependencies(libclang ${GENERATED_HEADERS} clang-headers)
 
-  if(WIN32)
+  if(WIN32 AND NOT MINGW)
     set_target_properties(libclang
       PROPERTIES
       OUTPUT_NAME "libclang"