-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Building Terminal from the main branch says it can't find nugget.exe #16379
Comments
Very curious error message there - does it work if you put the Terminal source in a directory that doesn't have an accented character like the The build script probably should be resilient to that sorta scenario, but that's my theory |
You were right, the build script doesn't include accented character to work. I replaced with a simple "e" without accent and it's actually building but is it normal that it takes a lot of time to build? I've been waiting for like 20+ minutes. |
Edit: It finally finished building however the building failed with 5 errors:
|
Yes, unfortunately cppwinrt is very slow to compile. There are ways to speed it up (by sharing the PCH file), but we haven't had time yet to figure out how to make it work.
While I'm not entirely sure why your build failed, the difference between |
How to clean building cache before doing again the build process? Is there an argument to pass to the "Invoke-OpenConsoleBuild" command? |
I solved my build issue as suggested in https://learn.microsoft.com/en-us/nuget/consume-packages/package-restore by doing |
This is almost certainly the root cause: terminal/tools/OpenConsole.psm1 Lines 7 to 14 in 3b5e5cf
PowerShell parses it in the ACP, which explodes here |
I don't really understand what's wrong in the code you showed as I debugged the script by printing different output to ensure everything is fine. My supposition tells me that it's certainly the nuget cause when invoking the Invoke-OpenConsoleBuild() as with running msbuild t:restore, the build appears to be success(packages missing and not retrieved with nuget?): terminal/tools/OpenConsole.psm1 Lines 260 to 266 in 3b5e5cf
|
Update: I actually downloaded the latest nuget executable from the official website and replaced the one used on this project and the error about "Exception thrown" disappeared. Can I have my hands on this issue to fix this bug? Thank you in advance. |
Windows Terminal version
1.18.3181.0
Windows build number
10.0.19045.0
Other Software
No response
Steps to reproduce
PS C:\Users\vinny\Documents\programming\projet_tutoré\terminal> Import-Module .\tools\OpenConsole.psm1 PS C:\Users\vinny\Documents\programming\projet_tutoré\terminal> Set-MsBuildDevEnvironment Dev environment variables set PS C:\Users\vinny\Documents\programming\projet_tutoré\terminal> Invoke-OpenConsoleBuild &: The term 'C:/Users/vinny/Documents/programming/projet_tutor├⌐/terminal\dep\nuget\nuget.exe' is not recognized as a name of a cmdlet, function, script file, or executable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again. &: The term 'C:/Users/vinny/Documents/programming/projet_tutor├⌐/terminal\dep\nuget\nuget.exe' is not recognized as a name of a cmdlet, function, script file, or executable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again. MSBuild version 17.8.3+195e7f5a3 for .NET Framework MSBUILD : error MSB1009: Project file does not exist. Switch: C:/Users/vinny/Documents/programming/projet_tutor├⌐/terminal\OpenConsole.sln
Expected Behavior
No response
Actual Behavior
The build process as described in the developer documentation is not working as it says the nugget.exe package is missing
The text was updated successfully, but these errors were encountered: