[ docbook-Bugs-1928775 ] man: wrong conversion of non-breaking spaces

[SourceForge.net <no...@so...>]

Bugs item #1928775, was opened at 2008-03-30 01:00
Message generated for change (Settings changed) made by xmldoc
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=373747&aid=1928775&group_id=21935

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: XSL
Group: output: manpages
>Status: Pending
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Daniel Leidert (dleidert)
Assigned to: Michael(tm) Smith (xmldoc)
Summary: man: wrong conversion of non-breaking spaces

Initial Comment:
A user reported this bug to the Debian BTS:

This bug causes wrong formatting in the French manpages of (at least)
aptitude or shadow (e.g. "«ptitude install..." or "«\fIYou have new mail. »")

Currently, the non breaking spaces (0x00a0) are converted to the groff
sequence "\ " in other.xsl (xsl:template name="convert.nobreak-space").

Unfortunately, the space following '\' is removed, and bad groff sequences
are generated.


The non breaking spaces are also converted in charmap.groff.xsl, and I
propose to remove the conversion done in other.xsl, and keep the one in
charmap.groff.xsl (this one works correctly).

I got good results with the shadow manpages applying the attached patch.

http://bugs.debian.org/473244

I'm too busy to test and verify it atm. aptitude AFAIK belongs to your regression repository. Maybe you could check it yourself.

----------------------------------------------------------------------

>Comment By: Michael(tm) Smith (xmldoc)
Date: 2009-02-10 03:02

Message:
moving to status=Pending per latest comment from nekral. 

----------------------------------------------------------------------

Comment By: Nicolas François (nekral)
Date: 2008-03-31 00:13

Message:
Logged In: YES 
user_id=467217
Originator: NO

Hello Michael,

I'm the submitter of the original Debian bug, and I had a look at the fix
you committed in revision 7806.

It looks OK to me (I could check it with aptitude and shadow), and I agree
that you close this bug.

----------------------------------------------------------------------

Comment By: Michael(tm) Smith (xmldoc)
Date: 2008-03-30 14:00

Message:
Logged In: YES 
user_id=118135
Originator: NO

I don't have the DocBook source for the aptitude man page. I tried
apt-get'ting the aptitude source package, but I get "E: Unable to find a
source package for aptitude". I do have the shadow pages and can test with
those.

Note that I made a recent change, r7806, that probably fixed this already.
Anyway, I take a look and see.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=373747&aid=1928775&group_id=21935