Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752244AbdGDTYS (ORCPT ); Tue, 4 Jul 2017 15:24:18 -0400 Received: from ms.lwn.net ([45.79.88.28]:56300 "EHLO ms.lwn.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752141AbdGDTYQ (ORCPT ); Tue, 4 Jul 2017 15:24:16 -0400 Date: Tue, 4 Jul 2017 13:24:13 -0600 From: Jonathan Corbet To: Linus Torvalds Cc: LKML , "open list:DOCUMENTATION" Subject: Re: [PULL] Docs for 4.13 Message-ID: <20170704132413.55274d6d@lwn.net> In-Reply-To: References: <20170703072040.7ccde29b@lwn.net> Organization: LWN.net X-Mailer: Claws Mail 3.14.1 (GTK+ 2.24.31; x86_64-redhat-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1491 Lines: 37 On Mon, 3 Jul 2017 21:32:33 -0700 Linus Torvalds wrote: > Eg things like > > Error: Cannot open file ./kernel/rcu/srcu.c > Error: Cannot open file ./kernel/rcu/srcu.c > > happen simply because that file no longer exists, and the docs never > got updated. > > So my merge didn't even try to fix those kinds of things at all. I > literally just looked at the conflicts and moved those over to the rst > files, and that was it. There's a lot of other changes that never > cause conflicts for the simple reason that those changes never caused > documentation changes to begin with. > > Now, this is obviously not new, but it does strike me that if checking > for these kinds of things was easier and part of "make allmodconfig", > then we might have less of it happen. I see Markus already tossed out a patch using the sphinx "dummy mode". It might be possible to create a dead-simple linter for this kind of thing that would be quite a bit faster, but I wonder how much we really need it. Problems like this pop up with great regularity, but they tend to be caught and fixed fairly quickly. Meanwhile, the world stubbornly refuses to end if the docs build tosses out a few (more) errors for a few days. I don't think we have to slow down everybody's build for this. (Getting something into the build-and-boot testers might not be a bad idea, though). I've committed a patch to fix this particular problem, will send it youward in a few days. jon