Skip to content

Manual pages: preprocess once, format with docbook#141

Open
asarhaddon wants to merge 1 commit into
shlomif:masterfrom
asarhaddon:docbook-man-process-once.diff
Open

Manual pages: preprocess once, format with docbook#141
asarhaddon wants to merge 1 commit into
shlomif:masterfrom
asarhaddon:docbook-man-process-once.diff

Conversation

@asarhaddon
Copy link
Copy Markdown
Contributor

No description provided.

@asarhaddon asarhaddon mentioned this pull request Apr 13, 2026
@shlomif
Copy link
Copy Markdown
Owner

shlomif commented Apr 14, 2026

@asarhaddon : hi. I dislike the fact that the generating code was converted from perl5 to sh. i also prefer named arguments over positional ones. i may have other comments.

@asarhaddon asarhaddon force-pushed the docbook-man-process-once.diff branch from ff67996 to e88b1d7 Compare April 14, 2026 16:00
@shlomif
Copy link
Copy Markdown
Owner

shlomif commented May 11, 2026

This pull req will break compatibility with fedora. @asarhaddon

@asarhaddon
Copy link
Copy Markdown
Contributor Author

asarhaddon commented May 12, 2026 via email

@shlomif
Copy link
Copy Markdown
Owner

shlomif commented May 15, 2026

This pull req will break compatibility with fedora.
Could you be please describe the issue, so that I can try and fix it without duplicating your investigations first?

fedora's fortune binary rpm ships without the offensive fortunes and wo the -o flag.

@asarhaddon
Copy link
Copy Markdown
Contributor Author

asarhaddon commented May 18, 2026 via email

@shlomif
Copy link
Copy Markdown
Owner

shlomif commented May 27, 2026

@asarhaddon : hi. I dislike the fact that the generating code was converted from perl5 to sh. i also prefer named arguments over positional ones. i may have other comments.

Given these two issues seem to be the raison d'etre of this merge-request, I don't see a point to merging it.

@asarhaddon : what do you think?

( I appreciate the previous merge-requests you sent.)

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