[CB-13877] MSBuildTools selection is a mess

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Component/s: cordova-windows
• Assignee: Jan Piotrowski (janpio)
• Reporter: Jan Piotrowski (janpio)
• Created: 14/Feb/18 15:03
• Updated: 23/Aug/18 07:50
• Resolved: 15/Feb/18 15:18

Description

Now with VS2017 out, the code for the detection of the correct MSBuildTools installation is a total mess and works just by accident.

Things to do:

• Give more output that can help understand what is going on Done via PR 255
• Give more ways to understand selection Done via PR 259
• Integrate new restrictions like no Win8.1 support with VS2017-only installations Acceptable limitation for a dead platform I think
• Simplify selection code that is currently spread over 2 files: `MSBuildTools.js` and `build.js` Done via PR 259

Issue Links

• supercedes: CB-13870 `cordova requirements` is broken since change to default=UAP and with modern tooling
• links to: GitHub Pull Request #255
• links to: GitHub Pull Request #257
• links to: GitHub Pull Request #259
• links to: GitHub Pull Request #260