Skip to content

Commit

Permalink
Automated rollback of commit 5586c47.
Browse files Browse the repository at this point in the history
*** Reason for rollback ***

Removing this because it's less surprising for rule authors if execution configurations start from default flags.

*** Original change description ***

Add getExec method for AppleCommandLineOptions.

Part of work on execution transitions, #7935.

PiperOrigin-RevId: 246571620
  • Loading branch information
katre authored and copybara-github committed May 3, 2019
1 parent c5b0c79 commit ee5ed35
Showing 1 changed file with 0 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -436,7 +436,6 @@ public FragmentOptions getHost() {
host.tvOsSdkVersion = tvOsSdkVersion;
host.macOsSdkVersion = macOsSdkVersion;
host.appleBitcodeMode = appleBitcodeMode;
// TODO(http://b/131411178): Replace with the execution platform.
// The host apple platform type will always be MACOS, as no other apple platform type can
// currently execute build actions. If that were the case, a host_apple_platform_type flag might
// be needed.
Expand All @@ -445,16 +444,6 @@ public FragmentOptions getHost() {
return host;
}

@Override
public FragmentOptions getExec() {
AppleCommandLineOptions exec = (AppleCommandLineOptions) super.getExec();
// TODO(http://b/131411178): Replace with the execution platform.
// The exec apple platform type will always be MACOS, as no other apple platform type can
// currently execute build actions.
exec.applePlatformType = PlatformType.MACOS;
return exec;
}

void serialize(SerializationContext context, CodedOutputStream out)
throws IOException, SerializationException {
context.serialize(this, out);
Expand Down

0 comments on commit ee5ed35

Please sign in to comment.