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

Include package dbs of source loaded dependencies in unit files #2228

Merged
merged 1 commit into from
Aug 15, 2024

Conversation

iteratee
Copy link
Contributor

@iteratee iteratee commented Aug 9, 2024

Package dbs are one of the few things that need to be transitively included in a unit file. Fix the following case: There are two units A and B, and A depends on B. B depends on an external dependency C. In this case the unit file for A needs to include the package db for C. HLS and GHC will not pick up the package db indirectly through B's unit file. Fix this by passing a set of package-db-only HaskellReplDepInfo when merging.

@iteratee iteratee requested a review from avdv as a code owner August 9, 2024 07:23
Package dbs are one of the few things that need to be transitively
included in a unit file. Fix the following case: There are two units A
and B, and A depends on B. B depends on an external dependency C. In
this case the unit file for A needs to include the package db for C. HLS
and GHC will not pick up the package db indirectly through B's unit
file. Fix this by passing a set of package-db-only HaskellReplDepInfo
when merging.
Copy link
Member

@avdv avdv left a comment

Choose a reason for hiding this comment

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

Thank you!

@avdv avdv added the merge-queue merge on green CI label Aug 15, 2024
@avdv avdv merged commit 3b67ec0 into tweag:master Aug 15, 2024
47 of 49 checks passed
@mergify mergify bot removed the merge-queue merge on green CI label Aug 15, 2024
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