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

curl.1 generated manpage contains invalid ".IP "false" (default)" #15506

Closed
samueloph opened this issue Nov 7, 2024 · 0 comments
Closed

curl.1 generated manpage contains invalid ".IP "false" (default)" #15506

samueloph opened this issue Nov 7, 2024 · 0 comments
Assignees

Comments

@samueloph
Copy link
Member

Specify which documentation you found a problem with

Generated manpage curl.1

The problem

Debian's lintian flagged the following issue when I packaged the latest version:
W: curl: groff-message troff:<standard input>:1315: warning: expected numeric expression, got 'f' [usr/share/man/man1/curl.1.gz:1]

This is regarding the line ".IP "false" (default)" which is under the section for --ech <config>.

The generated website looks correct at https://curl.se/docs/manpage.html:

The values allowed for can be:

false (default)

Do not attempt ECH 

But the manpage is missing that line, I can see it in the manpage source, it just doesn't get rendered, the result is:

The values allowed for can be:

Do not attempt ECH 

I believe something is breaking on scripts/managen due to the https://github.com/curl/curl/blob/master/docs/cmdline-opts/ech.md?plain=1#L23

My wild guess is that the presence of (default) is throwing off some managen logic to translate the double quotes. Grepping through the source code, this is the first case where the docs have a pattern of ## "something" (default).

I would assume it's something simple to fix, but I'm still dealing with a packaging issue regarding multi-arch on the Debian side that I need to fix first (to get the package uploaded).

@bagder bagder self-assigned this Nov 7, 2024
bagder added a commit that referenced this issue Nov 7, 2024
Reported-by: Samuel Henrique
Fixes #15506
@bagder bagder closed this as completed in f4ee7ba Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

2 participants