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

Automatic visibility of stack_snapshot dependencies to vendored packages #1583

Merged
merged 5 commits into from
Aug 24, 2021

Conversation

aherrmann
Copy link
Member

@aherrmann aherrmann commented Aug 18, 2021

Closes #1571

If a vendored package depends on an otherwise not public package in the snapshot, then this dependency is made visible to the vendored package.

To handle executable components (tool dependencies) stack_snapshot generates a JSON file listing all components and the -exe repository reads this file to generate the necessary aliases with the correct visibilities.

haskell/cabal.bzl Show resolved Hide resolved
WORKSPACE Show resolved Hide resolved
haskell/cabal.bzl Outdated Show resolved Hide resolved
Copy link
Member

@Radvendii Radvendii left a comment

Choose a reason for hiding this comment

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

Alright, I haven't tested that it works in practice, but the code itself makes sense to me and seems clean enough.

@aherrmann aherrmann added the merge-queue merge on green CI label Aug 24, 2021
aherrmann and others added 5 commits August 24, 2021 07:26
If a vendored package depends on an otherwise not public package in the
snapshot, then it is made visible to this vendored package.

To handle executable components (tool dependencies) `stack_snapshot`
generates a JSON file listing all components and the `-exe` repository
reads this file to generate the necessary aliases with the correct
visibilities.
Co-authored-by: Taeer Bar-Yam <Radvendii@users.noreply.github.com>
@mergify mergify bot merged commit 673e74a into master Aug 24, 2021
@mergify mergify bot deleted the stack-visibility branch August 24, 2021 08:08
@mergify mergify bot removed the merge-queue merge on green CI label Aug 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants