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

Fix the typo for building USD on Windows. #1120

Conversation

HamedSabri-adsk
Copy link
Contributor

@HamedSabri-adsk HamedSabri-adsk commented Feb 22, 2020

Visual Studio is a 32 bit application, however certain components (e.g. MSBuild, compilers, etc... ) are 64-bit if available.

https://developercommunity.visualstudio.com/content/problem/517541/vs-2019-downloaded-x64-version-installed-x86.html

Visual Studio is a 32 bit application, however certain components (e.g. MSBuild, compilers, etc... ) are 64-bit if available.

https://developercommunity.visualstudio.com/content/problem/517541/vs-2019-downloaded-x64-version-installed-x86.html
@@ -142,7 +142,7 @@ then build and install USD into ```/opt/local/USD```.
Launch the "Developer Command Prompt" for your version of Visual Studio and
run the script in the opened shell. Make sure to use the 64-bit (x64) command
prompt and not the 32-bit (x86) command prompt. (Note if you're trying to
build with Visual Studio 2017, use the "x86 Native Tools Command Prompt for VS
build with Visual Studio 2017, use the "x64 Native Tools Command Prompt for VS
2017").
Copy link
Contributor Author

@HamedSabri-adsk HamedSabri-adsk Feb 22, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think below sentence is still redundant

Note if you're trying to build with Visual Studio 2017, use the "x64 Native Tools Command Prompt for VS
2017") 

since prior sentence provides this information.

@jilliene
Copy link

Filed as internal issue #USD-5878

@sunyab
Copy link
Contributor

sunyab commented Feb 24, 2020

Thanks for catching this @HamedSabri-adsk! I've merged your PR internally, but I agree that the instructions are a bit redundant so I'll clean that up as a follow-up.

@pixar-oss pixar-oss merged commit 0c5cc6f into PixarAnimationStudios:dev Feb 25, 2020
@HamedSabri-adsk
Copy link
Contributor Author

@sunyab Great, thank you!

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.

4 participants