Create msbuild only when using MSVC

[mati865]

Summary

I could be wrong but it seems to have no use for MinGW.

Related diff: https://reviews.llvm.org/D29772

[rnk]

Seems reasonable. You might want to investigate if things go wrong if you try to package an NSIS installer of clang that targets mingw. Right now it will probably attempt to install these files into ProgFiles/MSBuild, and that will likely fail with this change.

[mati865]

That's valid concern, I'll try tomorrow to find out how it works.

P.S.
The only reason of this patch is tools/msbuild doesn't look nice with Filesystem Hierarchy Standard adopted by MSYS2 and wastes some space (copy instead of symlink).

[mati865]

I suppose those 4 lines were your concern: https://github.com/llvm-mirror/llvm/blob/3ca564103abdd0c70f28f270873a2e535c245a6f/CMakeLists.txt#L231

I ran make package in the build dir patched with this diff and https://reviews.llvm.org/D29772 and it returned no errors.
Install and uninstall of created package was also successful.

[mati865]

Ping

[mati865]

Thank you @rnk
Could you check https://reviews.llvm.org/D29772 ? Looks like there was problem with commit

llvm/trunk/tools/msbuild/CMakeLists.txt

-if (WIN32)
+if (MSVC)
   # CPack will install a registry key in this format that we wish to reference.
   set(REG_KEY "${CPACK_PACKAGE_INSTALL_REGISTRY_KEY}")
   set(LIB_PATH_VERSION "${LLVM_VERSION_MAJOR}.${LLVM_VERSION_MINOR}")
 
   foreach (platform "Win32" "x64")
     set(prop_file_in "Microsoft.Cpp.Win32.llvm.props.in")
     set(prop_file_v100 "Microsoft.Cpp.${platform}.LLVM-vs2010.props")
     set(prop_file_v110 "Microsoft.Cpp.${platform}.LLVM-vs2012.props")