Page MenuHomePhabricator

Create msbuild only when using MSVC

Authored by mati865 on Feb 14 2017, 11:29 AM.



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

Related diff:

Diff Detail


Event Timeline

mati865 created this revision.Feb 14 2017, 11:29 AM
rnk accepted this revision.Feb 14 2017, 11:36 AM
rnk added a subscriber: 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.

This revision is now accepted and ready to land.Feb 14 2017, 11:36 AM

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

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).

I suppose those 4 lines were your concern:

I ran make package in the build dir patched with this diff and and it returned no errors.
Install and uninstall of created package was also successful.

This revision was automatically updated to reflect the committed changes.

Thank you @rnk
Could you check ? Looks like there was problem with commit