Skip to content

fix(melange): read imported virtual-library dependencies from CMTs - #78

Draft
anmonteiro wants to merge 1 commit into
mainfrom
anmonteiro/fix-imported-melange-vlib-cmt-deps
Draft

fix(melange): read imported virtual-library dependencies from CMTs#78
anmonteiro wants to merge 1 commit into
mainfrom
anmonteiro/fix-imported-melange-vlib-cmt-deps

Conversation

@anmonteiro

@anmonteiro anmonteiro commented Aug 21, 2026

Copy link
Copy Markdown
Owner

Read installed Melange virtual-library implementation dependencies from CMTs, including transitive private dependencies.

Follow-up to ocaml#16066. The no-CMT fallback remains in #58.

@anmonteiro
anmonteiro force-pushed the anmonteiro/fix-imported-melange-vlib-cmt-deps branch from 6f32a11 to 62963d5 Compare August 25, 2026 04:19
@anmonteiro
anmonteiro changed the base branch from anmonteiro/test-imported-melange-vlib-cmt-deps to main August 25, 2026 04:19
anmonteiro added a commit to ocaml/dune that referenced this pull request Aug 25, 2026
Strengthen the installed Melange virtual-library regression test with a
private module whose implementation-only dependency is hidden by an
interface.

Follow-up to #16066; prepares anmonteiro#78.

Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>
@anmonteiro
anmonteiro force-pushed the anmonteiro/fix-imported-melange-vlib-cmt-deps branch 2 times, most recently from 2a78089 to 070a618 Compare August 26, 2026 00:21
Signed-off-by: Antonio Nuno Monteiro <anmonteiro@gmail.com>
@anmonteiro
anmonteiro force-pushed the anmonteiro/fix-imported-melange-vlib-cmt-deps branch from 070a618 to c0b25c3 Compare August 26, 2026 00:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant