lkml.org 
[lkml]   [2019]   [Jan]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v2 0/2] Revamp admin-guide/tainted-kernels.rst to make it more comprehensible
From
Date
Jonathan Corbet schrieb am 09.01.19 um 00:47:
> On Tue, 8 Jan 2019 20:40:05 +0100
> Thorsten Leemhuis <linux@leemhuis.info> wrote:
>
> OK, so we're so close... I think the patches are good; they are a welcome
> improvement to the documentation and the script looks great. Thanks for
> doing this!

More to come, but the revamp for reporting-bugs.rst that I started
already will take a little bit longer to get done, as there is much
ground to cover... (and there are three Linux conferences ahead for me
in the next few weeks, too)

> Patches applied.

thx

> Now for the "I love your patch, something to improve part" :)
> - Patch 1 didn't apply to docs-next; probably you got caught by the
> fast-forward to 5.0-rc1. I've applied it to 4.20 then merged to
> resolve the conflict in tools/Makefile.

Ugh, sorry, rechecking was on my mental todo list, but fell through the
cracks

> - I was tempted to ask you to redo the changelogs, but ended up just
> tweaking them myself.

Thx.

> Changelogs should focus on what the patch does
> and why; they aren't usually the place for the history of how the patch
> came to exist in its current form.

Looks like I got carried away a little bit :-/

> I'll append what actually got
> committed. (Filling to 72 columns or so is also a nice touch for
> changelogs).

Ohh, I thought it was 80. *makesnote*

> [...]

Ciao, Thorsten

\
 
 \ /
  Last update: 2019-01-09 12:53    [W:0.198 / U:0.024 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site