Re: [PATCH] Minor fix to Documentation/powerpc/00-INDEX

[Rob Landley]

On Saturday 11 August 2007 4:33:34 pm Randy Dunlap wrote:
> On Thu, 9 Aug 2007 23:42:35 -0500 Rob Landley wrote:
> > Signed-off-by: Rob Landley <rob@xxxxxxxxxxx>
> >
> > I have a python script to convert 00-INDEX files into index.html files,
> > and a second script to show 404 errors in the result as well as
> > files/directories nothing links to.   (It's not very useful yet, but in
> > case you're wondering http://kernel.org/doc/docdiridx.py and
> > http://kernel.org/doc/doclinkcheck.py .)
> >
> > Anyway, my simple index.html generator breaks on the
> > Documentation/powerpc directory because one of the description lines is
> > two lines long.  This patch joins those two lines together into one line.
> >  This is the only instance (so far) of this problem.
>
> If Paul wants to merge this, then OK, but I'm not aware of any rule
> that the file descriptions inside INDEX files must be only one line
> long... is that documented somewhere?  (if so, where?)

Documentation/00-INDEX line 5:

> Please try and keep the descriptions small enough to fit on one line.

There was only one instance of it not being the case.  I can use indentation 
level instead (which is what Kconfig uses to identify help text, so I'm not 
leaking pythonisms into the kernel that aren't already there), but the one 
line thing seemed to be an existing standard.

I'm also relying on the first file in the list being documentation for 
00-INDEX itself.  (If it breaks, I'll either try to derive a new heuristic 
and change the script, or submit a patch to change the index.  That one would 
be a more difficult heuristic to replace, though.)

> Maybe the script should allow for this?

*shrug*  I can go either way on that, but people get all funny about 
whitespace being significant and the one line thing wasn't my idea. :)

Rob
-- 
"One of my most productive days was throwing away 1000 lines of code."
  - Ken Thompson.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/