diff options
author | dannark <dannark@google.com> | 2018-06-05 16:06:05 -0700 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-06-05 16:08:12 -0700 |
commit | 043c1274bff1a3c0f1c141aabf2c1089c8501b6f (patch) | |
tree | a7d53538beea0a544e89691f8faf52222550c95c /src/main/java/com/google/devtools/build/lib/packages | |
parent | b796e5094aa3e35fa3e42eede13f174ee2412e6e (diff) |
Disable `repo_mapping` by default via a flag.
RELNOTES: None
PiperOrigin-RevId: 199374368
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/packages')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/packages/WorkspaceFactory.java | 33 |
1 files changed, 17 insertions, 16 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/packages/WorkspaceFactory.java b/src/main/java/com/google/devtools/build/lib/packages/WorkspaceFactory.java index 132c511c04..26b84c669e 100644 --- a/src/main/java/com/google/devtools/build/lib/packages/WorkspaceFactory.java +++ b/src/main/java/com/google/devtools/build/lib/packages/WorkspaceFactory.java @@ -490,22 +490,23 @@ public class WorkspaceFactory { + kwargs.get("name") + "')"); } - if (kwargs.containsKey("repo_mapping")) { - if (!(kwargs.get("repo_mapping") instanceof Map)) { - throw new EvalException( - ast.getLocation(), - "Invalid value for 'repo_mapping': '" + kwargs.get("repo_mapping") - + "'. Value must be a map." - ); - } - @SuppressWarnings("unchecked") - Map<String, String> map = (Map<String, String>) kwargs.get("repo_mapping"); - String externalRepoName = (String) kwargs.get("name"); - for (Map.Entry<String, String> e : map.entrySet()) { - builder.addRepositoryMappingEntry( - RepositoryName.createFromValidStrippedName(externalRepoName), - RepositoryName.create((String) e.getKey()), - RepositoryName.create((String) e.getValue())); + if (env.getSemantics().experimentalEnableRepoMapping()) { + if (kwargs.containsKey("repo_mapping")) { + if (!(kwargs.get("repo_mapping") instanceof Map)) { + throw new EvalException( + ast.getLocation(), + "Invalid value for 'repo_mapping': '" + kwargs.get("repo_mapping") + + "'. Value must be a map."); + } + @SuppressWarnings("unchecked") + Map<String, String> map = (Map<String, String>) kwargs.get("repo_mapping"); + String externalRepoName = (String) kwargs.get("name"); + for (Map.Entry<String, String> e : map.entrySet()) { + builder.addRepositoryMappingEntry( + RepositoryName.createFromValidStrippedName(externalRepoName), + RepositoryName.create((String) e.getKey()), + RepositoryName.create((String) e.getValue())); + } } } RuleClass ruleClass = ruleFactory.getRuleClass(ruleClassName); |