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] Switch to pandoc and remove generated files #1955

Merged
merged 5 commits into from
Mar 5, 2024

Conversation

lbonn
Copy link
Collaborator

@lbonn lbonn commented Mar 2, 2024

No description provided.

@lbonn lbonn marked this pull request as ready for review March 5, 2024 13:19
@lbonn
Copy link
Collaborator Author

lbonn commented Mar 5, 2024

Oops this whole lua transformation needs pandoc 2.17 and 22.04 is stuck on 2.9...

This method has the advantage of giving a good layout on manpages, github preview and mkdocs. But the lua filter is a bit complicated.

I also tried to rewrite the man files with definition lists but they don't render very well. Mkdocs is supposed to handle them but in practice it breaks if a code block is nested inside a definition.

Solutions?

  • wait for github to have 24.04 as ubuntu-latest...
  • #### for command definitions, not as pretty in makefiles
  • graceful degradation if pandoc < 2.17

EDIT: went with the graceful degradation!

@lbonn
Copy link
Collaborator Author

lbonn commented Mar 5, 2024

 ERROR: files left in build directory after distclean:
./=

hmm I guess something is wrong with my autoconf. The file appears when running ./configure

EDIT: yes it was the wrong version check for pandoc. Gave up on it...

* convert existing markup to use definition list (a bit hacky...)
* rewrite title to pandoc man metadata
* reduce heading nesting to match most man pages
* inline code displayed as bold, also common man style
- autoconf:
  * generate in source directory if pandoc found
  * install from source directory

- meson:
  * generate and install with build directory if pandoc found
  * try to install from source directory if pandoc not found
  * bail otherwise
Man pages look better with pandoc >= 2.17
@DaveDavenport DaveDavenport merged commit 433f966 into davatorium:next Mar 5, 2024
7 checks passed
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.

None yet

2 participants