Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DOC_TARGETS dropped pdf,man from doc.mk but not the installation guide #8188

Closed
paulo-ferraz-oliveira opened this issue Feb 25, 2024 · 2 comments · Fixed by #8129
Closed
Assignees
Labels
bug Issue is reported as a bug

Comments

@paulo-ferraz-oliveira
Copy link
Contributor

Describe the bug

DOC_TARGETS (introduced "to limit which types of documentation" are built) first appeared in 48ecc7a.

At that moment, it seems the following choices were possible: man, pdf, chunks, and/or html.

A more recent 218ecbe removed targets man and html.

This is not mentioned in the changelog, and the removed values are still mentioned in the upcoming release's (OTP 27) documentation.

To Reproduce

Look at the mentioned files, then look at the documentation.

You can also try to build 27 with make docs DOC_TARGETS='man html pdf chunks', which should show something like

=== Entering application stdlib
make[3]: *** No rule to make target `pdf', needed by `docs'.  Stop

and have the build failed.

Expected behavior

I'm not sure if the removal is the expectation (since it's still in the documentation), or the documentation not being updated was oversight.

Affected versions

  • 27.0-rc1

Additional context

In preparing for a newer kerl release it would be important to know what bit is wrong (code or doc) so we can adjust.

@garazdawi
Copy link
Contributor

pdf and man have been dropped in 27. I will send a PR to update the docs accordingly.

@garazdawi
Copy link
Contributor

Fixed in #8129

@garazdawi garazdawi linked a pull request Feb 26, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue is reported as a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants