Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes failing to find msbuild when VS 2015 not installed and using 14393 SDK #1163

Merged
merged 2 commits into from Oct 27, 2017

Conversation

StephenHodgson
Copy link
Contributor

@StephenHodgson StephenHodgson commented Oct 14, 2017

Fixes #1146 Wouldn't build for SDKs that use ms build version 14 and VS 2015 was not installed.

@StephenHodgson StephenHodgson changed the title Build Fixes for ms build version 14< when VS 2015 not installed Fixes failing to find msbuild when VS 2015 not installed and using 14393 SDK Oct 14, 2017
@StephenHodgson
Copy link
Contributor Author

@reillydonovan, @brean let me know if this fixes it for you guys.

@jessemcculloch
Copy link

@brean @reillydonovan - Can you two test and see if this resolves the issues, so we can merge the fix?

Thanks!

@reillydonovan
Copy link

I can confirm that this works when publishing via the MRT build window to APPX and deployed to the hololens via the device manager.

Tested using:

Fall Creators Update
Unity 2017.2.0f3
VS 2017 v 15.4.0 with all recommended supported dependencies
Windows 10 SDK 10.0.16299

@StephenHodgson
Copy link
Contributor Author

StephenHodgson commented Oct 22, 2017

The fix is for publishing to an older SDK without vs 2015.

So the real test is building and deploying while targeting the 14393 SDK or lower

@StephenHodgson StephenHodgson merged commit bdc1197 into microsoft:master Oct 27, 2017
@StephenHodgson StephenHodgson deleted the MRTK-BuildFix branch October 27, 2017 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants