Skip to content

[172848427] Rework plug-in initialization to support a new plug-in layout.#1259

Open
cwakamo wants to merge 4 commits into
swiftlang:mainfrom
cwakamo:eng/PR-172848427-restructure-for-iPadOS-Swift-Playground-app-store-ingestion
Open

[172848427] Rework plug-in initialization to support a new plug-in layout.#1259
cwakamo wants to merge 4 commits into
swiftlang:mainfrom
cwakamo:eng/PR-172848427-restructure-for-iPadOS-Swift-Playground-app-store-ingestion

Conversation

@cwakamo

@cwakamo cwakamo commented Mar 27, 2026

Copy link
Copy Markdown
Contributor
  • Plug-ins are now allowed to be frameworks instead of bundles.
  • Assume plug-ins are relative to the SWBBuildService library instead of the SWBBuildService bundle.
  • In support of this, removed the pluginsDirectory parameter from the build service entry point.
  • Also reworked the responsibility for plug-in registration in tests so it's part of CoreTestSupport.swift directly.

This is part of <rdar://problem/172848427>.

cwakamo added 2 commits March 27, 2026 16:06
Assume plug-ins are relative to the SWBBuildService framework instead of the SWBBuildService bundle.
In support of this, removed the `pluginsDirectory` parameter from the build service entry point.
Also reworked the responsibility for plug-in registration in tests so it's part of CoreTestSupport.swift directly.

This is for <rdar://problem/172848427>.
@cwakamo cwakamo force-pushed the eng/PR-172848427-restructure-for-iPadOS-Swift-Playground-app-store-ingestion branch from ce93365 to 0a2cc46 Compare March 27, 2026 23:06
@cwakamo

cwakamo commented Mar 27, 2026

Copy link
Copy Markdown
Contributor Author

@swift-ci please test

@cwakamo

cwakamo commented Mar 30, 2026

Copy link
Copy Markdown
Contributor Author

@swift-ci please test

…TestSupport.swift.

This is for <rdar://problem/172848427>.
@cwakamo

cwakamo commented Apr 14, 2026

Copy link
Copy Markdown
Contributor Author

@swift-ci please test

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.

2 participants