Rollback of commit 75714b0d6e65905e41da8415ca3e7ec101575d51.

*** Reason for rollback ***

Broke Play Books tests

*** Original change description ***

Include SwiftSupport directory into the IPA when an app contains Swift.

* To pass App Store validation, an app that contains Swift code must have a SwiftSupport directory at the root of the IPA.

--
MOS_MIGRATED_REVID=128627984

1 files changed, 10 insertions, 37 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/rules/objc/ReleaseBundlingSupport.java b/src/main/java/com/google/devtools/build/lib/rules/objc/ReleaseBundlingSupport.java
index 39e6d2ce13..d52dad9ed1 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/objc/ReleaseBundlingSupport.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/objc/ReleaseBundlingSupport.java
@@ -22,9 +22,6 @@ import static com.google.devtools.build.lib.rules.objc.ObjcRuleClasses.ReleaseBu
 import static com.google.devtools.build.lib.rules.objc.ObjcRuleClasses.ReleaseBundlingRule.EXTRA_ENTITLEMENTS_ATTR;
 import static com.google.devtools.build.lib.rules.objc.TargetDeviceFamily.UI_DEVICE_FAMILY_VALUES;
 
-import com.dd.plist.NSArray;
-import com.dd.plist.NSDictionary;
-import com.dd.plist.NSObject;
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.base.Joiner;
 import com.google.common.collect.ImmutableList;
@@ -66,8 +63,14 @@ import com.google.devtools.build.lib.rules.objc.BundleSupport.ExtraActoolArgs;
 import com.google.devtools.build.lib.rules.objc.Bundling.Builder;
 import com.google.devtools.build.lib.shell.ShellUtils;
 import com.google.devtools.build.xcode.xcodegen.proto.XcodeGenProtos.XcodeprojBuildSetting;
+
+import com.dd.plist.NSArray;
+import com.dd.plist.NSDictionary;
+import com.dd.plist.NSObject;
+
 import java.util.List;
 import java.util.Map.Entry;
+
 import javax.annotation.Nullable;
 
 /**
@@ -296,7 +299,6 @@ public final class ReleaseBundlingSupport {
     registerCopyDsymPlistAction(dsymOutputType);
     registerCopyLinkmapFilesAction();
     registerSwiftStdlibActionsIfNecessary();
-    registerSwiftSupportActionsIfNecessary();
 
     registerEmbedLabelPlistAction();
     registerEnvironmentPlistAction();
@@ -1049,13 +1051,10 @@ public final class ReleaseBundlingSupport {
 
     AppleConfiguration appleConfiguration = ruleContext.getFragment(AppleConfiguration.class);
 
-    CustomCommandLine.Builder commandLine =
-        CustomCommandLine.builder()
-            .addPath(intermediateArtifacts.swiftFrameworksFileZip().getExecPath())
-            .add("Frameworks")
-            .add("--platform")
-            .add(AppleToolchain.swiftPlatform(appleConfiguration))
-            .addExecPath("--scan-executable", intermediateArtifacts.combinedArchitectureBinary());
+    CustomCommandLine.Builder commandLine = CustomCommandLine.builder()
+        .addPath(intermediateArtifacts.swiftFrameworksFileZip().getExecPath())
+        .add("--platform").add(AppleToolchain.swiftPlatform(appleConfiguration))
+        .addExecPath("--scan-executable", intermediateArtifacts.combinedArchitectureBinary());
 
     ruleContext.registerAction(
         ObjcRuleClasses.spawnAppleEnvActionBuilder(ruleContext)
@@ -1067,32 +1066,6 @@ public final class ReleaseBundlingSupport {
             .build(ruleContext));
   }
 
-  /** Registers an action to copy Swift standard library dylibs into SwiftSupport root directory. */
-  private void registerSwiftSupportActionsIfNecessary() {
-    if (!objcProvider.is(USES_SWIFT)) {
-      return;
-    }
-
-    AppleConfiguration appleConfiguration = ruleContext.getFragment(AppleConfiguration.class);
-
-    CustomCommandLine.Builder commandLine =
-        CustomCommandLine.builder()
-            .addPath(intermediateArtifacts.swiftSupportZip().getExecPath())
-            .add("SwiftSupport/" + AppleToolchain.swiftPlatform(appleConfiguration))
-            .add("--platform")
-            .add(AppleToolchain.swiftPlatform(appleConfiguration))
-            .addExecPath("--scan-executable", intermediateArtifacts.combinedArchitectureBinary());
-
-    ruleContext.registerAction(
-        ObjcRuleClasses.spawnAppleEnvActionBuilder(ruleContext)
-            .setMnemonic("SwiftCopySwiftSupport")
-            .setExecutable(attributes.swiftStdlibToolWrapper())
-            .setCommandLine(commandLine.build())
-            .addOutput(intermediateArtifacts.swiftSupportZip())
-            .addInput(intermediateArtifacts.combinedArchitectureBinary())
-            .build(ruleContext));
-  }
-
   private String extractPlistCommand(Artifact provisioningProfile) {
     return "security cms -D -i " + ShellUtils.shellEscape(provisioningProfile.getExecPathString());
   }