Skip to content

Use a language directory's manual.xml when present#322

Open
lacatoire wants to merge 1 commit into
php:masterfrom
lacatoire:feat/per-language-manual
Open

Use a language directory's manual.xml when present#322
lacatoire wants to merge 1 commit into
php:masterfrom
lacatoire:feat/per-language-manual

Conversation

@lacatoire

@lacatoire lacatoire commented Jul 16, 2026

Copy link
Copy Markdown
Member

When a language directory contains its own manual.xml, configure.php now builds from it instead of the main manual skeleton in doc-base. When absent, the doc-base manual.xml is used as before, so all existing builds (en and translations) are byte-for-byte unaffected.

This lets a manual be fully self-contained in its own repository: it can declare its own entity wiring and reference only its own books. Motivation is the third-party extension manual split (lacatoire/doc-contrib), where reusing the main skeleton fails because it references ~160 core books the contrib repo does not provide. Supersedes the file-entities scanning approach discussed in #320.

Let a language directory ship its own manual.xml to build a
self-contained manual that references only its own books, instead of the
main manual skeleton in doc-base. Falls back to the doc-base manual.xml
when absent, so existing builds are unaffected.
@lacatoire lacatoire force-pushed the feat/per-language-manual branch from 19df0db to fe2ce5c Compare July 16, 2026 08:50

@alfsb alfsb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is ok for me, and aligns with what I'm developing in doc-en 5682. But it may conflict with doc-base 321, so a bit of manual merge would be necessary.

In the meantime, apply these changes locally, so you are not blocked while all configure.php PRs are not merged.

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