Skip to content

man: Fix search-query link on doveadm fetch page#1540

Merged
slusarz merged 1 commit into
dovecot:mainfrom
slusarz:doveadm_fetch_man_link
May 17, 2026
Merged

man: Fix search-query link on doveadm fetch page#1540
slusarz merged 1 commit into
dovecot:mainfrom
slusarz:doveadm_fetch_man_link

Conversation

@slusarz
Copy link
Copy Markdown
Contributor

@slusarz slusarz commented May 15, 2026

No description provided.

@cmouse
Copy link
Copy Markdown
Contributor

cmouse commented May 16, 2026

Huh.. why does it need two commas?

@slusarz
Copy link
Copy Markdown
Contributor Author

slusarz commented May 16, 2026

https://github.com/dovecot/documentation#man-pages

First entry is name, second entry is hash link, third entry is section number. Right now the link is being created to the wrong page URL with a hash of "#7". See https://doc.dovecot.org/2.4.4/core/man/doveadm-fetch.1.html#arguments

@cmouse
Copy link
Copy Markdown
Contributor

cmouse commented May 16, 2026

I've got a feeling this might not be the only wrong link.

@slusarz
Copy link
Copy Markdown
Contributor Author

slusarz commented May 16, 2026

I verified it's the only wrong link for search-query. And it's practically the only man page that could be broken (since all other man pages are section 1, which is the default). But I can create some kind of grep regex to check.

Update: Confirmed - there was one additional section 7 man page in CE (pigeonhole), and one in Pro, and all man links to those pages are correctly formatted.

@slusarz slusarz merged commit 9fe50bb into dovecot:main May 17, 2026
5 checks passed
@slusarz slusarz deleted the doveadm_fetch_man_link branch May 17, 2026 00:17
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