From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: ARC-Seal: i=1; a=rsa-sha256; t=1525901258; cv=none; d=google.com; s=arc-20160816; b=w/1rF97qhhG2fv4NDwF7dW3lB//itHqSWqz+z+j0p7KA2ZOJ8bQbANRHuCFfFT+rUM 38aUshB/0ogEqFQtHe/Dz5vCM4gM4nP3Nkjs5fuvDosM4bQvraNnkBZjNidzXiDB9Wfv 6Sv7TMbNKbT8KnNke6UFd66RqXwN4PIseqzgNlNIlDd1SROpqaf/NXCi7h4oMi5IyDGI E9XD1hQY/IG3YZgvDltnMtHBqKmuXoTKCTI2l6kpMkm35wBzKYrrqJFQHeyHEUUz/Ljr kCWr57fnZCj1D7kmuO+79G330SIVUAnnwup9e+hO+nlCpSLDxMqqzx1Mv6zaJhgOI+5j NzjA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=user-agent:in-reply-to:content-disposition:mime-version:references :message-id:subject:cc:to:from:date:dkim-signature :arc-authentication-results; bh=l0JvjXNDoFEvAS1HHbq0IQeq6hyEO9taiaSHHg2zf8M=; b=z0KtRNq2uesRPr2CyAGvQR/ZSG+sVmV05wr/Rxic2boHfkI6MFvZKQhYevxmpngTg2 seocQp9FhUYMdxzhNgnqUwdnqCG3ltGrFGDlNlyQUbhS1jnHPX5QjxUjQsbRKT5VOvPN hQwvQQPolxkAEVDG3+3HD+eTxkIN4yTlQpbu5QlF2fu+R8RikDqhWGAdIFj83R2IQ85h 53K2PhBmfZPsUPklbxVBTXYkB9gFsdTM6pfB28pNeHdPuf/gAv4sp8lvT92YsChy2nBy A7bnLtCQM1b7uBUH4t7sDioXxDNLQscOcTnHQkZf6KkRBrHhmcFsDvn7sFaz4XGl3K8J zWhA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=XiRnI7xq; spf=pass (google.com: domain of parri.andrea@gmail.com designates 209.85.220.65 as permitted sender) smtp.mailfrom=parri.andrea@gmail.com; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=XiRnI7xq; spf=pass (google.com: domain of parri.andrea@gmail.com designates 209.85.220.65 as permitted sender) smtp.mailfrom=parri.andrea@gmail.com; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com X-Google-Smtp-Source: AB8JxZo/MG1WyFDmkwc4H5iVDioVdQ2bNE+iQwb74Ju/TvwJrp/30l5/MmE9fefRtmAr9PmZQGbnow== Date: Wed, 9 May 2018 23:27:28 +0200 From: Andrea Parri To: Mauro Carvalho Chehab Cc: Alan Stern , Linux Doc Mailing List , Mauro Carvalho Chehab , linux-kernel@vger.kernel.org, Jonathan Corbet , Dmitry Torokhov , Rob Herring , Mark Rutland , Linus Walleij , Ananth N Mavinakayanahalli , Anil S Keshavamurthy , "David S. Miller" , Masami Hiramatsu , Mathieu Poirier , Alexander Shishkin , Harry Wei , Russell King , Catalin Marinas , Will Deacon , Michal Simek , Jeff Dike , Richard Weinberger , Guan Xuetao , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , x86@kernel.org, Chris Zankel , Max Filippov , Jens Axboe , David Howells , David Woodhouse , Herbert Xu , Arnd Bergmann , Greg Kroah-Hartman , Michael Turquette , Stephen Boyd , David Airlie , Gustavo Padovan , Maarten Lankhorst , Sean Paul , Pali =?iso-8859-1?Q?Roh=E1r?= , Matias Bjorling , Michael Lyle , Kent Overstreet , Malcolm Priestley , Antoine Jacquet , Sudip Mukherjee , Steven Rostedt , "Paul E. McKenney" , Josh Triplett , Mathieu Desnoyers , Lai Jiangshan , Boris Ostrovsky , Juergen Gross , Li Zefan , Paul Moore , Stephen Smalley , Eric Paris , James Morris , "Serge E. Hallyn" , Jaroslav Kysela , Takashi Iwai , Peter Zijlstra , Boqun Feng , Nicholas Piggin , Jade Alglave , Luc Maranget , Akira Yokosawa , Arnaldo Carvalho de Melo , Jiri Olsa , Namhyung Kim , linux-input@vger.kernel.org, devicetree@vger.kernel.org, linux-gpio@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@zh-kernel.org, user-mode-linux-devel@lists.sourceforge.net, user-mode-linux-user@lists.sourceforge.net, linux-xtensa@linux-xtensa.org, linux-block@vger.kernel.org, keyrings@vger.kernel.org, linux-crypto@vger.kernel.org, linux-clk@vger.kernel.org, dri-devel@lists.freedesktop.org, patches@opensource.cirrus.com, linux-bcache@vger.kernel.org, linux-usb@vger.kernel.org, devel@driverdev.osuosl.org, xen-devel@lists.xenproject.org, cgroups@vger.kernel.org, selinux@tycho.nsa.gov, linux-security-module@vger.kernel.org, alsa-devel@alsa-project.org Subject: Re: [PATCH v2 09/11] docs: Fix some broken references Message-ID: <20180509212728.GA4178@andrea> References: <20180509152045.722f0c51@vento.lan> <20180509164153.5b9ab9db@vento.lan> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180509164153.5b9ab9db@vento.lan> User-Agent: Mutt/1.5.24 (2015-08-30) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: =?utf-8?q?1599992730865491295?= X-GMAIL-MSGID: =?utf-8?q?1600023438064515608?= X-Mailing-List: linux-kernel@vger.kernel.org List-ID: On Wed, May 09, 2018 at 04:41:53PM -0300, Mauro Carvalho Chehab wrote: > Em Wed, 9 May 2018 15:11:07 -0400 (EDT) > Alan Stern escreveu: > > > On Wed, 9 May 2018, Mauro Carvalho Chehab wrote: > > > > > Em Wed, 9 May 2018 19:15:01 +0200 > > > Andrea Parri escreveu: > > > > > > > tools/memory-model/README | 10 +++++----- > > > > > > > > As mentioned in the previous thread, I am for keeping the current > > > > references: the REAMDE is listing the doc files, as well as other > > > > files in tools/memory-model/, relatively to that directory. > > > > > > Yeah, at least this hunk deserves some rework, as now some > > > references are Documentation/.../foo, while others are just > > > bar. > > > > > > As on (almost) all other places (except for tools/memory-model/README), > > > the references are always from the main directory, I would make all > > > patches there also relative to main dir. If you're afraid of > > > not being too clearer, we could prefix all of them with something > > > like: > > > > > > ${LINUX}/tools/memory-model/... > > > > > > just like some DT binding files do: > > > > > > Documentation/devicetree/bindings/sound/audio-graph-card.txt:see ${LINUX}/Documentation/devicetree/bindings/graph.txt > > > > > > A bonus of doing that is that the broken reference detect script can > > > keep parsing it without changes (well, it wouldn't be hard to make > > > it also accept a relative file, but doing that just due to > > > tools/memory-model/README seems overkill). > > > > > > Another advantage is that it would allow to easily add references > > > there from the main kernel Documentation, if needed in the future, > > > without messing with local x non-local relative namespace. > > > > How about changing the relative references so that something like > > Documentation/recipes.txt becomes ./Documentation/recipes.txt? > > Sounds a good idea to me. Andrea? Counterintuitive ;-) Documentation/abc.txt vs. ./Documentation/abc.txt But let me sleep over this ;-) for the time being, I renew my vote for "just don't commit these changes (and keep checking automated changes)". Andrea > > Thanks, > Mauro