public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Randy Dunlap <randy.dunlap@oracle.com>
To: Rob Landley <rob@landley.net>
Cc: linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org
Subject: Re: [patch] tweak make htmldocs (nochunks and better index).
Date: Wed, 30 May 2007 12:28:15 -0700	[thread overview]
Message-ID: <20070530122815.31ca9587.randy.dunlap@oracle.com> (raw)
In-Reply-To: <200705301447.41421.rob@landley.net>

On Wed, 30 May 2007 14:47:41 -0400 Rob Landley wrote:

> On Wednesday 30 May 2007 12:18 am, Randy Dunlap wrote:
> > What is <\p> supposed to do?
> 
> Sorry, typo.  Should be </p> instead.  My bad.
> 
> As to what it does in a larger sense, "make the xml people happy".  Closing
> parentheses tags make the various xhtml validators shut up.  The main
> difference between xhtml and html is that you close all the tags you open.  I
> can yank it if people prefer it that way.
> 
> > Here's what I see in index.html (beginning lines of it):
> > 
> > Linux Kernel HTML Documentation
> > Kernel Version: 2.6.22-rc3
> > 
> > deviceiobookBus-Independent Device Accesses <\p>
> 
> Yeah, that's screwed up.  (It looked fine in Konqueror. :)  *rummage*...
> 
> Here's an updated patch.  This time I checked that it looked ok in Firefox
> too:

Yes, /me too.  Thanks.

I still disagree on the use of "nochunks" though.

> --- git/Documentation/DocBook/Makefile	2007-05-23 16:36:56.000000000 -0400
> +++ work/Documentation/DocBook/Makefile	2007-05-30 14:36:49.000000000 -0400
> @@ -141,9 +141,12 @@
>  		   cat $(HTML) >> $(main_idx)
>  
>  quiet_cmd_db2html = HTML   $@
> -      cmd_db2html = xmlto xhtml $(XMLTOFLAGS) -o $(patsubst %.html,%,$@) $< && \
> -		echo '<a HREF="$(patsubst %.html,%,$(notdir $@))/index.html"> \
> -        $(patsubst %.html,%,$(notdir $@))</a><p>' > $@
> +      cmd_db2html = xmlto xhtml-nochunks $(XMLTOFLAGS) -o $(patsubst %.html,%,$@) $< && \
> +		NAME='$(patsubst %.html,%,$(notdir $@))'; \
> +		echo -n "<p><a HREF=\"$$NAME/$$NAME.html\">$$NAME</a> - " >$@; \
> +		sed -nre 's@.*<title>(.*)</title>.*@\1@p' \
> +			"Documentation/DocBook/$$NAME/$$NAME.html" >> $@ ; \
> +		echo '</p>' >> $@
>  
>  %.html:	%.xml
>  	@(which xmlto > /dev/null 2>&1) || \
> 
> Rob

---
~Randy
*** Remember to use Documentation/SubmitChecklist when testing your code ***

  reply	other threads:[~2007-05-30 19:30 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-29 19:28 [patch] tweak make htmldocs (nochunks and better index) Rob Landley
2007-05-30  4:18 ` Randy Dunlap
2007-05-30 18:47   ` Rob Landley
2007-05-30 19:28     ` Randy Dunlap [this message]
2007-05-30  5:12 ` Randy Dunlap

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20070530122815.31ca9587.randy.dunlap@oracle.com \
    --to=randy.dunlap@oracle.com \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rob@landley.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox