Skip to content

Commit 1acdfc4

Browse files
lberkicopybara-github
authored andcommitted
Remove --apple_compiler.
RELNOTES[INC]: The --apple_compiler command line option is not available anymore. PiperOrigin-RevId: 555975854 Change-Id: I41f06333d47224c3fb799bd89dec6752199595df
1 parent 60abcf1 commit 1acdfc4

File tree

4 files changed

+3
-20
lines changed

4 files changed

+3
-20
lines changed

src/main/java/com/google/devtools/build/lib/rules/apple/AppleCommandLineOptions.java

Lines changed: 0 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -195,20 +195,6 @@ public class AppleCommandLineOptions extends FragmentOptions {
195195
/** The default Catalyst CPU value. */
196196
public static final String DEFAULT_CATALYST_CPU = "x86_64";
197197

198-
@Option(
199-
name = "apple_compiler",
200-
defaultValue = "null",
201-
documentationCategory = OptionDocumentationCategory.TOOLCHAIN,
202-
effectTags = {
203-
OptionEffectTag.AFFECTS_OUTPUTS,
204-
OptionEffectTag.LOADING_AND_ANALYSIS,
205-
OptionEffectTag.LOSES_INCREMENTAL_STATE,
206-
},
207-
help = "The Apple target compiler. Useful for selecting variants of a toolchain "
208-
+ "(e.g. xcode-beta)."
209-
)
210-
public String cppCompiler;
211-
212198
@Option(
213199
name = "apple_grte_top",
214200
defaultValue = "null",

src/main/java/com/google/devtools/build/lib/rules/objc/MultiArchBinarySupport.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -801,7 +801,7 @@ private static void setAppleCrosstoolTransitionSharedConfiguration(
801801

802802
AppleCommandLineOptions appleOptions = from.get(AppleCommandLineOptions.class);
803803
CppOptions toCppOptions = to.get(CppOptions.class);
804-
toCppOptions.cppCompiler = appleOptions.cppCompiler;
804+
toCppOptions.cppCompiler = null;
805805
toCppOptions.libcTopLabel = appleOptions.appleLibcTop;
806806

807807
// OSX toolchains do not support fission.

src/main/starlark/builtins_bzl/common/objc/transitions.bzl

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ DEFAULT_CATALYST_CPU = "x86_64"
9090
def _output_dictionary(settings, cpu, platform_type, platforms):
9191
return {
9292
"//command_line_option:apple configuration distinguisher": "applebin_" + platform_type,
93-
"//command_line_option:compiler": settings["//command_line_option:apple_compiler"],
93+
"//command_line_option:compiler": None,
9494
"//command_line_option:cpu": cpu,
9595
"//command_line_option:crosstool_top": (
9696
settings["//command_line_option:apple_crosstool_top"]
@@ -122,7 +122,6 @@ def _apple_crosstool_transition_impl(settings, attr):
122122

123123
_apple_rule_base_transition_inputs = [
124124
"//command_line_option:apple configuration distinguisher",
125-
"//command_line_option:apple_compiler",
126125
"//command_line_option:apple_platform_type",
127126
"//command_line_option:apple_platforms",
128127
"//command_line_option:apple_crosstool_top",

src/test/java/com/google/devtools/build/lib/rules/objc/ObjcRuleTestCase.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -526,7 +526,6 @@ protected static void addAppleBinaryStarlarkRule(Scratch scratch) throws Excepti
526526
scratch.file(
527527
"test_starlark/apple_binary_starlark.bzl",
528528
"_apple_platform_transition_inputs = [",
529-
" '//command_line_option:apple_compiler',",
530529
" '//command_line_option:apple_crosstool_top',",
531530
" '//command_line_option:cpu',",
532531
" '//command_line_option:ios_multi_cpus',",
@@ -552,8 +551,7 @@ protected static void addAppleBinaryStarlarkRule(Scratch scratch) throws Excepti
552551
" '//command_line_option:apple_platform_type': platform_type,",
553552
" '//command_line_option:apple_platforms': [],",
554553
" '//command_line_option:apple_split_cpu': environment_arch,",
555-
" '//command_line_option:compiler': ",
556-
" settings['//command_line_option:apple_compiler'],",
554+
" '//command_line_option:compiler': None,",
557555
" '//command_line_option:cpu': ",
558556
" 'darwin_' + environment_arch if platform_type == 'macos'",
559557
" else platform_type + '_' + environment_arch,",

0 commit comments

Comments
 (0)