diff options
| author |  cparsons <cparsons@google.com> | 2017-06-15 22:02:06 +0200 |
|---|---|---|
| committer |  Yun Peng <pcloudy@google.com> | 2017-06-16 09:27:18 +0200 |
| commit | 8effaa779808cc22d3312b0a353cfa7e3e084f20 (patch) | |
| tree | ab70cec4f202531dee10c74f409075a9a8c73943 /src/main/java/com/google/devtools/build/lib/rules/apple | |
| parent | daa17773ce757ba72db83b03d4a31bbbbbb70d18 (diff) | |
Rollback the copt-changing functionality of the extension_safe attribute
This change impacted build times of builds with libraries that needed to be built both extension-safe and non-extension-safe.
This also caused duplicate symbol issues in such builds.
The extension_safe attribute will now be a no-op, with subsequent removal to follow.
RELNOTES: The extension_safe attribute of apple_binary no longer validates transitive dependencies are compiled against extension_safe APIs.
PiperOrigin-RevId: 159142000
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/apple')
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/apple/AppleConfiguration.java | 16 |
1 files changed, 4 insertions, 12 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/apple/AppleConfiguration.java b/src/main/java/com/google/devtools/build/lib/rules/apple/AppleConfiguration.java index ace529bafb..1ca3eedcb2 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/apple/AppleConfiguration.java +++ b/src/main/java/com/google/devtools/build/lib/rules/apple/AppleConfiguration.java @@ -679,22 +679,14 @@ public class AppleConfiguration extends BuildConfiguration.Fragment { FRAMEWORK("framework"), /** Split transition distinguisher for {@code apple_watch1_extension} rule. */ WATCH_OS1_EXTENSION("watch_os1_extension"), - /** Distinguisher for non-extension {@code apple_binary} rule with "ios" platform_type. */ + /** Distinguisher for {@code apple_binary} rule with "ios" platform_type. */ APPLEBIN_IOS("applebin_ios"), - /** Distinguisher for non-extension {@code apple_binary} rule with "watchos" platform_type. */ + /** Distinguisher for {@code apple_binary} rule with "watchos" platform_type. */ APPLEBIN_WATCHOS("applebin_watchos"), - /** Distinguisher for non-extension {@code apple_binary} rule with "tvos" platform_type. */ + /** Distinguisher for {@code apple_binary} rule with "tvos" platform_type. */ APPLEBIN_TVOS("applebin_tvos"), - /** Distinguisher for non-extension {@code apple_binary} rule with "macos" platform_type. */ + /** Distinguisher for {@code apple_binary} rule with "macos" platform_type. */ APPLEBIN_MACOS("applebin_macos"), - /** Distinguisher for extension {@code apple_binary} rule with "ios" platform_type. */ - APPLEBIN_IOS_EXT("applebin_ios_ext"), - /** Distinguisher for extension {@code apple_binary} rule with "watchos" platform_type. */ - APPLEBIN_WATCHOS_EXT("applebin_watchos_ext"), - /** Distinguisher for extension {@code apple_binary} rule with "tvos" platform_type. */ - APPLEBIN_TVOS_EXT("applebin_tvos_ext"), - /** Distinguisher for extension {@code apple_binary} rule with "macos" platform_type. */ - APPLEBIN_MACOS_EXT("applebin_macos_ext"), /** * Distinguisher for the apple crosstool configuration. We use "apl" for output directory |