From: Eli Schwartz <eschwartz93@gmail.com>
To: gentoo-dev@lists.gentoo.org
Cc: gnome@gentoo.org, xfce@gentoo.org, binhost@gentoo.org
Subject: [gentoo-dev] [PATCH 2/5] net-libs/gtk-vnc: prevent automagically building against gtk[X,wayland]
Date: Sun, 23 Jun 2024 13:35:48 -0400 [thread overview]
Message-ID: <20240623173646.3368935-3-eschwartz93@gmail.com> (raw)
In-Reply-To: <20240623173646.3368935-1-eschwartz93@gmail.com>
e.g. the upstream code checks whether GDK_WINDOWING_WAYLAND is defined
by the gtk headers, and if so will compile against the wayland symbols
it provides. This means that gtk-vnc built on a system with
gtk+[-wayland] will be compatible with anything, but when built on a
system with gtk+[wayland], requires that at runtime.
This cannot be expressed with USE flags.
We could bind tightly to whether gtk was built with wayland (and X), and
in the process, arbitrarily restrict gtk-vnc[-wayland] to only build and
install on a system with gtk+[-wayland].
But we recently added a hack to gtk itself which allows hiding the
automagic macros entirely. Inject this via append-cflags if the USE
flags aren't set, to simulate building on a system with more minimal gtk
packages.
Bug: https://bugs.gentoo.org/624960
Closes: https://bugs.gentoo.org/865659
Signed-off-by: Eli Schwartz <eschwartz93@gmail.com>
---
...{gtk-vnc-1.3.1.ebuild => gtk-vnc-1.3.1-r1.ebuild} | 12 ++++++++----
1 file changed, 8 insertions(+), 4 deletions(-)
copy net-libs/gtk-vnc/{gtk-vnc-1.3.1.ebuild => gtk-vnc-1.3.1-r1.ebuild} (76%)
diff --git a/net-libs/gtk-vnc/gtk-vnc-1.3.1.ebuild b/net-libs/gtk-vnc/gtk-vnc-1.3.1-r1.ebuild
similarity index 76%
copy from net-libs/gtk-vnc/gtk-vnc-1.3.1.ebuild
copy to net-libs/gtk-vnc/gtk-vnc-1.3.1-r1.ebuild
index 252a0fcc4541..3c2eb470aa1a 100644
--- a/net-libs/gtk-vnc/gtk-vnc-1.3.1.ebuild
+++ b/net-libs/gtk-vnc/gtk-vnc-1.3.1-r1.ebuild
@@ -5,15 +5,15 @@ EAPI=8
PYTHON_COMPAT=( python3_{10..12} )
-inherit gnome.org vala meson python-any-r1 xdg
+inherit flag-o-matic gnome.org vala meson python-any-r1 xdg
DESCRIPTION="VNC viewer widget for GTK"
HOMEPAGE="https://wiki.gnome.org/Projects/gtk-vnc https://gitlab.gnome.org/GNOME/gtk-vnc"
LICENSE="LGPL-2.1+"
SLOT="0"
-KEYWORDS="~alpha amd64 ~arm arm64 ~ia64 ~loong ~ppc ppc64 ~riscv ~sparc x86"
-IUSE="+introspection pulseaudio sasl +vala"
+KEYWORDS="~alpha ~amd64 ~arm ~arm64 ~ia64 ~loong ~ppc ~ppc64 ~riscv ~sparc ~x86"
+IUSE="+introspection pulseaudio sasl +vala wayland X"
REQUIRED_USE="vala? ( introspection )"
RDEPEND="
@@ -23,7 +23,7 @@ RDEPEND="
>=net-libs/gnutls-3.6.0:0=
>=sys-libs/zlib-1.2.11
sasl? ( >=dev-libs/cyrus-sasl-2.1.27:2 )
- >=x11-libs/gtk+-3.22.0:3[introspection?]
+ >=x11-libs/gtk+-3.22.0:3[introspection?,wayland?,X?]
>=x11-libs/cairo-1.15.0
>=x11-libs/libX11-1.6.5
pulseaudio? ( media-libs/libpulse )
@@ -48,6 +48,10 @@ src_prepare() {
}
src_configure() {
+ # defang automagic dependencies, bug #927952
+ use wayland || append-cflags -DGENTOO_GTK_HIDE_WAYLAND
+ use X || append-cflags -DGENTOO_GTK_HIDE_X11
+
local emesonargs=(
$(meson_feature introspection)
$(meson_feature pulseaudio)
--
2.44.2
next prev parent reply other threads:[~2024-06-23 17:37 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-06-23 17:35 [gentoo-dev] [PATCH 0/5] Fixing automagic dependencies on gtk[wayland,X], Eli Schwartz
2024-06-23 17:35 ` [gentoo-dev] [PATCH 1/5] gui-libs/gtk: add a "poison" macro support to disable X/wayland Eli Schwartz
2024-06-24 9:08 ` Florian Schmaus
2024-06-26 9:03 ` Sam James
2024-06-27 4:52 ` Eli Schwartz
2024-06-27 4:58 ` Sam James
2024-07-03 11:16 ` Mart Raudsepp
2024-07-03 17:26 ` Eli Schwartz
2024-06-23 17:35 ` Eli Schwartz [this message]
2024-06-23 17:35 ` [gentoo-dev] [PATCH 3/5] x11-libs/wxGTK: prevent automagically building against gtk[X,wayland] Eli Schwartz
2024-06-23 17:35 ` [gentoo-dev] [PATCH 4/5] xfce-base/libxfce4ui: prevent automagically building against gtk[wayland] Eli Schwartz
2024-06-23 17:35 ` [gentoo-dev] [PATCH 5/5] dev-libs/libportal: prevent automagically building against gtk[X,wayland] Eli Schwartz
2024-06-23 18:33 ` [gentoo-dev] [PATCH 0/5] Fixing automagic dependencies on gtk[wayland,X], James Le Cuirot
2024-06-26 9:02 ` Sam James
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=20240623173646.3368935-3-eschwartz93@gmail.com \
--to=eschwartz93@gmail.com \
--cc=binhost@gentoo.org \
--cc=gentoo-dev@lists.gentoo.org \
--cc=gnome@gentoo.org \
--cc=xfce@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