Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Package Manager Allow Targets to Depend on Products in the Same Package #2234

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

stackotter
Copy link

This is a WIP proposal based off the discussion in this forum thread. @tammofreese created an initial implementation a year ago which is a great way to test out the proposed changes (although it may differ from the proposal slightly, and requires updating to Swift 5.9).

Forum thread: https://forums.swift.org/t/pitch-swiftpm-allow-targets-to-depend-on-products-in-the-same-package/57717/32
WIP implementation: https://github.com/tammofreese/swift-package-manager/tree/use_internal_product_as_target_dependency_experiment

Copy link
Member

@heckj heckj left a comment

Choose a reason for hiding this comment

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

I stumbled hard on the intro paragraph referencing the code block below and then moving on, assuming I'd read it. I think this would read a whole lot easier if the detail with the heart of the issue is after the code block illustrating the product dependency scenario.

@stackotter
Copy link
Author

Thanks for the feedback! I agree 👍 I've moved the detail after the code block now

@rjmccall rjmccall added the swiftpm Proposal relates to the Swift Package Manager label Dec 14, 2023
@rjmccall rjmccall added the PSG Contains topics under the domain of the Platform Steering Group label Apr 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PSG Contains topics under the domain of the Platform Steering Group swiftpm Proposal relates to the Swift Package Manager
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants