Skip to content

Disable package customization of module ABI name#1601

Merged
stmontgomery merged 2 commits intoswiftlang:mainfrom
stmontgomery:undo-package-module-abi
Mar 3, 2026
Merged

Disable package customization of module ABI name#1601
stmontgomery merged 2 commits intoswiftlang:mainfrom
stmontgomery:undo-package-module-abi

Conversation

@stmontgomery
Copy link
Contributor

This disables a portion of the changes introduced in #1471 which specified an alternate module ABI name when building this project as a Swift package.

Resolves rdar://170069880

Motivation:

These changes regressed our ability to build documentation for the package. In order to reintroduce the original fix in full, we'll need a way to customize the ABI name of a module for the purpose of building documentation.

Checklist:

  • Code and documentation should follow the style of the Style Guide.
  • If public symbols are renamed or modified, DocC references should be updated.

@stmontgomery stmontgomery added this to the Swift 6.4.0 (main) milestone Mar 2, 2026
@stmontgomery stmontgomery self-assigned this Mar 2, 2026
@stmontgomery stmontgomery added bug 🪲 Something isn't working documentation 📚 Improvements or additions to documentation workaround Workaround for an issue in another component (may need to revert later) labels Mar 2, 2026
Copy link
Contributor

@grynspan grynspan left a comment

Choose a reason for hiding this comment

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

Suggestion for a different way to phrase the change in Package.swift.

@stmontgomery stmontgomery merged commit 568d9aa into swiftlang:main Mar 3, 2026
27 checks passed
@stmontgomery stmontgomery deleted the undo-package-module-abi branch March 3, 2026 01:09
stmontgomery added a commit that referenced this pull request Mar 3, 2026
- **Explanation**: Resolve an issue which prevents building
documentation for the package locally.
- **Scope**: Affects the package build only, does not affect production
builds.
- **Issues**: rdar://170069880
- **Original PRs**: #1601
- **Risk**: Low; disables a compiler flag added within this release.
- **Testing**: Local documentation build no longer emits these warnings.
- **Reviewers**: @grynspan
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🪲 Something isn't working documentation 📚 Improvements or additions to documentation workaround Workaround for an issue in another component (may need to revert later)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants