From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=proton.me header.i=@proton.me header.b="Ku3Fwe5I" Received: from mail-40134.protonmail.ch (mail-40134.protonmail.ch [185.70.40.134]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 57DDB1732 for ; Fri, 8 Dec 2023 10:04:25 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=proton.me; s=protonmail; t=1702058660; x=1702317860; bh=PTmEXgI7BxufT0npzlsJT5kiKHIgc9s02WiJV5XBNQU=; h=Date:To:From:Cc:Subject:Message-ID:In-Reply-To:References: Feedback-ID:From:To:Cc:Date:Subject:Reply-To:Feedback-ID: Message-ID:BIMI-Selector; b=Ku3Fwe5Ivvvv7/4sAslKhSLbNE2c16L8W9Vp2kURUwCdrCDOtvKhiKIMRbTdK+mLb Ti7kJUTNntqDHGEdE/Z0ZzRQtCNOd1Ub7YXRTUkkM6VYDHMchM9FzkXNzQT/blk/cd VxjB0iLT27kzVChTAGy0f6kaeZxfXQjbAyOG8tlH9uJunL93YkuR+lVz0cyeuMFjM6 +cjKRCuJp7gCslhzCntRKnuKz4z9XlyqusEBM3xbN5iiNhb1niGGQIdNc/FDO37NjK BdutuscjCKkjvnVgI/ubh+oIPvQ+s8WYwvM1jJehNprfolLSsWuogbdxl87AYgN+o+ CdGM0doVGmiOQ== Date: Fri, 08 Dec 2023 18:04:01 +0000 To: Viresh Kumar , Miguel Ojeda , Alex Gaynor , Wedson Almeida Filho , Boqun Feng , Gary Guo , =?utf-8?Q?Bj=C3=B6rn_Roy_Baron?= , Andreas Hindborg , Alice Ryhl , Jonathan Corbet From: Benno Lossin Cc: Vincent Guittot , rust-for-linux@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] docs: rust: Clarify that 'rustup override' applies to build directory Message-ID: <4738ad1c-eb54-4ad6-98c8-3852de3e8fc3@proton.me> In-Reply-To: References: Feedback-ID: 71780778:user:proton Precedence: bulk X-Mailing-List: linux-doc@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 12/8/23 11:18, Viresh Kumar wrote: > Rustup override is required to be set for the build directory and not > necessarily the kernel source tree (unless the build directory is its > subdir). >=20 > Clarify the same in quick-start guide. >=20 > Signed-off-by: Viresh Kumar > --- > Documentation/rust/quick-start.rst | 9 +++++++-- > 1 file changed, 7 insertions(+), 2 deletions(-) >=20 > diff --git a/Documentation/rust/quick-start.rst b/Documentation/rust/quic= k-start.rst > index f382914f4191..a7a08955fe46 100644 > --- a/Documentation/rust/quick-start.rst > +++ b/Documentation/rust/quick-start.rst > @@ -39,8 +39,13 @@ If ``rustup`` is being used, enter the checked out sou= rce code directory > =09rustup override set $(scripts/min-tool-version.sh rustc) >=20 > This will configure your working directory to use the correct version of > -``rustc`` without affecting your default toolchain. If you are not using > -``rustup``, fetch a standalone installer from: > +``rustc`` without affecting your default toolchain. > + > +Note that the override applies to the build directory (and its sub-direc= tories). Shouldn't this be "Note that the override only applies to the current working directory (and its sub-directories)."? I think it would also be useful to continue with this: "But in order to build the kernel, this override must affect the build directory.". And then you could also mention that in the default location for the build directory is in the repository. --=20 Cheers, Benno > +If the kernel is built with `O=3D`, the override must b= e set for > +the build directory instead. > + > +If you are not using ``rustup``, fetch a standalone installer from: >=20 > =09https://forge.rust-lang.org/infra/other-installation-methods.html#sta= ndalone