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

Unified Diff: build/config/win/visual_studio_version.gni

Issue 2770193005: build with MSVS will only need VS console environment set up
Patch Set: build with MSVS will need VS console environment set up Created 3 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 | « AUTHORS ('k') | build/vs_toolchain.py » ('j') | build/vs_toolchain.py » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/config/win/visual_studio_version.gni
diff --git a/build/config/win/visual_studio_version.gni b/build/config/win/visual_studio_version.gni
index 982fbe8d3f0d0294e28e206638cfa3ee3a902745..11eed00e997e7d1410c93fa45ca0ceca83b8aa2d 100644
--- a/build/config/win/visual_studio_version.gni
+++ b/build/config/win/visual_studio_version.gni
@@ -23,6 +23,10 @@ declare_args() {
}
if (visual_studio_path == "") {
+ test1 = getenv("VSINSTALLDIR") != ""
+ assert(test1, "Please run from within a VS console (with ENV variables)")
+ test2 = getenv("VisualStudioVersion") != ""
+ assert(test1, "Please run from within a VS console (with ENV variables)")
toolchain_data =
exec_script("../../vs_toolchain.py", [ "get_toolchain_dir" ], "scope")
visual_studio_path = toolchain_data.vs_path
« no previous file with comments | « AUTHORS ('k') | build/vs_toolchain.py » ('j') | build/vs_toolchain.py » ('J')

Powered by Google App Engine
This is Rietveld 408576698