From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f74.google.com (mail-wm1-f74.google.com [209.85.128.74]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E5C1B31353D for ; Mon, 10 Nov 2025 13:34:54 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.74 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762781697; cv=none; b=JiKU/cND6mwOtBqiSbwZV0+Qa3+Up6ooWYcyr7ZpS6lD6LbLteCG+9jNNO7DZh90wX64qlmvc9xyk2n42ePItANmDOv+cE/7T5iL1C95WWsTxmnhlsZIN7SZkNwZmlfI0zRMKN3wFS7CQKCBJYhBDLKQ2v8pgNgqGrktsMOrYGM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762781697; c=relaxed/simple; bh=hOoHELQJUct+/jjBWDM/iGci2iRQNFOKXIkYMSUGeuo=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=hjdVR2ykpWrXj1yGXw79fZ+CQKpa87we2Ltd2VkD8qgevDb2JUdVoIcY9F7hhsGO+28xK66K6/6VX/MsUIuIMSElwH1evUN2BDVQFhk0M1zx4bQg8ibW0ODkaYGJBYqavbqjSL1emy8pb9wS0XYmkMYYS93YV9QVqZZRBuJuYAk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--aliceryhl.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=WlgablEf; arc=none smtp.client-ip=209.85.128.74 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--aliceryhl.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="WlgablEf" Received: by mail-wm1-f74.google.com with SMTP id 5b1f17b1804b1-47754e6bddbso23096285e9.3 for ; Mon, 10 Nov 2025 05:34:54 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1762781693; x=1763386493; darn=vger.kernel.org; h=content-transfer-encoding:cc:to:from:subject:message-id:references :mime-version:in-reply-to:date:from:to:cc:subject:date:message-id :reply-to; bh=hOoHELQJUct+/jjBWDM/iGci2iRQNFOKXIkYMSUGeuo=; b=WlgablEf2ZxZFD/npMLOVEj4uxhJYiLDKZGJG/iBj83rnJ4g16cib1qrr2NHi7KIpD kL8Qw7IZemZ/suNsJsqBhO8n7AIVB19rQ7AwpLB9mTZ3R69ToB76lYFKnM50fyxBbyno RbsEI/Nqc66EBkpukv87OndMroafuYMybqhjRuSNtbf2uhcoLu44EX1ZU8GNXu7Aj5dW JGnbf3OPqPGH9PicuHDLBL81x/BQLV/g803szp0ZZaqgGxe9Bl/kn7sILwf57pDEbJwT TXi+Pg/JYqAAqOGEwHo9+4KBBW3L9G3OspQLjQ9INlqvEB5762GlcPXlFdQDdUU46LJf EPGQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1762781693; x=1763386493; h=content-transfer-encoding:cc:to:from:subject:message-id:references :mime-version:in-reply-to:date:x-gm-message-state:from:to:cc:subject :date:message-id:reply-to; bh=hOoHELQJUct+/jjBWDM/iGci2iRQNFOKXIkYMSUGeuo=; b=cixX42J+wdWE3dPx8OdTR+R+4R5Q5QuyTTv/pncoWDUDA5Qr9plT36k99zaNU9Hb5I w437Z2SykqKJCeFKSH6OeIFgyaWnDdmPFVrgv+5/q04SywGvqRfkrgt2zlEkQVlLHNA8 og0xe3d8PJ6jNnXbXcxTHnY1YO6YoBrHCeKs6b0d5b4x02sK9TBfCg7X3sfOZnE1JBhi Z6xYSf0/83JsGioLF0hPfN5d7tMnwVBoSPsgUiANdn6DEoRDmAxn7uWb9bOxsvGZR498 psfU0XAO43z6jje0xkcwY+gjM0dvIp1a8lxKKe0eq7n7uuNxQgm0kv1wBLiG1Ln2oP10 m4Nw== X-Forwarded-Encrypted: i=1; AJvYcCUl9+L/9WZpb1/Vw/EfMOUr7OlNR98Bu9USby/NHPc/RDmeur3nDboEWkdarrrCC+J67pqBwwkjjnObHe4=@vger.kernel.org X-Gm-Message-State: AOJu0Yw+8lrBQ44YMjwQlEOh6eyX4LGQsml8vnGQRvlZkmhlNmHQdW1q Fl87i1I9QlrBsEfrDy1VQg6PJ9Sm7NGd9YXFV7BfQG0npimsjUtUOHgptH7axe7CIAMvXaQ2VuX 4sidQFOSt22KH4Nk1ZA== X-Google-Smtp-Source: AGHT+IEhcd7v1I50vLKbfj11e6ZHlmUaM3O4D8Dzo9IdUzmf1+ToowaPgG4/VPB3oFSYHqCUiGhbSA+XEU/bilA= X-Received: from wmoo8-n1.prod.google.com ([2002:a05:600d:108:10b0:477:55e7:9973]) (user=aliceryhl job=prod-delivery.src-stubby-dispatcher) by 2002:a05:600c:1ca0:b0:477:7a63:611f with SMTP id 5b1f17b1804b1-4777a6362e2mr49082565e9.36.1762781693427; Mon, 10 Nov 2025 05:34:53 -0800 (PST) Date: Mon, 10 Nov 2025 13:34:52 +0000 In-Reply-To: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20251110095025.1475896-1-ojeda@kernel.org> <20251110095025.1475896-10-ojeda@kernel.org> <20251110131049.18e2f5c9@eugeo> Message-ID: Subject: Re: [PATCH 09/18] rust: proc-macro2: enable support in kbuild From: Alice Ryhl To: Miguel Ojeda Cc: Gary Guo , Miguel Ojeda , Alex Gaynor , Nathan Chancellor , Nicolas Schier , Boqun Feng , "=?utf-8?B?QmrDtnJu?= Roy Baron" , Benno Lossin , Andreas Hindborg , Trevor Gross , Danilo Krummrich , rust-for-linux@vger.kernel.org, linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org, patches@lists.linux.dev Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable On Mon, Nov 10, 2025 at 02:28:06PM +0100, Miguel Ojeda wrote: > On Mon, Nov 10, 2025 at 2:10=E2=80=AFPM Gary Guo wrote= : > > > > So I think the implementation here is sensible. I believe Miguel's > > patch is also pretty much replicating the logic in proc-macro2's > > build.rs. >=20 > Exactly, I was essentially following what upstream does for stable compil= ers. >=20 > (More generally, even if a feature may work, if upstream doesn't > usually test "older nightlies", then I wonder if we should enable such > combinations/setups anyway, unless we need them for a particular > reason). If these features change the public API exposed by proc-macro2, then it seems better to always disable it to prevent situations where you accidentally wrote code that does not work on our minimum rustc version. (Of course, this is a polyfill and no API is changed, then that's a different situation and then this LGTM.) Alice