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

Side by Side Diff: tools/gn/settings.cc

Issue 2940873002: Implement tracking of BUILD.gn files used to define target, toolchain or (Closed)
Patch Set: Fix compilation after rebase. Created 3 years, 5 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 unified diff | Download patch
« no previous file with comments | « tools/gn/scope_unittest.cc ('k') | tools/gn/setup.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "tools/gn/settings.h" 5 #include "tools/gn/settings.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "build/build_config.h" 8 #include "build/build_config.h"
9 #include "tools/gn/filesystem_utils.h" 9 #include "tools/gn/filesystem_utils.h"
10 10
11 Settings::Settings(const BuildSettings* build_settings, 11 Settings::Settings(const BuildSettings* build_settings,
12 const std::string& output_subdir_name) 12 const std::string& output_subdir_name)
13 : build_settings_(build_settings), 13 : build_settings_(build_settings),
14 import_manager_(), 14 import_manager_(),
15 base_config_(this), 15 base_config_(this, {}),
16 greedy_target_generation_(false) { 16 greedy_target_generation_(false) {
17 if (output_subdir_name.empty()) { 17 if (output_subdir_name.empty()) {
18 toolchain_output_dir_ = build_settings->build_dir(); 18 toolchain_output_dir_ = build_settings->build_dir();
19 } else { 19 } else {
20 // We guarantee this ends in a slash. 20 // We guarantee this ends in a slash.
21 DCHECK(output_subdir_name[output_subdir_name.size() - 1] == '/'); 21 DCHECK(output_subdir_name[output_subdir_name.size() - 1] == '/');
22 toolchain_output_subdir_.value().append(output_subdir_name); 22 toolchain_output_subdir_.value().append(output_subdir_name);
23 23
24 DCHECK(!build_settings->build_dir().is_null()); 24 DCHECK(!build_settings->build_dir().is_null());
25 toolchain_output_dir_ = SourceDir(build_settings->build_dir().value() + 25 toolchain_output_dir_ = SourceDir(build_settings->build_dir().value() +
26 toolchain_output_subdir_.value()); 26 toolchain_output_subdir_.value());
27 } 27 }
28 // The output dir will be null in some tests and when invoked to parsed 28 // The output dir will be null in some tests and when invoked to parsed
29 // one-off data without doing generation. 29 // one-off data without doing generation.
30 if (!toolchain_output_dir_.is_null()) 30 if (!toolchain_output_dir_.is_null())
31 toolchain_gen_dir_ = SourceDir(toolchain_output_dir_.value() + "gen/"); 31 toolchain_gen_dir_ = SourceDir(toolchain_output_dir_.value() + "gen/");
32 } 32 }
33 33
34 Settings::~Settings() { 34 Settings::~Settings() {
35 } 35 }
OLDNEW
« no previous file with comments | « tools/gn/scope_unittest.cc ('k') | tools/gn/setup.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698