Page MenuHomeFreeBSD

Typesetting fixes for arch(7).
ClosedPublic

Authored by kib on Jul 20 2016, 12:57 PM.
Tags
None
Referenced Files
F103210481: D7261.id18596.diff
Fri, Nov 22, 6:47 AM
Unknown Object (File)
Thu, Nov 21, 10:32 PM
Unknown Object (File)
Thu, Nov 21, 10:30 PM
Unknown Object (File)
Wed, Nov 13, 8:03 AM
Unknown Object (File)
Thu, Nov 7, 5:18 PM
Unknown Object (File)
Thu, Nov 7, 5:18 PM
Unknown Object (File)
Tue, Nov 5, 7:29 PM
Unknown Object (File)
Tue, Nov 5, 2:10 AM
Subscribers

Details

Reviewers
emaste
bapt
Group Reviewers
manpages
Commits
rS303134: Improve typesetting.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kib retitled this revision from to Typesetting fixes for arch(7)..
kib updated this object.
kib edited the test plan for this revision. (Show Details)
kib added a reviewer: emaste.
kib set the repository for this revision to rS FreeBSD src repository - subversion.

I got this from mdoc(7),

The table:

      String    Nroff    Troff
      <=        <=       ≤
      >=        >=       ≥

was produced by:

.Bl -column -offset indent ".Sy String" ".Sy Nroff" ".Sy Troff"
.It Sy String Ta Sy Nroff Ta Sy Troff
.It Li <= Ta <= Ta \*(<=
.It Li >= Ta >= Ta \*(>=
.El

There are existing man pages with both ".Sy (e.g. share/man/man1/builtin.1) and "Sy, so either way we should do a pass to make them consistent.

There are existing man pages with both ".Sy (e.g. share/man/man1/builtin.1) and "Sy, so either way we should do a pass to make them consistent.

GNU groff with our mdoc is unhappy:

$ troff -Tdvi -mdoc share/man/man7/arch.7 | grodvi >|~/tmp/arch.7.dvi
Usage: .Sy symbolic_text ... (#50)
Usage: .Sy symbolic_text ... (#69)
$
bapt added inline comments.
share/man/man7/arch.7
143 ↗(On Diff #18589)

Should be

.Fx

instead of hard coded FreeBSD

share/man/man7/arch.7
136 ↗(On Diff #18589)

Did you want to also clarify this is available only when sys/endian.h is included?

kib edited edge metadata.

Consistently use "Sy text" for symbolic. Use .Fx instead of "FreeBSD".

share/man/man7/arch.7
136 ↗(On Diff #18589)

I want to leave the scope of this change to typesetting only.

Note that sys/endian.h is not the only such problem in the text. E.g. __ARM_ARCH requires machine/asle.h for older compilers.

@wblock, should mdoc.7 be updated to remove the . from the example I pasted?

@wblock, should mdoc.7 be updated to remove the . from the example I pasted?

As far as I know, mdoc(7) in FreeBSD 10 is the old one, and mdoc(7) in FreeBSD 11 and later is the new one from mandoc. I don't know if they contain the same example, or what is going on with the versions, really.

The example looks correct to me. It's just that mdoc understands macros without the dot in some contexts.

There are existing man pages with both ".Sy (e.g. share/man/man1/builtin.1) and "Sy, so either way we should do a pass to make them consistent.

I do not see any Sy in builtin.1, but our style is to only have the '.' in column 1.

It's just that mdoc understands macros without the dot in some contexts.

It stays in macro context until the end of the line.

I mdoc format as properly stated in the mdoc(7) manpage in head

macros does not start with a '.'
the lines started with a '.' are special:

In an mdoc document, lines beginning with the control character ‘.’ are called “macro lines”.  The first word is the macro.
bapt added a reviewer: bapt.
This revision is now accepted and ready to land.Jul 21 2016, 6:06 AM
This revision was automatically updated to reflect the committed changes.