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

["Justin P. Mattock" <justinmattock@gmail.com>]

On 09/03/2010 02:13 AM, Finn Thain wrote:
>
> 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
>

so the safest is to just ignore the column limit for urls to make things 
legit.

Justin P. Mattock