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.

Sign up for GitHub

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

Jump to bottom

[regression] Non-defined PI macro emitted with -Tman #111

Closed
paravoid opened this issue Jan 8, 2023 · 4 comments
Closed

[regression] Non-defined PI macro emitted with -Tman #111

paravoid opened this issue Jan 8, 2023 · 4 comments

Comments

@paravoid
Copy link
Contributor

paravoid commented Jan 8, 2023 

$ lowdown -s -Tman regress/simple.md  | preconv -e UTF-8 | groff -Tutf8 -man -w w > /dev/null 
troff: <standard input>:183: warning: macro 'PI' not defined

I believe this was introduced with 4780534.
@kristapsdz
Copy link
Owner

Yes, seems that's an -ms macro, not a -man. Thanks!

@paravoid
Copy link
Contributor Author

Thanks for looking into it! So is the fix reverting the aforementioned commit or something else? I'd like to backport the fix to this (either in isolation, or bundled with other bugfixes) to Debian before the bookworm release!

kristapsdz added a commit that referenced this issue Mar 28, 2023
@kristapsdz
Don't use \(PI for -tman: it doesn't exist.
02491bf 
The \(PI register only exists for -ms macros.  Use the default value
of 5n for this (for definition lists) when emitting for -tman.

References #111
@kristapsdz
Copy link
Owner

Does the above commit fix this issue for you? I use the width set by default in groff (5n) instead of the PI register. If so, it'll be rolling into the next release, which I'll be doing in a day or so.

@paravoid
Copy link
Contributor Author

Yes it does!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kristapsdz @paravoid