Skip to content

Remove macOS installer dependencies#557

Open
cmcgee1024 wants to merge 1 commit into
swiftlang:mainfrom
cmcgee1024:remove_installer_dep
Open

Remove macOS installer dependencies#557
cmcgee1024 wants to merge 1 commit into
swiftlang:mainfrom
cmcgee1024:remove_installer_dep

Conversation

@cmcgee1024

@cmcgee1024 cmcgee1024 commented Jun 11, 2026

Copy link
Copy Markdown
Member

The macOS installer doesn't permit useful integration tests since it doesn't work well in headless CI environments, and also it can't do directory level customization to the install path. This allows bugs to escape that aren't manually tested since the code paths are never covered by the tests.

Use the same pkgutil approach as used by the tests, and cases where toolchains directory has been customized in all cases for macOS.

The macOS installer doesn't permit useful integration tests
since it doesn't work well in headless CI environments, and also
it doesn't permit directory level customization to the install
path. This allows bugs to escape that aren't manually tested since
the code paths are never covered by the tests.

Use the same pkgutil approach as used by the tests, and cases
where toolchains directory has been customized in all cases for
macOS.
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