Skip to content

Give all static libraries produced by the apple_static_xcframework rule file names based directly on the inferred bundle name.#1478

Merged
keith merged 1 commit intomasterfrom
ks/give-all-static-libraries-produced-by-the-apple_static_xcframework-rule-file-names-based-directly-on-the-inferred-bundle-name
May 24, 2022
Merged

Give all static libraries produced by the apple_static_xcframework rule file names based directly on the inferred bundle name.#1478
keith merged 1 commit intomasterfrom
ks/give-all-static-libraries-produced-by-the-apple_static_xcframework-rule-file-names-based-directly-on-the-inferred-bundle-name

Conversation

@keith
Copy link
Copy Markdown
Member

@keith keith commented May 24, 2022

PiperOrigin-RevId: 448546861
(cherry picked from commit e9a8823)

@keith keith enabled auto-merge (rebase) May 24, 2022 20:37
keith referenced this pull request May 24, 2022
…le file names based directly on the inferred bundle name.

PiperOrigin-RevId: 448546861
@keith
Copy link
Copy Markdown
Member Author

keith commented May 24, 2022

There were some test conflicts here which I picked the right version of for now but will have to be re-updated with #1447, it's fine they will fail because the numbers like 2 and 7 become strings instead

@lyft-lint-bot
Copy link
Copy Markdown

Lyft integration job started: https://buildkite.com/lyft/rules-apple/builds/616 (must be Lyft employee to view)

@keith keith force-pushed the ks/give-all-static-libraries-produced-by-the-apple_static_xcframework-rule-file-names-based-directly-on-the-inferred-bundle-name branch from d8da817 to 9b40d93 Compare May 24, 2022 21:57
…le file names based directly on the inferred bundle name.

PiperOrigin-RevId: 448546861
(cherry picked from commit e9a8823)
@keith keith force-pushed the ks/give-all-static-libraries-produced-by-the-apple_static_xcframework-rule-file-names-based-directly-on-the-inferred-bundle-name branch from 9b40d93 to 0cfe9c4 Compare May 24, 2022 21:58
@keith keith merged commit 012fc12 into master May 24, 2022
@keith keith deleted the ks/give-all-static-libraries-produced-by-the-apple_static_xcframework-rule-file-names-based-directly-on-the-inferred-bundle-name branch May 24, 2022 22:14
jpsim added a commit to envoyproxy/envoy-mobile that referenced this pull request May 26, 2022
To pull in bazelbuild/rules_apple#1478

```
Envoy.xcframework
├── Info.plist
├── ios-arm64
│ ├── Envoy.a
│ ├── Envoy.swiftmodule
│ │ ├── arm64.swiftdoc
│ │ └── arm64.swiftinterface
│ └── Headers
│     ├── Envoy.h
│     └── module.modulemap
└── ios-arm64_x86_64-simulator
    ├── Envoy.a
    ├── Envoy.swiftmodule
    │ ├── arm64.swiftdoc
    │ ├── arm64.swiftinterface
    │ ├── x86_64.swiftdoc
    │ └── x86_64.swiftinterface
    └── Headers
        ├── Envoy.h
        └── module.modulemap
```

Signed-off-by: JP Simard <jp@jpsim.com>
jpsim added a commit to envoyproxy/envoy-mobile that referenced this pull request May 26, 2022
To pull in bazelbuild/rules_apple#1478

```
Envoy.xcframework
├── Info.plist
├── ios-arm64
│ ├── Envoy.a
│ ├── Envoy.swiftmodule
│ │ ├── arm64.swiftdoc
│ │ └── arm64.swiftinterface
│ └── Headers
│     ├── Envoy.h
│     └── module.modulemap
└── ios-arm64_x86_64-simulator
    ├── Envoy.a
    ├── Envoy.swiftmodule
    │ ├── arm64.swiftdoc
    │ ├── arm64.swiftinterface
    │ ├── x86_64.swiftdoc
    │ └── x86_64.swiftinterface
    └── Headers
        ├── Envoy.h
        └── module.modulemap
```

Risk Level: Low
Testing: Validated by pulling in this version in Xcode and running on device + simulator
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: JP Simard <jp@jpsim.com>
jpsim added a commit to envoyproxy/envoy that referenced this pull request Nov 29, 2022
To pull in bazelbuild/rules_apple#1478

```
Envoy.xcframework
├── Info.plist
├── ios-arm64
│ ├── Envoy.a
│ ├── Envoy.swiftmodule
│ │ ├── arm64.swiftdoc
│ │ └── arm64.swiftinterface
│ └── Headers
│     ├── Envoy.h
│     └── module.modulemap
└── ios-arm64_x86_64-simulator
    ├── Envoy.a
    ├── Envoy.swiftmodule
    │ ├── arm64.swiftdoc
    │ ├── arm64.swiftinterface
    │ ├── x86_64.swiftdoc
    │ └── x86_64.swiftinterface
    └── Headers
        ├── Envoy.h
        └── module.modulemap
```

Risk Level: Low
Testing: Validated by pulling in this version in Xcode and running on device + simulator
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: JP Simard <jp@jpsim.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants