Re: [PATCH] Documentation: use "spurious .sp" XSLT if DOCBOOK_SUPPRESS_SP is set

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2009 Apr 1, at 05:34, Jeff King wrote:
On Wed, Apr 01, 2009 at 03:50:34AM -0500, Chris Johnsen wrote:

The key revisions in the docbook SVN repo seem to be 5144 (before
docbook-xsl 1.69.1) and 6359 (before docbook-xsl 1.71.1).

Testing done with asciidoc 8.3.1 and docbook-xsl 1.74.0.

In the course of your SVN research, did you find the fixes between
1.73.1 and 1.74.3 that fixed the spacing issue? If so, I wonder if it's
worth backporting that fix to DOCBOOK_FIX_LIST_SPACING.

I guess you are referring to an issue different from the one created by using the "spurious .sp" simpara template, but I am not familiar with another one. If not, then I am confused. The new patch to avoid using the "spurious .sp" template fixes the list spacing in pu's git- cvsimport.1 when I generate it here (using docbook-xsl 1.74.0). For example, the extra blank line after "Problems related to timestamps:" goes away and a new blank line is inserted before "Problems related to branches:".

My poking around in the docbook SVN repo was largely limited to the manpages/block.xsl file since that is where the normal simpara template lives. If this other issue is list specific, it seems likely that fixes would be in manpages/lists.xsl. It looks like there have only been around ten commits to that lists.xsl since 1.73.1, but none of them jumped out at me as likely culprits unless the spacing you mean is indentation or "bullet"-to-text spacing (though my brain is tired right now).

--
Chris
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux