diff options
author | 2017-11-22 05:28:10 -0800 | |
---|---|---|
committer | 2017-11-22 05:34:30 -0800 | |
commit | 3275b2555d1c043af0ee0397d1d6a4a3df64c011 (patch) | |
tree | 79c9a86a19515cd74fce34da94e8442b67a326f5 /src/main/java | |
parent | ebdb4ac639f2c7574100cfa1373d760248433969 (diff) |
Improve error message when missing default_toolchain in CROSSTOOL
RELNOTES: None.
PiperOrigin-RevId: 176643685
Diffstat (limited to 'src/main/java')
3 files changed, 14 insertions, 6 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CrosstoolConfigurationIdentifier.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CrosstoolConfigurationIdentifier.java index 02028d10e2..accd29c195 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CrosstoolConfigurationIdentifier.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CrosstoolConfigurationIdentifier.java @@ -14,6 +14,7 @@ package com.google.devtools.build.lib.rules.cpp; +import com.google.common.base.Preconditions; import com.google.devtools.build.lib.analysis.config.BuildConfiguration; import com.google.devtools.build.lib.analysis.config.BuildConfiguration.Options; import com.google.devtools.build.lib.analysis.config.BuildOptions; @@ -39,7 +40,7 @@ public final class CrosstoolConfigurationIdentifier implements CrosstoolConfigur /** Creates a new {@link CrosstoolConfigurationIdentifier} with the given parameters. */ CrosstoolConfigurationIdentifier(String cpu, String compiler, String libc) { - this.cpu = cpu; + this.cpu = Preconditions.checkNotNull(cpu); this.compiler = compiler; this.libc = libc; } diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CrosstoolConfigurationLoader.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CrosstoolConfigurationLoader.java index c712b186fd..aa12a860c3 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CrosstoolConfigurationLoader.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CrosstoolConfigurationLoader.java @@ -368,8 +368,11 @@ public class CrosstoolConfigurationLoader { cpuBuilder.append(" ").append(selector.getCpu()).append(",\n"); } throw new InvalidConfigurationException( - "No toolchain found for cpu '" + desiredCpu - + "'. Valid cpus are: [\n" + cpuBuilder + "]"); + "No default_toolchain found for cpu '" + + desiredCpu + + "'. Valid cpus are: [\n" + + cpuBuilder + + "]"); } checkToolChain(selectedIdentifier, desiredCpu); diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CrosstoolConfigurationOptions.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CrosstoolConfigurationOptions.java index 16a05777a4..46af308d23 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CrosstoolConfigurationOptions.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CrosstoolConfigurationOptions.java @@ -14,16 +14,20 @@ package com.google.devtools.build.lib.rules.cpp; +import javax.annotation.Nullable; + /** * A container object which provides crosstool configuration options to the build. */ public interface CrosstoolConfigurationOptions { /** Returns the CPU associated with this crosstool configuration. */ - public String getCpu(); + String getCpu(); /** Returns the compiler associated with this crosstool configuration. */ - public String getCompiler(); + @Nullable + String getCompiler(); /** Returns the libc version associated with this crosstool configuration. */ - public String getLibc(); + @Nullable + String getLibc(); } |