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

trurl.md: the manpage in markdown #337

Merged
merged 1 commit into from
Aug 27, 2024
Merged

trurl.md: the manpage in markdown #337

merged 1 commit into from
Aug 27, 2024

Conversation

bagder
Copy link
Member

@bagder bagder commented Aug 27, 2024

Switch format for the main manpage to markdown for convenience - make it easier to work with.

This version can be converted to an nroff manpage using the cd2nroff script in the curl repository.

Switch format for the main manpage to markdown for convenience - make it
easier to work with.

This version can be converted to an nroff manpage using the cd2nroff
script in the curl repository.
@bagder
Copy link
Member Author

bagder commented Aug 27, 2024

This also makes it much easier to browse the documentation directly on GitHub.

trurl returns a non-zero exit code to indicate problems.
.IP 1

## 1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These headings seem really big for the amount of text associated with them
Screenshot 2024-08-27 at 9 09 57 AM

Especially compared with the manpage
Screenshot 2024-08-27 at 9 10 34 AM

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

markdown is somewhat limited in what kind of formatting it offers. This is the best mapping I have managed to make. The format is tuned so that the tool can generate a fine manpage from it.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, that makes sense.

@bagder bagder merged commit bada531 into master Aug 27, 2024
11 checks passed
@bagder bagder deleted the bagder/markdown-manpage branch August 27, 2024 13:40
@bagder
Copy link
Member Author

bagder commented Aug 27, 2024

The web version of the manpage is now rendered from the markdown.

@abbbi
Copy link
Contributor

abbbi commented Aug 28, 2024

hi,

im looking into packaging the latest version for debian and this makes it a little hard for me.
The cd2nroff executable is not shipped by any debian package, so to include a correct manpage during the trurl package build becomes more complicated than it must be. Currently i dont see a sane way to ship the manpage other by creating a patch for the debian package which adds the manpage pre-rendered by myself.

Would it possible for further releases to ship the manpage pre-rendered or are there other tools i could use to get decent output (i tried with go-md2man which is in debian, but the result is bad)

@bagder
Copy link
Member Author

bagder commented Aug 28, 2024

Yes. When I changed the format I figured this would become something people will want. A fair request I think.

I will make sure to render an nroff version of the manpage and generate a tarball with that file inside in future trurl releases.

If you want a "quick fix" for right now, the nroff converted version of trurl.md is available at https://curl.se/trurl/trurl.1

@abbbi
Copy link
Contributor

abbbi commented Aug 28, 2024

I will make sure to render an nroff version of the manpage and generate a tarball with that file inside in future trurl releases.

thank you, that would really ease up things for package maintainers.

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