Skip to content

workaround(erlang): disable doc subpackage — fop not available#16625

Merged
christopherco merged 1 commit intotomls/base/mainfrom
chrco/fix-erlang2
Apr 13, 2026
Merged

workaround(erlang): disable doc subpackage — fop not available#16625
christopherco merged 1 commit intotomls/base/mainfrom
chrco/fix-erlang2

Conversation

@christopherco
Copy link
Copy Markdown
Collaborator

fop (Apache FOP) is not packaged in Azure Linux. It is only needed to build the erlang-doc subpackage (PDF documentation from XSL-FO). The upstream spec already has a conditional to disable docs on RHEL 8+, but AZL hits the Fedora codepath where docs are enabled by default.

Use build.without = ["doc"] to disable doc generation.

Build verified locally; erlang-erts RPM contents inspected in mock.

fop (Apache FOP) is not packaged in Azure Linux. It is only needed to
build the erlang-doc subpackage (PDF documentation from XSL-FO). The
upstream spec already has a conditional to disable docs on RHEL 8+,
but AZL hits the Fedora codepath where docs are enabled by default.

Use build.without = ["doc"] to disable doc generation.

Build verified locally; erlang-erts RPM contents inspected in mock.
@christopherco christopherco marked this pull request as ready for review April 13, 2026 00:02
Copilot AI review requested due to automatic review settings April 13, 2026 00:02
Copy link
Copy Markdown
Contributor

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

This PR disables Erlang PDF documentation generation in Azure Linux builds because Apache FOP isn’t packaged, and the erlang spec enables docs by default on the Fedora codepath.

Changes:

  • Add a dedicated erlang.comp.toml with build.without = ["doc"] to pass --without doc to the RPM build.
  • Remove the bare [components.erlang] entry from components.toml to avoid duplicate component tables now that a dedicated *.comp.toml exists.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
base/comps/erlang/erlang.comp.toml Introduces build.without = ["doc"] to disable the doc subpackage build path that requires FOP.
base/comps/components.toml Removes the inline erlang component entry since the dedicated component file will be included via **/*.comp.toml.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@christopherco christopherco merged commit 05e9d39 into tomls/base/main Apr 13, 2026
9 checks passed
@christopherco christopherco deleted the chrco/fix-erlang2 branch April 13, 2026 04:09
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