bugSystem V style init programs - Bugs: bug #33023, typo in man page of...

 
 

bug #33023: typo in man page of man8/shutdown.8.bz2 of sys-apps/sysvinit-2.88-r1

Submitted by:  None
Submitted on:  Fri 08 Apr 2011 10:04:33 AM UTC  
 
Category: NoneSeverity: 2 - Minor
Item Group: NoneStatus: Works For Me
Privacy: PublicAssigned to: None
Open/Closed: Open

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

(Jump to the original submission Jump to the original submission)

Fri 29 Apr 2011 10:57:22 PM UTC, comment #12:

my understanding of man-db is that it automatically takes care of detecting and recoding (when necessary) for the active locale. so is declaring UTF8 explicitly even necessary ?

for the patch in question, `sed -i` isnt portable. you'd want to run sed into a local temp file and then install that into the final location. i think autoconf also takes care of setting up SED, so you might want to use $(SED) or @SED@ in the makefile instead.

Mike Frysinger <vapier>
Wed 27 Apr 2011 11:11:41 AM UTC, comment #11:

the attached patch solved the issue :-)

Toralf Förster <tfoerste>
Mon 18 Apr 2011 02:08:21 PM UTC, comment #10:

Please give the attched patch a try

(file #23267)

Dr. Werner Fink <wfink>
Project Administrator
Fri 15 Apr 2011 09:56:22 PM UTC, comment #9:

looking at the oldest man currently available (released in mid-2005), it has the same behavior. so this isnt anything new.

Mike Frysinger <vapier>
Fri 15 Apr 2011 11:32:20 AM UTC, comment #8:

There are two type of comments in roff, normal comments
which starts with

dot backslash double quote

and in the very first line with

single quote backslash double quote

both are ignored by roff, but the later one is parsed by
both man and man-db to search for the use preprocessors
marked with their starting letter like eqn (e), grap (g),
pic (p), and tbl (t) ... and further man-db search for
its coding tag to be able to detect the used coding of the
manual page. With this tag it is possible to detect
the encoding regardless where the manual page is located.

Now it seems that the cannonical man program does not ignore
unknown tags or letters but simply show a warning message.

Dr. Werner Fink <wfink>
Project Administrator
Fri 15 Apr 2011 10:18:03 AM UTC, comment #7:

i would have thought man-db parsed that as well. is there a "man" spec which could be reviewed to see what the expected behavior for apps ? is this tag parsed by *roff, or simply ignored as a comment ?

my understanding of man-db is that it transparently takes care of converting encodings to whatever the user currently has active.

if sysvinit has always had this tag, i wonder if the parsing is something new to canonical man. i'll have to dig up the older versions to see what it's doing.

Mike Frysinger <vapier>
Tue 12 Apr 2011 09:05:56 AM UTC, comment #6:

... the editor form has eaten the backslashes from the
sed command line 8)

Dr. Werner Fink <wfink>
Project Administrator
Tue 12 Apr 2011 08:46:05 AM UTC, comment #5:

Hmmm ... the question rises: why does the canonical man simply
not ignore this information tag of man-db in the preprocessor
line?

Both Debian and openSUSE are using man-db and it seems that
the tag for man-db was added to all of the manual pages
(beside fstab-decode.8).

For me this looks like this was already part of revision 1
before Savannah, that is the original tar ball from 2009-11-20.

Simply solution would be something like

sed -ri "1{ s@[[:blank:]]-\-[[:blank:]][^\][[:blank:]]-\-@@; }" man/.[1-8]

maybe based on a configure option

Dr. Werner Fink <wfink>
Project Administrator
Mon 11 Apr 2011 07:25:18 PM UTC, comment #4:

it's the very first line. `man` looks at all lines that start with the four chars:
'\"
and uses those as directives for how to invoke the relevant roff

but the request:
-- coding: UTF-8 --

is not recognized by `man` thus it emits the error in question.

also as i said, the commands you listed are not relevant to groff because the error in question isnt coming from groff. it's coming from `man`. thus there is nothing specific to Gentoo that is causing this issue.

i imagine the man package you're using is actually "man-db" rather than the canonical "man" which is why you arent seeing the output we are.

Mike Frysinger <vapier>
Mon 11 Apr 2011 08:06:20 AM UTC, comment #3:

Which directive is specific to the man-db package?
Beside this piping the manual page with cat through
nroff show no errors here. That is that all the
command in my comment #1 work flawless.

Dr. Werner Fink <wfink>
Project Administrator
Mon 11 Apr 2011 12:44:40 AM UTC, comment #2:

the error isnt coming from groff. the shutdown(8) man page is using a directive that is specific to the man-db package.

Mike Frysinger <vapier>
Fri 08 Apr 2011 10:27:37 AM UTC, comment #1:

I do not see any error nor any warning message[1], maybe the
(g)roff installation and/or your locale setup is broken.
Beside this you're talking about a typobut only cite
the message: which typo do you meant?

IMHO you should report this on the bugzilla of Gentoo
and asign this to the groff maintainer at Gentoo.

[1] Tested with

man 8 shutdown

and in the source tree with

cat man/shutdown.8 | nroff -mandoc | less -R
cat man/shutdown.8 | nroff -man | less -R
cat man/shutdown.8 | nroff -mandb | less -R

where the last roff macro package mandb is only
usabel for the mandb package used here.

Sidemark: `nroff' == `groff -T utf8'

Dr. Werner Fink <wfink>
Project Administrator
Fri 08 Apr 2011 10:04:33 AM UTC, original submission:

Under an almost stable Gentoo I get this (probably harmless) error message :

tfoerste@n22 ~ $ man shutdown
Error parsing *roff command from file /usr/share/man/man8/shutdown.8.bz2

B/c the likelihood that the *roff preprocessor under Gentoo has a problem (only) with this particular file, I file this bug report here first.

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #23267:  sysvinit-mandb.patch added by wfink (2kB - text/x-diff - patch to remove mandb tag on others than Debian or openSUSE)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by tfoerste (Posted a comment)
  • -unavailable- added by vapier (Posted a comment)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 18 Apr 2011 02:08:21 PM UTCwfinkAttached File-=>Added sysvinit-mandb.patch, #23267
    Fri 08 Apr 2011 10:28:31 AM UTCwfinkStatusNone=>Works For Me
    Fri 08 Apr 2011 10:27:37 AM UTCwfinkSeverity3 - Normal=>2 - Minor

    Back to the top


    Powered by Savane 3.1-cleanup