Starlarkify all starlark build configurations-related work thus far (s/skylark/starlark) #5577 PiperOrigin-RevId: 223357995
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/config/BuildOptions.java b/src/main/java/com/google/devtools/build/lib/analysis/config/BuildOptions.java index 5f322b8..587e3ea 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/config/BuildOptions.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/config/BuildOptions.java
@@ -98,9 +98,9 @@ public BuildOptions createHostOptions() { Builder builder = builder(); for (FragmentOptions options : fragmentOptionsMap.values()) { - builder.add(options.getHost()); + builder.addFragmentOptions(options.getHost()); } - return builder.addSkylarkOptions(skylarkOptionsMap).build(); + return builder.addStarlarkOptions(skylarkOptionsMap).build(); } /** @@ -114,10 +114,10 @@ // TODO(bazel-team): make this non-hacky while not requiring BuildConfiguration access // to BuildOptions. || options.toString().contains("BuildConfiguration$Options")) { - builder.add(options); + builder.addFragmentOptions(options); } } - return builder.addSkylarkOptions(skylarkOptionsMap).build(); + return builder.addStarlarkOptions(skylarkOptionsMap).build(); } /** @@ -128,9 +128,9 @@ Iterable<Class<? extends FragmentOptions>> optionsList, OptionsProvider provider) { Builder builder = builder(); for (Class<? extends FragmentOptions> optionsClass : optionsList) { - builder.add(provider.getOptions(optionsClass)); + builder.addFragmentOptions(provider.getOptions(optionsClass)); } - return builder.addSkylarkOptions(provider.getSkylarkOptions()).build(); + return builder.addStarlarkOptions(provider.getStarlarkOptions()).build(); } /** @@ -146,7 +146,7 @@ ImmutableList.<Class<? extends OptionsBase>>copyOf(optionsList)); parser.parse(args); for (Class<? extends FragmentOptions> optionsClass : optionsList) { - builder.add(parser.getOptions(optionsClass)); + builder.addFragmentOptions(parser.getOptions(optionsClass)); } return builder.build(); } @@ -156,7 +156,7 @@ */ @VisibleForTesting public static BuildOptions of(Map<String, Object> skylarkOptions) { - return builder().addSkylarkOptions(skylarkOptions).build(); + return builder().addStarlarkOptions(skylarkOptions).build(); } /** Returns the actual instance of a FragmentOptions class. */ @@ -221,7 +221,7 @@ return fragmentOptionsMap.values(); } - public ImmutableMap<String, Object> getSkylarkOptions() { + public ImmutableMap<String, Object> getStarlarkOptions() { return skylarkOptionsMap; } @@ -324,26 +324,26 @@ for (FragmentOptions options : fragmentOptionsMap.values()) { FragmentOptions newOptions = optionsDiff.transformOptions(options); if (newOptions != null) { - builder.add(newOptions); + builder.addFragmentOptions(newOptions); } } for (FragmentOptions extraSecondFragment : optionsDiff.extraSecondFragments) { - builder.add(extraSecondFragment); + builder.addFragmentOptions(extraSecondFragment); } Map<String, Object> skylarkOptions = new HashMap<>(); for (Map.Entry<String, Object> buildSettingAndValue : skylarkOptionsMap.entrySet()) { String buildSetting = buildSettingAndValue.getKey(); - if (optionsDiff.extraFirstSkylarkOptions.contains(buildSetting)) { + if (optionsDiff.extraFirstStarlarkOptions.contains(buildSetting)) { continue; - } else if (optionsDiff.differingSkylarkOptions.containsKey(buildSetting)) { - skylarkOptions.put(buildSetting, optionsDiff.differingSkylarkOptions.get(buildSetting)); + } else if (optionsDiff.differingStarlarkOptions.containsKey(buildSetting)) { + skylarkOptions.put(buildSetting, optionsDiff.differingStarlarkOptions.get(buildSetting)); } else { skylarkOptions.put(buildSetting, skylarkOptionsMap.get(buildSetting)); } } - skylarkOptions.putAll(optionsDiff.extraSecondSkylarkOptions); - builder.addSkylarkOptions(skylarkOptions); + skylarkOptions.putAll(optionsDiff.extraSecondStarlarkOptions); + builder.addStarlarkOptions(skylarkOptions); return builder.build(); } @@ -358,28 +358,28 @@ * Adds a new FragmentOptions instance to the builder. Overrides previous instances of the exact * same subclass of FragmentOptions. */ - public <T extends FragmentOptions> Builder add(T options) { - builderMap.put(options.getClass(), options); + public <T extends FragmentOptions> Builder addFragmentOptions(T options) { + fragmentOptions.put(options.getClass(), options); return this; } - Builder addSkylarkOptions(Map<String, Object> options) { - builderSkylarkOptionsMap.putAll(options); + Builder addStarlarkOptions(Map<String, Object> options) { + starlarkOptions.putAll(options); return this; } public BuildOptions build() { return new BuildOptions( - ImmutableSortedMap.copyOf(builderMap, lexicalFragmentOptionsComparator), - ImmutableSortedMap.copyOf(builderSkylarkOptionsMap, skylarkOptionsComparator)); + ImmutableSortedMap.copyOf(fragmentOptions, lexicalFragmentOptionsComparator), + ImmutableSortedMap.copyOf(starlarkOptions, skylarkOptionsComparator)); } - private Map<Class<? extends FragmentOptions>, FragmentOptions> builderMap; - private Map<String, Object> builderSkylarkOptionsMap; + private final Map<Class<? extends FragmentOptions>, FragmentOptions> fragmentOptions; + private final Map<String, Object> starlarkOptions; private Builder() { - builderMap = new HashMap<>(); - builderSkylarkOptionsMap = new HashMap<>(); + fragmentOptions = new HashMap<>(); + starlarkOptions = new HashMap<>(); } } @@ -401,7 +401,7 @@ */ public static OptionsDiff diff( OptionsDiff diff, @Nullable BuildOptions first, @Nullable BuildOptions second) { - if (diff.hasSkylarkOptions) { + if (diff.hasStarlarkOptions) { throw new IllegalStateException( "OptionsDiff cannot handle multiple 'second' BuildOptions with skylark options " + "and is trying to diff against a second BuildOptions with skylark options."); @@ -450,16 +450,16 @@ } // Compare skylark options for the two classes - Map<String, Object> skylarkFirst = first.getSkylarkOptions(); - Map<String, Object> skylarkSecond = second.getSkylarkOptions(); - diff.setHasSkylarkOptions(!skylarkFirst.isEmpty() || !skylarkSecond.isEmpty()); + Map<String, Object> skylarkFirst = first.getStarlarkOptions(); + Map<String, Object> skylarkSecond = second.getStarlarkOptions(); + diff.setHasStarlarkOptions(!skylarkFirst.isEmpty() || !skylarkSecond.isEmpty()); for (String buildSetting : Sets.union(skylarkFirst.keySet(), skylarkSecond.keySet())) { if (skylarkFirst.get(buildSetting) == null) { - diff.addExtraSecondSkylarkOption(buildSetting, skylarkSecond.get(buildSetting)); + diff.addExtraSecondStarlarkOption(buildSetting, skylarkSecond.get(buildSetting)); } else if (skylarkSecond.get(buildSetting) == null) { - diff.addExtraFirstSkylarkOption(buildSetting); + diff.addExtraFirstStarlarkOption(buildSetting); } else if (!skylarkFirst.get(buildSetting).equals(skylarkSecond.get(buildSetting))) { - diff.putSkylarkDiff( + diff.putStarlarkDiff( buildSetting, skylarkFirst.get(buildSetting), skylarkSecond.get(buildSetting)); } } @@ -509,19 +509,16 @@ first.maybeInitializeFingerprintAndHashCode(); return new OptionsDiffForReconstruction( differingOptions, - diff.extraFirstFragments - .stream() + diff.extraFirstFragments.stream() .sorted(lexicalFragmentOptionsComparator) .collect(ImmutableSet.toImmutableSet()), - diff.extraSecondFragments - .stream() - .sorted(Comparator.comparing(o -> o.getClass().getName())) - .collect(ImmutableList.toImmutableList()), + ImmutableList.sortedCopyOf( + Comparator.comparing(o -> o.getClass().getName()), diff.extraSecondFragments), first.fingerprint, second.computeChecksum(), diff.skylarkSecond, - diff.extraSkylarkOptionsFirst, - diff.extraSkylarkOptionsSecond); + diff.extraStarlarkOptionsFirst, + diff.extraStarlarkOptionsSecond); } /** @@ -547,10 +544,10 @@ // skylark options anywhere yet so add that feature when necessary. private final Map<String, Object> skylarkSecond = new LinkedHashMap<>(); - private final List<String> extraSkylarkOptionsFirst = new ArrayList<>(); - private final Map<String, Object> extraSkylarkOptionsSecond = new HashMap<>(); + private final List<String> extraStarlarkOptionsFirst = new ArrayList<>(); + private final Map<String, Object> extraStarlarkOptionsSecond = new HashMap<>(); - private boolean hasSkylarkOptions = false; + private boolean hasStarlarkOptions = false; @VisibleForTesting Set<Class<? extends FragmentOptions>> getExtraFirstFragmentClassesForTesting() { @@ -588,41 +585,41 @@ extraSecondFragments.add(options); } - private void putSkylarkDiff(String buildSetting, Object firstValue, Object secondValue) { + private void putStarlarkDiff(String buildSetting, Object firstValue, Object secondValue) { skylarkFirst.put(buildSetting, firstValue); skylarkSecond.put(buildSetting, secondValue); } - private void addExtraFirstSkylarkOption(String buildSetting) { - extraSkylarkOptionsFirst.add(buildSetting); + private void addExtraFirstStarlarkOption(String buildSetting) { + extraStarlarkOptionsFirst.add(buildSetting); } - private void addExtraSecondSkylarkOption(String buildSetting, Object value) { - extraSkylarkOptionsSecond.put(buildSetting, value); + private void addExtraSecondStarlarkOption(String buildSetting, Object value) { + extraStarlarkOptionsSecond.put(buildSetting, value); } - private void setHasSkylarkOptions(boolean hasSkylarkOptions) { - this.hasSkylarkOptions = hasSkylarkOptions; + private void setHasStarlarkOptions(boolean hasStarlarkOptions) { + this.hasStarlarkOptions = hasStarlarkOptions; } @VisibleForTesting - Map<String, Object> getSkylarkFirstForTesting() { + Map<String, Object> getStarlarkFirstForTesting() { return skylarkFirst; } @VisibleForTesting - Map<String, Object> getSkylarkSecondForTesting() { + Map<String, Object> getStarlarkSecondForTesting() { return skylarkSecond; } @VisibleForTesting - List<String> getExtraSkylarkOptionsFirstForTesting() { - return extraSkylarkOptionsFirst; + List<String> getExtraStarlarkOptionsFirstForTesting() { + return extraStarlarkOptionsFirst; } @VisibleForTesting - Map<String, Object> getExtraSkylarkOptionsSecondForTesting() { - return extraSkylarkOptionsSecond; + Map<String, Object> getExtraStarlarkOptionsSecondForTesting() { + return extraStarlarkOptionsSecond; } /** @@ -637,8 +634,8 @@ && differingOptions.isEmpty() && skylarkFirst.isEmpty() && skylarkSecond.isEmpty() - && extraSkylarkOptionsFirst.isEmpty() - && extraSkylarkOptionsSecond.isEmpty(); + && extraStarlarkOptionsFirst.isEmpty() + && extraStarlarkOptionsSecond.isEmpty(); } public String prettyPrint() { @@ -672,9 +669,9 @@ private final byte[] baseFingerprint; private final String checksum; - private final Map<String, Object> differingSkylarkOptions; - private final List<String> extraFirstSkylarkOptions; - private final Map<String, Object> extraSecondSkylarkOptions; + private final Map<String, Object> differingStarlarkOptions; + private final List<String> extraFirstStarlarkOptions; + private final Map<String, Object> extraSecondStarlarkOptions; @VisibleForTesting public OptionsDiffForReconstruction( @@ -683,17 +680,17 @@ ImmutableList<FragmentOptions> extraSecondFragments, byte[] baseFingerprint, String checksum, - Map<String, Object> differingSkylarkOptions, - List<String> extraFirstSkylarkOptions, - Map<String, Object> extraSecondSkylarkOptions) { + Map<String, Object> differingStarlarkOptions, + List<String> extraFirstStarlarkOptions, + Map<String, Object> extraSecondStarlarkOptions) { this.differingOptions = differingOptions; this.extraFirstFragmentClasses = extraFirstFragmentClasses; this.extraSecondFragments = extraSecondFragments; this.baseFingerprint = baseFingerprint; this.checksum = checksum; - this.differingSkylarkOptions = differingSkylarkOptions; - this.extraFirstSkylarkOptions = extraFirstSkylarkOptions; - this.extraSecondSkylarkOptions = extraSecondSkylarkOptions; + this.differingStarlarkOptions = differingStarlarkOptions; + this.extraFirstStarlarkOptions = extraFirstStarlarkOptions; + this.extraSecondStarlarkOptions = extraSecondStarlarkOptions; } private static OptionsDiffForReconstruction getEmpty(byte[] baseFingerprint, String checksum) { @@ -738,9 +735,9 @@ return differingOptions.isEmpty() && extraFirstFragmentClasses.isEmpty() && extraSecondFragments.isEmpty() - && differingSkylarkOptions.isEmpty() - && extraFirstSkylarkOptions.isEmpty() - && extraSecondSkylarkOptions.isEmpty(); + && differingStarlarkOptions.isEmpty() + && extraFirstStarlarkOptions.isEmpty() + && extraSecondStarlarkOptions.isEmpty(); } @Override @@ -762,9 +759,9 @@ .add("differingOptions", differingOptions) .add("extraFirstFragmentClasses", extraFirstFragmentClasses) .add("extraSecondFragments", extraSecondFragments) - .add("differingSkylarkOptions", differingSkylarkOptions) - .add("extraFirstSkylarkOptions", extraFirstSkylarkOptions) - .add("extraSecondSkylarkOptions", extraSecondSkylarkOptions) + .add("differingStarlarkOptions", differingStarlarkOptions) + .add("extraFirstStarlarkOptions", extraFirstStarlarkOptions) + .add("extraSecondStarlarkOptions", extraSecondStarlarkOptions) .toString(); } @@ -797,9 +794,9 @@ context.serialize(diff.extraSecondFragments, bytesOut); bytesOut.writeByteArrayNoTag(diff.baseFingerprint); context.serialize(diff.checksum, bytesOut); - context.serialize(diff.differingSkylarkOptions, bytesOut); - context.serialize(diff.extraFirstSkylarkOptions, bytesOut); - context.serialize(diff.extraSecondSkylarkOptions, bytesOut); + context.serialize(diff.differingStarlarkOptions, bytesOut); + context.serialize(diff.extraFirstStarlarkOptions, bytesOut); + context.serialize(diff.extraSecondStarlarkOptions, bytesOut); bytesOut.flush(); byteStringOut.flush(); int optionsDiffSize = byteStringOut.size(); @@ -832,9 +829,9 @@ ImmutableList<FragmentOptions> extraSecondFragments = context.deserialize(codedInput); byte[] baseFingerprint = codedInput.readByteArray(); String checksum = context.deserialize(codedInput); - Map<String, Object> differingSkylarkOptions = context.deserialize(codedInput); - List<String> extraFirstSkylarkOptions = context.deserialize(codedInput); - Map<String, Object> extraSecondSkylarkOptions = context.deserialize(codedInput); + Map<String, Object> differingStarlarkOptions = context.deserialize(codedInput); + List<String> extraFirstStarlarkOptions = context.deserialize(codedInput); + Map<String, Object> extraSecondStarlarkOptions = context.deserialize(codedInput); diff = new OptionsDiffForReconstruction( differingOptions, @@ -842,9 +839,9 @@ extraSecondFragments, baseFingerprint, checksum, - differingSkylarkOptions, - extraFirstSkylarkOptions, - extraSecondSkylarkOptions); + differingStarlarkOptions, + extraFirstStarlarkOptions, + extraSecondStarlarkOptions); cache.putBytesFromOptionsDiff(diff, bytes); } return diff;
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkRuleContext.java b/src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkRuleContext.java index 72ac8b0..2f9d6a5 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkRuleContext.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkRuleContext.java
@@ -587,7 +587,7 @@ ruleLabelCanonicalName)); } ImmutableMap<String, Object> skylarkFlagSettings = - ruleContext.getConfiguration().getOptions().getSkylarkOptions(); + ruleContext.getConfiguration().getOptions().getStarlarkOptions(); Type<?> buildSettingType = ruleContext.getRule().getRuleClassObject().getBuildSetting().getType();
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkConfig.java b/src/main/java/com/google/devtools/build/lib/analysis/skylark/StarlarkConfig.java similarity index 89% rename from src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkConfig.java rename to src/main/java/com/google/devtools/build/lib/analysis/skylark/StarlarkConfig.java index 061e0b5..e5b1f7c 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkConfig.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/skylark/StarlarkConfig.java
@@ -22,14 +22,12 @@ import static com.google.devtools.build.lib.syntax.Type.STRING_LIST; import com.google.devtools.build.lib.packages.BuildSetting; -import com.google.devtools.build.lib.skylarkbuildapi.SkylarkConfigApi; +import com.google.devtools.build.lib.skylarkbuildapi.StarlarkConfigApi; import com.google.devtools.build.lib.skylarkinterface.SkylarkPrinter; -/** - * Skylark namespace for creating build settings. - * TODO(juliexxia): Consider adding more types of build settings, specifically other label types. - */ -public class SkylarkConfig implements SkylarkConfigApi { +/** Starlark namespace for creating build settings. */ +// TODO(juliexxia): Consider adding more types of build settings, specifically other label types. +public class StarlarkConfig implements StarlarkConfigApi { @Override public BuildSetting intSetting(Boolean flag) {
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/test/TestTrimmingTransitionFactory.java b/src/main/java/com/google/devtools/build/lib/analysis/test/TestTrimmingTransitionFactory.java index 5a35c16..956a645 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/test/TestTrimmingTransitionFactory.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/test/TestTrimmingTransitionFactory.java
@@ -54,7 +54,7 @@ BuildOptions.Builder builder = BuildOptions.builder(); for (FragmentOptions options : originalOptions.getNativeOptions()) { if (!(options instanceof TestOptions)) { - builder.add(options); + builder.addFragmentOptions(options); } } return builder.build();
diff --git a/src/main/java/com/google/devtools/build/lib/buildtool/BuildRequest.java b/src/main/java/com/google/devtools/build/lib/buildtool/BuildRequest.java index 67c333c..7003f0e 100644 --- a/src/main/java/com/google/devtools/build/lib/buildtool/BuildRequest.java +++ b/src/main/java/com/google/devtools/build/lib/buildtool/BuildRequest.java
@@ -51,7 +51,7 @@ public class BuildRequest implements OptionsProvider { private final UUID id; private final LoadingCache<Class<? extends OptionsBase>, Optional<OptionsBase>> optionsCache; - private final Map<String, Object> skylarkOptions; + private final Map<String, Object> starlarkOptions; /** A human-readable description of all the non-default option settings. */ private final String optionsDescription; @@ -107,17 +107,16 @@ return Optional.fromNullable(result); } }); - this.skylarkOptions = options.getSkylarkOptions(); + this.starlarkOptions = options.getStarlarkOptions(); for (Class<? extends OptionsBase> optionsClass : MANDATORY_OPTIONS) { Preconditions.checkNotNull(getOptions(optionsClass)); } } - @Override - public Map<String, Object> getSkylarkOptions() { - return skylarkOptions; + public Map<String, Object> getStarlarkOptions() { + return starlarkOptions; } /**
diff --git a/src/main/java/com/google/devtools/build/lib/packages/BuildSetting.java b/src/main/java/com/google/devtools/build/lib/packages/BuildSetting.java index 506aaab..f3f6773 100644 --- a/src/main/java/com/google/devtools/build/lib/packages/BuildSetting.java +++ b/src/main/java/com/google/devtools/build/lib/packages/BuildSetting.java
@@ -14,7 +14,7 @@ package com.google.devtools.build.lib.packages; import com.google.common.annotations.VisibleForTesting; -import com.google.devtools.build.lib.skylarkbuildapi.SkylarkConfigApi.BuildSettingApi; +import com.google.devtools.build.lib.skylarkbuildapi.StarlarkConfigApi.BuildSettingApi; import com.google.devtools.build.lib.skylarkinterface.SkylarkPrinter; import com.google.devtools.build.lib.syntax.Type;
diff --git a/src/main/java/com/google/devtools/build/lib/rules/config/ConfigRules.java b/src/main/java/com/google/devtools/build/lib/rules/config/ConfigRules.java index 2f48187..6edf1e6 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/config/ConfigRules.java +++ b/src/main/java/com/google/devtools/build/lib/rules/config/ConfigRules.java
@@ -17,7 +17,7 @@ import com.google.devtools.build.lib.analysis.BaseRuleClasses; import com.google.devtools.build.lib.analysis.ConfiguredRuleClassProvider; import com.google.devtools.build.lib.analysis.ConfiguredRuleClassProvider.RuleSet; -import com.google.devtools.build.lib.analysis.skylark.SkylarkConfig; +import com.google.devtools.build.lib.analysis.skylark.StarlarkConfig; import com.google.devtools.build.lib.rules.core.CoreRules; import com.google.devtools.build.lib.rules.platform.PlatformRules; import com.google.devtools.build.lib.skylarkbuildapi.config.ConfigBootstrap; @@ -44,9 +44,7 @@ builder.addRuleDefinition(new ConfigRuleClasses.ConfigFeatureFlagRule()); builder.addSkylarkBootstrap( new ConfigBootstrap( - new ConfigSkylarkCommon(), - new SkylarkConfig(), - new ConfigGlobalLibrary())); + new ConfigSkylarkCommon(), new StarlarkConfig(), new ConfigGlobalLibrary())); } @Override
diff --git a/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/SkylarkRuleFunctionsApi.java b/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/SkylarkRuleFunctionsApi.java index 654b8d3..aab2e53 100644 --- a/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/SkylarkRuleFunctionsApi.java +++ b/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/SkylarkRuleFunctionsApi.java
@@ -16,7 +16,7 @@ import com.google.devtools.build.lib.cmdline.Label; import com.google.devtools.build.lib.events.Location; -import com.google.devtools.build.lib.skylarkbuildapi.SkylarkConfigApi.BuildSettingApi; +import com.google.devtools.build.lib.skylarkbuildapi.StarlarkConfigApi.BuildSettingApi; import com.google.devtools.build.lib.skylarkinterface.Param; import com.google.devtools.build.lib.skylarkinterface.ParamType; import com.google.devtools.build.lib.skylarkinterface.SkylarkCallable;
diff --git a/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/SkylarkConfigApi.java b/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/StarlarkConfigApi.java similarity index 96% rename from src/main/java/com/google/devtools/build/lib/skylarkbuildapi/SkylarkConfigApi.java rename to src/main/java/com/google/devtools/build/lib/skylarkbuildapi/StarlarkConfigApi.java index eb8da0a..446183d 100644 --- a/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/SkylarkConfigApi.java +++ b/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/StarlarkConfigApi.java
@@ -23,7 +23,7 @@ /** * The "config" module of the Build API. * - * <p>This exposes methods to describe what kind of build setting (if any) a skylark rule is using + * <p>This exposes methods to describe what kind of build setting (if any) a starlark rule is using * the {@code build_setting} attr of the {@code rule(...)} function. */ @SkylarkModule( @@ -46,9 +46,9 @@ + " build_setting = config.int(flag = True),\n" + " ...\n" + " )</pre>") -// TODO(juliexxia): Create formal documentation for skylark build configuration efforts +// TODO(juliexxia): Create formal documentation for starlark build configuration efforts // (b/112545834) -public interface SkylarkConfigApi extends SkylarkValue { +public interface StarlarkConfigApi extends SkylarkValue { static final String FLAG_ARG = "flag"; static final String FLAG_ARG_DOC =
diff --git a/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/config/ConfigBootstrap.java b/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/config/ConfigBootstrap.java index 7300f5f..752a74d 100644 --- a/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/config/ConfigBootstrap.java +++ b/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/config/ConfigBootstrap.java
@@ -16,7 +16,7 @@ import com.google.common.collect.ImmutableMap.Builder; import com.google.devtools.build.lib.skylarkbuildapi.Bootstrap; -import com.google.devtools.build.lib.skylarkbuildapi.SkylarkConfigApi; +import com.google.devtools.build.lib.skylarkbuildapi.StarlarkConfigApi; import com.google.devtools.build.lib.syntax.Runtime; /** @@ -25,22 +25,22 @@ public class ConfigBootstrap implements Bootstrap { private final ConfigSkylarkCommonApi configSkylarkCommonApi; - private final SkylarkConfigApi skylarkConfigApi; + private final StarlarkConfigApi starlarkConfigApi; private final ConfigGlobalLibraryApi configGlobalLibrary; public ConfigBootstrap( ConfigSkylarkCommonApi configSkylarkCommonApi, - SkylarkConfigApi skylarkConfigApi, + StarlarkConfigApi starlarkConfigApi, ConfigGlobalLibraryApi configGlobalLibrary) { this.configSkylarkCommonApi = configSkylarkCommonApi; - this.skylarkConfigApi = skylarkConfigApi; + this.starlarkConfigApi = starlarkConfigApi; this.configGlobalLibrary = configGlobalLibrary; } @Override public void addBindingsToBuilder(Builder<String, Object> builder) { builder.put("config_common", configSkylarkCommonApi); - builder.put("config", skylarkConfigApi); + builder.put("config", starlarkConfigApi); Runtime.setupSkylarkLibrary(builder, configGlobalLibrary); } }
diff --git a/src/main/java/com/google/devtools/build/skydoc/fakebuildapi/FakeBuildSettingDescriptor.java b/src/main/java/com/google/devtools/build/skydoc/fakebuildapi/FakeBuildSettingDescriptor.java index 9135f97..9b00b26 100644 --- a/src/main/java/com/google/devtools/build/skydoc/fakebuildapi/FakeBuildSettingDescriptor.java +++ b/src/main/java/com/google/devtools/build/skydoc/fakebuildapi/FakeBuildSettingDescriptor.java
@@ -14,7 +14,7 @@ package com.google.devtools.build.skydoc.fakebuildapi; -import com.google.devtools.build.lib.skylarkbuildapi.SkylarkConfigApi.BuildSettingApi; +import com.google.devtools.build.lib.skylarkbuildapi.StarlarkConfigApi.BuildSettingApi; import com.google.devtools.build.lib.skylarkinterface.SkylarkPrinter; /**
diff --git a/src/main/java/com/google/devtools/build/skydoc/fakebuildapi/FakeConfigApi.java b/src/main/java/com/google/devtools/build/skydoc/fakebuildapi/FakeConfigApi.java index 2bd322b..4fd7d0b 100644 --- a/src/main/java/com/google/devtools/build/skydoc/fakebuildapi/FakeConfigApi.java +++ b/src/main/java/com/google/devtools/build/skydoc/fakebuildapi/FakeConfigApi.java
@@ -14,13 +14,11 @@ package com.google.devtools.build.skydoc.fakebuildapi; -import com.google.devtools.build.lib.skylarkbuildapi.SkylarkConfigApi; +import com.google.devtools.build.lib.skylarkbuildapi.StarlarkConfigApi; import com.google.devtools.build.lib.skylarkinterface.SkylarkPrinter; -/** - * Fake implementation of {@link SkylarkConfigApi}. - */ -public class FakeConfigApi implements SkylarkConfigApi { +/** Fake implementation of {@link StarlarkConfigApi}. */ +public class FakeConfigApi implements StarlarkConfigApi { @Override public BuildSettingApi intSetting(Boolean flag) {
diff --git a/src/main/java/com/google/devtools/common/options/OptionsParser.java b/src/main/java/com/google/devtools/common/options/OptionsParser.java index c6fb96d..fb0b9e3 100644 --- a/src/main/java/com/google/devtools/common/options/OptionsParser.java +++ b/src/main/java/com/google/devtools/common/options/OptionsParser.java
@@ -180,7 +180,7 @@ private final List<String> residue = new ArrayList<String>(); private final List<String> postDoubleDashResidue = new ArrayList<>(); private boolean allowResidue = true; - private Map<String, Object> skylarkOptions = new HashMap<>(); + private Map<String, Object> starlarkOptions = new HashMap<>(); OptionsParser(OptionsData optionsData) { impl = new OptionsParserImpl(optionsData); @@ -197,13 +197,13 @@ } @Override - public Map<String, Object> getSkylarkOptions() { - return skylarkOptions; + public Map<String, Object> getStarlarkOptions() { + return starlarkOptions; } @VisibleForTesting - public void setSkylarkOptionsForTesting(Map<String, Object> skylarkOptions) { - this.skylarkOptions = skylarkOptions; + public void setStarlarkOptionsForTesting(Map<String, Object> starlarkOptions) { + this.starlarkOptions = starlarkOptions; } /**
diff --git a/src/main/java/com/google/devtools/common/options/OptionsProvider.java b/src/main/java/com/google/devtools/common/options/OptionsProvider.java index 73ef80e..28fb5ef 100644 --- a/src/main/java/com/google/devtools/common/options/OptionsProvider.java +++ b/src/main/java/com/google/devtools/common/options/OptionsProvider.java
@@ -22,17 +22,19 @@ * a specific class, but not e.g. the residue any other information pertaining to the command line. */ public interface OptionsProvider { - public static final OptionsProvider EMPTY = new OptionsProvider() { - @Override @Nullable - public <O extends OptionsBase> O getOptions(Class<O> optionsClass) { - return null; - } + public static final OptionsProvider EMPTY = + new OptionsProvider() { + @Override + @Nullable + public <O extends OptionsBase> O getOptions(Class<O> optionsClass) { + return null; + } - @Override - public Map<String, Object> getSkylarkOptions() { - return ImmutableMap.of(); - } - }; + @Override + public Map<String, Object> getStarlarkOptions() { + return ImmutableMap.of(); + } + }; /** * Returns the options instance for the given {@code optionsClass}, that is, @@ -44,13 +46,13 @@ @Nullable <O extends OptionsBase> O getOptions(Class<O> optionsClass); /** - * Returns the skylark options in a name:value map. + * Returns the starlark options in a name:value map. * * <p>These follow the basics of the option syntax, --<name>=<value> but are parsed and stored * differently than native options based on <name> starting with "//". This is a sufficient - * demarcation between skylark flags and native flags for now since all skylark flags are targets - * and are identified by their package path. But in the future when we implement short names for - * skylark options, this will need to change. + * demarcation between starlark flags and native flags for now since all starlark flags are + * targets and are identified by their package path. But in the future when we implement short + * names for starlark options, this will need to change. */ - Map<String, Object> getSkylarkOptions(); + Map<String, Object> getStarlarkOptions(); }
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/config/BuildOptionsTest.java b/src/test/java/com/google/devtools/build/lib/analysis/config/BuildOptionsTest.java index c5bcd36..b19c014 100644 --- a/src/test/java/com/google/devtools/build/lib/analysis/config/BuildOptionsTest.java +++ b/src/test/java/com/google/devtools/build/lib/analysis/config/BuildOptionsTest.java
@@ -187,7 +187,7 @@ } @Test - public void optionsDiff_sameSkylarkOptions() throws Exception { + public void optionsDiff_sameStarlarkOptions() throws Exception { String flagName = "//foo/flag"; String flagValue = "value"; BuildOptions one = BuildOptions.of(ImmutableMap.of(flagName, flagValue)); @@ -197,7 +197,7 @@ } @Test - public void optionsDiff_differentSkylarkOptions() throws Exception { + public void optionsDiff_differentStarlarkOptions() throws Exception { String flagName = "//bar/flag"; String flagValueOne = "valueOne"; String flagValueTwo = "valueTwo"; @@ -207,15 +207,15 @@ OptionsDiff diff = BuildOptions.diff(one, two); assertThat(diff.areSame()).isFalse(); - assertThat(diff.getSkylarkFirstForTesting().keySet()) - .isEqualTo(diff.getSkylarkSecondForTesting().keySet()); - assertThat(diff.getSkylarkFirstForTesting().keySet()).containsExactly(flagName); - assertThat(diff.getSkylarkFirstForTesting().values()).containsExactly(flagValueOne); - assertThat(diff.getSkylarkSecondForTesting().values()).containsExactly(flagValueTwo); + assertThat(diff.getStarlarkFirstForTesting().keySet()) + .isEqualTo(diff.getStarlarkSecondForTesting().keySet()); + assertThat(diff.getStarlarkFirstForTesting().keySet()).containsExactly(flagName); + assertThat(diff.getStarlarkFirstForTesting().values()).containsExactly(flagValueOne); + assertThat(diff.getStarlarkSecondForTesting().values()).containsExactly(flagValueTwo); } @Test - public void optionsDiff_extraSkylarkOptions() throws Exception { + public void optionsDiff_extraStarlarkOptions() throws Exception { String flagNameOne = "//extra/flag/one"; String flagNameTwo = "//extra/flag/two"; String flagValue = "foo"; @@ -225,13 +225,13 @@ OptionsDiff diff = BuildOptions.diff(one, two); assertThat(diff.areSame()).isFalse(); - assertThat(diff.getExtraSkylarkOptionsFirstForTesting()).containsExactly(flagNameOne); - assertThat(diff.getExtraSkylarkOptionsSecondForTesting().entrySet()) + assertThat(diff.getExtraStarlarkOptionsFirstForTesting()).containsExactly(flagNameOne); + assertThat(diff.getExtraStarlarkOptionsSecondForTesting().entrySet()) .containsExactly(Maps.immutableEntry(flagNameTwo, flagValue)); } @Test - public void applyDiff_sameSkylarkOptions() throws Exception { + public void applyDiff_sameStarlarkOptions() throws Exception { String flagName = "//foo/flag"; String flagValue = "value"; BuildOptions one = BuildOptions.of(ImmutableMap.of(flagName, flagValue)); @@ -248,7 +248,7 @@ } @Test - public void applyDiff_differentSkylarkOptions() throws Exception { + public void applyDiff_differentStarlarkOptions() throws Exception { String flagName = "//bar/flag"; String flagValueOne = "valueOne"; String flagValueTwo = "valueTwo"; @@ -262,7 +262,7 @@ } @Test - public void applyDiff_extraSkylarkOptions() throws Exception { + public void applyDiff_extraStarlarkOptions() throws Exception { String flagNameOne = "//extra/flag/one"; String flagNameTwo = "//extra/flag/two"; String flagValue = "foo";
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java b/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java index 2edf89d..a33f235 100644 --- a/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java +++ b/src/test/java/com/google/devtools/build/lib/analysis/util/BuildViewTestCase.java
@@ -358,8 +358,8 @@ optionsParser.parse(allArgs); optionsParser.parse(args); - // TODO(juliexxia): when the skylark options parsing work goes in, add type verification here. - optionsParser.setSkylarkOptionsForTesting(skylarkOptions); + // TODO(juliexxia): when the starlark options parsing work goes in, add type verification here. + optionsParser.setStarlarkOptionsForTesting(skylarkOptions); InvocationPolicyEnforcer optionsPolicyEnforcer = getAnalysisMock().getInvocationPolicyEnforcer();
diff --git a/src/test/java/com/google/devtools/build/lib/rules/config/ConfigFeatureFlagTransitionFactoryTest.java b/src/test/java/com/google/devtools/build/lib/rules/config/ConfigFeatureFlagTransitionFactoryTest.java index 9a06276..e9f2825 100644 --- a/src/test/java/com/google/devtools/build/lib/rules/config/ConfigFeatureFlagTransitionFactoryTest.java +++ b/src/test/java/com/google/devtools/build/lib/rules/config/ConfigFeatureFlagTransitionFactoryTest.java
@@ -60,7 +60,7 @@ } @Override - public Map<String, Object> getSkylarkOptions() { + public Map<String, Object> getStarlarkOptions() { return ImmutableMap.of(); } }
diff --git a/src/test/java/com/google/devtools/build/lib/skyframe/MacOSXFsEventsDiffAwarenessTest.java b/src/test/java/com/google/devtools/build/lib/skyframe/MacOSXFsEventsDiffAwarenessTest.java index cd5fbb7..bd3c992 100644 --- a/src/test/java/com/google/devtools/build/lib/skyframe/MacOSXFsEventsDiffAwarenessTest.java +++ b/src/test/java/com/google/devtools/build/lib/skyframe/MacOSXFsEventsDiffAwarenessTest.java
@@ -141,7 +141,7 @@ } @Override - public Map<String, Object> getSkylarkOptions() { + public Map<String, Object> getStarlarkOptions() { return ImmutableMap.of(); } }