Skip to content

docs(forge): explain linked library policy - #2129

Merged
mablr merged 4 commits into
masterfrom
mablr/disallow-linked-libraries
Oct 26, 2026
Merged

mablr merged 4 commits into
masterfrom
mablr/disallow-linked-libraries

Conversation

@mablr

@mablr mablr commented Oct 6, 2026

Copy link
Copy Markdown
Member

Documents allow_linked_libraries and --disallow-linked-libraries, including executable compilation scope, configured library addresses, cache transitions, CLI precedence, and JSON failures. Explains how the prohibition preserves internal libraries and dynamic test linking, with a configuration example that enables both safeguards.

Companion documentation for foundry-rs/foundry#13585; keep this draft until that implementation is ready to merge.

Document the linked-library prohibition added by Foundry PR #13585, including CLI precedence, prelinked addresses, cache transitions, JSON errors, and compatibility with dynamic test linking.
@vercel

vercel Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
foundry-docs Ready Ready Preview Oct 23, 2026 11:51am UTC

Request Review

@mablr mablr left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Document the error output, which now names each linked library and
suggests a fix, and clarify when configured library addresses are
withheld from solc.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@mablr
mablr merged commit c7a1830 into master Oct 26, 2026
5 checks passed
@mablr
mablr deleted the mablr/disallow-linked-libraries branch October 26, 2026 08:56

This branch was successfully deployed

1 active deployment
Preview — dd75f8d5 Deployed Oct 23, 2026 by vercel[bot]
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