Skip to content

Conversation

justice-adams-apple
Copy link
Contributor

Use the associated library name for the CFBundleIdentifier within the plist. This will prevent cases where we have projects with multiple targets referencing the project name itself (i.e swiftSwiftOnoneSupport had it's bundle name set to SwiftCore)

@@ -158,5 +158,5 @@ emit_swift_interface(swift_Concurrency)
install_swift_interface(swift_Concurrency)

# Configure plist creation for Darwin platforms.
generate_plist("${CMAKE_PROJECT_NAME}" "${CMAKE_PROJECT_VERSION}" swift_Concurrency)
generate_plist(swiftConcurrency swift_Concurrency"${CMAKE_PROJECT_VERSION}" swift_Concurrency)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
generate_plist(swiftConcurrency swift_Concurrency"${CMAKE_PROJECT_VERSION}" swift_Concurrency)
generate_plist(swiftConcurrency swift_Concurrency "${CMAKE_PROJECT_VERSION}" swift_Concurrency)

@justice-adams-apple justice-adams-apple force-pushed the jadams/fix-plist-bundle-identifier branch from 0ae1a98 to a9ab210 Compare August 28, 2025 19:17
@justice-adams-apple
Copy link
Contributor Author

@swift-ci smoke test

@justice-adams-apple justice-adams-apple merged commit ff3fdd3 into main Aug 29, 2025
3 checks passed
@justice-adams-apple justice-adams-apple deleted the jadams/fix-plist-bundle-identifier branch August 29, 2025 17:28
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.

3 participants