Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6)

Unified Diff: site/user/quick/windows.md

Issue 1785743003: update site dox - windows prefer msvs-ninja and add vulkan (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: tiny Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | site/user/special/vulkan.md » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: site/user/quick/windows.md
diff --git a/site/user/quick/windows.md b/site/user/quick/windows.md
index de014a4e6a0be9e02b0fff3d02feaf13bd57831c..d21f9d2dcb9c5d1a7ad9e8af180d968bfdf65748 100644
--- a/site/user/quick/windows.md
+++ b/site/user/quick/windows.md
@@ -57,9 +57,14 @@ Three Gyp generators are used on Windows:
* `msvs-ninja` - Develop from a fully-integrated Visual Studio.
Gyp generates Visual-Studio-compatible project files that still
- ultimately build using ninja
-
-* `msvs` - Use Visual Studio's own (slower) build system
+ ultimately build using ninja. This is the preferred and better maintained
+ way of using Visual Studio.
+
+* `msvs` - Use Visual Studio's own (slower) build system. This is not as well
+ maintained as msvs-ninja. The msvs and ninja output can clobber each other
+ so building from both the command line and Visual Studio can produce bad
+ states that can be fixed by deleting the out directory and rerunning
+ gyp_skia.
To choose which ones to use, set the `GYP_GENERATORS` environment
variable to a comma-delimited list of generators before running
« no previous file with comments | « no previous file | site/user/special/vulkan.md » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698