public inbox for gentoo-commits@lists.gentoo.org
 help / color / mirror / Atom feed
From: "Eli Schwartz" <eschwartz@gentoo.org>
To: gentoo-commits@lists.gentoo.org
Subject: [gentoo-commits] repo/gentoo:master commit in: dev-util/bindgen/
Date: Fri, 26 Jul 2024 05:05:05 +0000 (UTC)	[thread overview]
Message-ID: <1721970280.2545d74b291af26f8ba088fbccd1bbe34acb0dc8.eschwartz@gentoo> (raw)

commit:     2545d74b291af26f8ba088fbccd1bbe34acb0dc8
Author:     Eli Schwartz <eschwartz <AT> gentoo <DOT> org>
AuthorDate: Fri Jul 26 04:17:39 2024 +0000
Commit:     Eli Schwartz <eschwartz <AT> gentoo <DOT> org>
CommitDate: Fri Jul 26 05:04:40 2024 +0000
URL:        https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=2545d74b

dev-util/bindgen: remove subslot binding to clang runtime dep

(For clarity, force it to bind to :* so no one tries to change this in
the future.)

It is NOT a build dependency. This is an indicator for the fact that it
is not needed to build, because bindgen simply compiles rust code that
knows how to try dlopening various names for libclang.

Consequently, it also doesn't matter which version of clang you have
installed -- any will do (well, technically, any version greater than
clang 6 will do), and in fact it is fully runtime switchable.

There's no reason to bind to whichever version may happen to have been
installed at the time of building. It is definitely not useful to
trigger a subslot rebuild every time clang updates.

Moreover, it would not bind to any subslot at all if you used
--buildpkgonly without clang installed at all. (In fact, this RDEPEND is
a direct violation of PMS, so the fact that it is mishandled isn't
terribly surprising.)

No revbump. The point of this change is so that people don't have to
keep rebuilding pointlessly, so the fact that people who don't use
--changed-deps won't see this change is a desirable feature.

Signed-off-by: Eli Schwartz <eschwartz <AT> gentoo.org>

 dev-util/bindgen/bindgen-0.68.1.ebuild | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dev-util/bindgen/bindgen-0.68.1.ebuild b/dev-util/bindgen/bindgen-0.68.1.ebuild
index 3038e2d54da7..23360f73eeba 100644
--- a/dev-util/bindgen/bindgen-0.68.1.ebuild
+++ b/dev-util/bindgen/bindgen-0.68.1.ebuild
@@ -118,7 +118,7 @@ KEYWORDS="amd64 arm64 ~ppc64 ~riscv"
 
 DEPEND="virtual/rust"
 RDEPEND="${DEPEND}
-	sys-devel/clang:="
+	sys-devel/clang:*"
 
 QA_FLAGS_IGNORED="usr/bin/bindgen"
 


             reply	other threads:[~2024-07-26  5:05 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-07-26  5:05 Eli Schwartz [this message]
  -- strict thread matches above, loose matches on Subject: below --
2025-01-17 16:13 [gentoo-commits] repo/gentoo:master commit in: dev-util/bindgen/ Sam James
2025-01-17 16:13 Sam James
2024-12-14 13:57 Arthur Zamarin
2024-10-18 17:32 Arthur Zamarin
2024-10-16 17:55 Sam James
2024-09-14 10:43 Michał Górny
2024-09-14 10:43 Michał Górny
2024-09-13 20:06 Arthur Zamarin
2024-09-13 20:06 Arthur Zamarin
2024-07-26  8:37 Arthur Zamarin
2024-04-01  6:27 Sam James
2024-01-31 13:56 Matt Turner
2023-12-21  6:52 Sam James
2023-09-14  3:05 Matt Turner
2023-09-13  4:13 Sam James
2023-09-13  2:15 Sam James
2023-07-02 11:18 Sam James
2023-05-21  8:35 Jakov Smolić
2022-12-23  7:57 Georgy Yakovlev
2022-12-20  4:58 Georgy Yakovlev
2022-12-20  4:57 Georgy Yakovlev
2022-12-20  4:57 Georgy Yakovlev

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1721970280.2545d74b291af26f8ba088fbccd1bbe34acb0dc8.eschwartz@gentoo \
    --to=eschwartz@gentoo.org \
    --cc=gentoo-commits@lists.gentoo.org \
    --cc=gentoo-dev@lists.gentoo.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox