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

Unified Diff: tools/gn/loader.cc

Issue 115643002: Hook up GN official build and branding. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years 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
« build/config/linux/sysroot.gni ('K') | « build/gyp_chromium ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/gn/loader.cc
diff --git a/tools/gn/loader.cc b/tools/gn/loader.cc
index 7e3ca3d0f3c78afbea48865aefb637d3d92cd11c..e0a0517882ff96b031ef12fe5df558c83438dc27 100644
--- a/tools/gn/loader.cc
+++ b/tools/gn/loader.cc
@@ -293,6 +293,9 @@ void LoaderImpl::BackgroundLoadBuildConfig(
trace.Done();
+ if (err.has_error())
+ g_scheduler->FailWithError(err);
+
base_config->ClearProcessingBuildConfig();
if (settings->is_default()) {
// The default toolchain must have been set in the default build config
« build/config/linux/sysroot.gni ('K') | « build/gyp_chromium ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698