Skip to content

test: singleton (wrapped false) lib with requires#14582

Open
robinbb wants to merge 1 commit into
mainfrom
robinbb-test-singleton-with-requires
Open

test: singleton (wrapped false) lib with requires#14582
robinbb wants to merge 1 commit into
mainfrom
robinbb-test-singleton-with-requires

Conversation

@robinbb
Copy link
Copy Markdown
Collaborator

@robinbb robinbb commented May 18, 2026

Regression guard for the build of a multi-module consumer of a (wrapped false) singleton library that itself has (libraries leaf). The consumer never names the leaf's module directly; coverage that this transitive shape continues to work.

Sister fixture to no-ocamldep-leaf-lib.t (the singleton-with-no-requires case).

@robinbb robinbb force-pushed the robinbb-test-singleton-with-requires branch from fc7f76f to 93004c2 Compare May 18, 2026 17:50
Regression guard for the build of a multi-module consumer of a `(wrapped false)` singleton library that itself has `(libraries leaf)`. The consumer never names the leaf's module directly; coverage that this transitive dependency shape continues to work.

Sister fixture to `no-ocamldep-leaf-lib.t` (the singleton-with-no-requires case).

Signed-off-by: Robin Bate Boerop <me@robinbb.com>
@robinbb robinbb force-pushed the robinbb-test-singleton-with-requires branch from 93004c2 to 8c921c2 Compare May 18, 2026 22:17
@robinbb robinbb self-assigned this May 18, 2026
@robinbb robinbb requested a review from Copilot May 18, 2026 22:18
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new cram regression test that ensures a multi-module consumer can build against a (wrapped false) singleton library (bridge) that itself depends on another leaf library (leaf), without the consumer naming the leaf's modules directly.

Changes:

  • New cram test singleton-with-requires.t exercising a transitive consumer → bridge → leaf build with (wrapped false) libraries.

@robinbb robinbb marked this pull request as ready for review May 18, 2026 22:24
@robinbb
Copy link
Copy Markdown
Collaborator Author

robinbb commented May 18, 2026

@Alizter This came from my 9-layer stack for 14492. I factored it out to make the stack smaller.

@robinbb robinbb requested a review from Alizter May 18, 2026 22:25
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.

3 participants