public inbox for gentoo-dev@lists.gentoo.org
 help / color / mirror / Atom feed
From: Mart Raudsepp <leio@gentoo.org>
To: gentoo-dev@lists.gentoo.org, Eli Schwartz <eschwartz93@gmail.com>
Cc: gnome@gentoo.org, xfce@gentoo.org, binhost@gentoo.org
Subject: Re: [gentoo-dev] [PATCH 1/5] gui-libs/gtk: add a "poison" macro support to disable X/wayland
Date: Wed, 03 Jul 2024 14:16:21 +0300	[thread overview]
Message-ID: <b2b6469873df58c0025f2782a6eed7da284fb638.camel@gentoo.org> (raw)
In-Reply-To: <874j9f3rsq.fsf@gentoo.org>

On Thu, 2024-06-27 at 05:58 +0100, Sam James wrote:
> Eli Schwartz <eschwartz93@gmail.com> writes:
> 
> > On 6/26/24 5:03 AM, Sam James wrote:
> > > Eli Schwartz <eschwartz93@gmail.com> writes:
> > > 
> > > > Many packages perform automagic dependencies on gdk's backend
> > > > implementations by checking if the macro is defined and then
> > > > using the
> > > > code it unlocks, rather than having a buildsystem option such
> > > > as
> > > > -Dwayland=true.
> > > > 
> > > Doesn't gtk3 need this too? Also, could we have an upstream
> > > report
> > > making them aware of this for gtk4?
> > 
> > 
> > Yes, gtk3 needs this too (and patches in this series depend on it).
> > 
> > At https://github.com/gentoo/gentoo/pull/37259 there are 6 patches,
> > not
> > 5 -- I appear to have accidentally excluded the first patch when
> > sending
> > it to the list, unsure how that happened. It's almost ccompletely
> > copy/paste from gtk4.
> 
> Thanks for clarifying - I was convinced I'd seen you show me it (I
> probably saw it on the branch) but I didn't think to check the PR.
> 
> > 
> > As far as reporting this upstream goes, I'm somewhat nervous they
> > will
> > suggest you should simply build against what you use and require
> > what
> > you build against. It's not a completely unreasonable suggestion,
> > in
> > fact it's the one I described as option 4 and Gentoo simply cannot
> > use
> > it today since it would require new EAPI.
> 
> I tend to agree. I see this as kind of our fault / a Gentoo-ish
> problem.


Please do see about approaching upstream about this in the context of
GTK4 in a constructive friendly manner, perhaps from the angle of how
to transition any distribution to a GTK that does not include X11
support with less migrational pain - this should be of interest for
more cases than Gentoo.

Maybe the idea about selecting backend stubs to include for their API
functions, which would just end up returning what a fully enabled
backend would return when the running environment is NOT that backend
(e.g. when gdk-x11 functions are called in a wayland native app or gdk-
wayland functions are called when X11 or Xwayland is in use).

If that gains some traction, maybe we could also do this for GTK3 as
well.

Given how we'd have to end up maintaining this long term, such upstream
discussions are sort of a soft blocker for proceeding with the
patchset, as far as I'm concerned.


Mart


  reply	other threads:[~2024-07-03 11:16 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 [this message]
2024-07-03 17:26           ` Eli Schwartz
2024-06-23 17:35 ` [gentoo-dev] [PATCH 2/5] net-libs/gtk-vnc: prevent automagically building against gtk[X,wayland] Eli Schwartz
2024-06-23 17:35 ` [gentoo-dev] [PATCH 3/5] x11-libs/wxGTK: " 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=b2b6469873df58c0025f2782a6eed7da284fb638.camel@gentoo.org \
    --to=leio@gentoo.org \
    --cc=binhost@gentoo.org \
    --cc=eschwartz93@gmail.com \
    --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