Skip to content

Minimize DTD sybsystem - #5742

Open
alfsb wants to merge 2 commits into
php:masterfrom
alfsb:minimize-dtd
Open

Minimize DTD sybsystem#5742
alfsb wants to merge 2 commits into
php:masterfrom
alfsb:minimize-dtd

Conversation

@alfsb

@alfsb alfsb commented Aug 6, 2026

Copy link
Copy Markdown
Member

The doc-en side of DTD subsystem minimization.

Convert language-defs.ent into XML, and rewrite manual.xml to use the minimized DTD configuration. Also in manual.xml:

  • Root <set> namespaces is reorganized,
  • Some xmlns re-declarations removed;
  • funcindex.xml is embedded, so &global.function-index; is not necessary anymore.

This PR depends on the PR with the same name on the doc-base side.

Comments and reviews are welcome. Plan to merge this in one week.

Comment thread language-defs.ent
@@ -1,115 +0,0 @@
<?xml version="1.0" encoding="utf-8"?>

@lacatoire lacatoire Aug 7, 2026

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 file is still referenced in docbookcs.xml:26

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.

In theory, this file still exists in translations, more so on the more than one with decade old outdated files. In this case, should they be removed as they are removed from doc-en, should they be removed only when all translations are updated (a.k.a., never) ?


<entity name="AuthenticationServices">Authentication Services</entity>

<entity name="CalendarandEventRelatedExtensions">Date and Time Related

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.

some have gain a \n, is it intended ?

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.

It is intended, and it is ugly. There are a few lines that pass the 80 column marker. I'm considering removing the breaks, as this file is still (and forever will be?) small.

@alfsb

alfsb commented Aug 7, 2026

Copy link
Copy Markdown
Member Author

The failures are expected, for now. The other PR on doc-base, that will be merged first, rename some doc-base/temp/ files and extensions.

The only remaining DTD entity files left are doc-base/entities/global.ent and doc-lang/language-snippets.ent.

The first one is to be simply erased after more changes are sent indo doc-base/configure.php, and the last one may linger for a while.

After XML trim lands, all these <entities> loading should be replaced by a single and stable <file>../doc-base/temp/doctype.dtd</file>.

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