microsoft / GalaxyExplorer

Galaxy Explorer is the application Microsoft is building as part of the Share Your Idea campaign.
MIT License
947 stars 331 forks source link

Build Fails 2017.2.0f3-MRTP3 WIN10 SDK 10.0.16299.0 #97

Closed RapukeMoana closed 7 years ago

RapukeMoana commented 7 years ago

Following error on build. Runs in Unity. Exception: Failed to locate env variables VS140COMNTOOLS or VS120COMNTOOLS. Utility.GetVSVersion () (at C:/buildslave/unity/build/PlatformDependent/WinRT/SharedSources/CSharp/Utility.cs:623) MetroCSharpVisualStudioSolutionCreator.WriteSolutionFile (System.String solutionFileName, UnityEditor.Scripting.ScriptCompilation.ScriptAssembly[] csharpAssemblies) (at C:/buildslave/unity/build/PlatformDependent/MetroPlayer/Extensions/Managed/MetroCSharpVisualStudioSolutionCreator.cs:196) MetroCSharpVisualStudioSolutionCreator.CreateSolutionFileFrom () (at C:/buildslave/unity/build/PlatformDependent/MetroPlayer/Extensions/Managed/MetroCSharpVisualStudioSolutionCreator.cs:724) PostProcessWSA.CreateVisualStudioSolution () (at C:/buildslave/unity/build/PlatformDependent/MetroPlayer/Extensions/Managed/PostProcessWSA.cs:320) PostProcessWinRT.Process () (at C:/buildslave/unity/build/PlatformDependent/WinRT/SharedSources/CSharp/PostProcessWinRT.cs:237) UnityEditor.WSA.BuildPostprocessor.DoPostProcess (BuildPostProcessArgs args) (at C:/buildslave/unity/build/PlatformDependent/MetroPlayer/Extensions/Managed/ExtensionModule.cs:142) UnityEditor.WSA.BuildPostprocessor.PostProcess (BuildPostProcessArgs args) (at C:/buildslave/unity/build/PlatformDependent/MetroPlayer/Extensions/Managed/ExtensionModule.cs:149) UnityEngine.GUIUtility:ProcessEvent(Int32, IntPtr)

timGerken commented 7 years ago

Can you provide a little more detail? It looks like it is the build step from Unity that is failing for you. Do you have Visual Studio 2017 installed? Do you have any other version of VS installed? On which branch are you seeing this? master or MixedReality?

RapukeMoana commented 7 years ago

Yes I have VS 2017 and today upgraded to 15.4.2 from 15.4.1. I have no other VS in fact this is a very new clean Surface Pro. I uninstalled Unity and reinstalled 2017.2.0f3-MRTP3. Windows is 1709 16299.19. Very frustrating but that said I am very happy to see this branch as I have plans to build onto it. I am on Mixed Reality branch.

RapukeMoana commented 7 years ago

What version of all these tools did you use to build this?

timGerken commented 7 years ago

Unity 2017.2.0f3-MRTP3 and VS 2017 15.4.0.

Do you have entries for VS100COMNTOOLS and VS140COMNTOOLS anywhere in your registry?

Look here: HKEY_LOCAL_MACHINE\SYSTEM\ControlSet001\Control\Session Manager\Environment

RapukeMoana commented 7 years ago

It appears not. What should I do. galaxyexplorerissue This is outside my knowledge. Thanks for your help.

jimshalo10 commented 7 years ago

My testing of the new release at If you try to load 2017.2.0f3 MR 3 it will stop as UWP removed 2017.2.0f3 is newest

Try Demo

timGerken commented 7 years ago

@RapukeMoana , I'm not able to reproduce what you are seeing. That said, I have Visual Studio 2010, 2015 and 2017 installed and am currently using Unity 2017.2.0p1-MRTP4. Even if I remove VS100COMNTOOLS and VS140COMNTOOLS from my environment, I don't see the issue you are describing.

A couple more questions: 1) can you re-confirm that you are seeing this error when building from within Unity? If so, what are the settings in your Build Settings window? 2) is this from a clean clone of the repository? git clone http://github.com/microsoft/galaxyexplorer -b MixedReality

ChimeraScorn commented 7 years ago

Does building a new app from Unity work on your machine?

jimshalo10 commented 7 years ago

As instructed loadingg Unity 2017.2.0p1-MRTP4 git clone http://github.com/microsoft/galaxyexplorer -b MixedReality

Built target being PC, everything build in VS2017 fine Just need to know how to send directly to HP WMR headset, never do this before Thanks Oh maybe will help. When starting with Unreal Engine VS 2017 had a problem and Quote from Private Epic board, my workaround answer was on loading a new copy VS2017

WORKAROUND: Until this issue gets officially fixed. This procedure will make the build package work. In File Explorer go to "C:\Program Files (x86)\Microsoft Visual Studio\Shared\14.0\VC" Create new directory .\vcpackages Copy platform.winmd from C:\Program Files (x86)\Microsoft Visual Studio\Shared\14.0\VC\lib\store\references\platform.winmd 4 . Into new directory C:\Program Files (x86)\Microsoft Visual Studio\Shared\14.0\VC\vcpackages

  1. Agree to Administrative rights to write the file to new platform.winmd location
  2. In Unreal File ->Package Project -> Universal Windows Platform -> UWP(x64-64bit) to rebuild package Now successful build log UATHelper: Packaging (UWP (x64-64bit)): Automation.Execute: BUILD SUCCESSFUL UATHelper: Packaging (UWP (x64-64bit)): Program.Main: AutomationTool exiting with ExitCode=0 (Success) Note: if you don't have Universal Windows Platform -> UWP(x64-64bit), you have not correctly built the Unreal 4.15 MICROSOFT_UWP_UNREAL Platform
RapukeMoana commented 7 years ago

Thank you for your help. I had been working from the zip and as soon as I cloned from GitHub it all builds. Very much looking forward to building on GE. This describes our starting point which has been a design class collaboration so far and is now moving into building. https://trello.com/b/uaCT48b5/chronospatial