Re: Fwd: [PATCH 35/91] arch/m68k/q40/README Update broken web addresses in the arch directory.

[Finn Thain <fthain@telegraphics.com.au>]

On Fri, 3 Sep 2010, Justin P. Mattock wrote:

> On 09/03/2010 01:47 AM, Finn Thain wrote:
> > 
> > On Fri, 3 Sep 2010, Justin P. Mattock wrote:
> > 
> > > On 09/03/2010 01:23 AM, Finn Thain wrote:
> > > > 
> > > > URLs can't be split across lines usefully, so disregard the 
> > > > checkpatch complaint.
> > > > 
> > > > Finn
> > > > 
> > > 
> > > with the coding style, what files should ignore coding style with 
> > > the tab rule i.e. Kconfig *.c, *.h, README, etc... (from here 
> > > Kconfig make sense due to the ncurses interface..)
> > 
> > I'm not sure what you mean (?)
> > 
> > Kconfig seems to use the<...> syntax for URLs. checkpatch doesn't 
> > complain about the ~90 character long URL in ./drivers/scsi/Kconfig...
> > 
> > Finn
> > 
> 
> yeah this is a bit confusing, especially with checkpatch in the mix.. So 
> if I understand correctly, using checkpatch, any address that uses the 
> column limit: (example)
> + http://web.archive.org/web/*/http://www.unix-ag.org/Linux-Alpha/
> + Architectures/Avanti.html.
> 
> breaks checkpatch(without the <.....>) in any of the files? as for 
> Kconfig make the url long so the she looks pretty for the user..

I don't know whether checkpatch special-cases Kconfig or angle brackets. 
You'd have to put it to the test. But it is not important.

The point is, you can't line-wrap URLs without breaking software that 
looks for them (text editors, browsers, terminal emulators ... your own 
scripts too probably).

Finn