Re: [Buildroot] [PATCH V2] package/x11r7/xwayland: fix glamor DRI ordering dependency on mesa3d
Adam Ford <[email protected]>
| Newsgroups | net.busybox.buildroot |
|---|---|
| Message-ID | <CAHCN7xJjR7tsaFBNGkxkk2orOC5srBuXW0S5mL4rdVSY+BY-HQ@mail.gmail.com> |
On Sun, Aug 9, 2026 at 12:44 PM John Ernberg <[email protected]> wrote: > Hi Adam, > > Apologies for the delay. > > On Mon, Aug 3, 2026 at 4:08 PM Adam Ford <[email protected]> wrote: > > > > Xwayland's glamor support enables a DRI configure check that looks for > > Mesa's dri.pc in staging. Without an explicit ordering dependency, a > > highly parallel build can run Xwayland's configure before mesa3d is > > installed, failing with: > > > > ERROR: Dependency "dri" not found > > > > Since mesa3d is what provides the DRI/EGL/GBM bits glamor relies on, fold > > it into the glamor condition: only enable -Dglamor=true and add it to the > > dependencies alongside libepoxy. > > > > Signed-off-by: Adam Ford <[email protected]> > > > > diff --git a/package/x11r7/xwayland/xwayland.mk > b/package/x11r7/xwayland/xwayland.mk > > index 02a21b96c1..3bd87a4552 100644 > > --- a/package/x11r7/xwayland/xwayland.mk > > +++ b/package/x11r7/xwayland/xwayland.mk > > @@ -32,7 +32,7 @@ XWAYLAND_CONF_OPTS = \ > > > > ifeq ($(BR2_PACKAGE_HAS_LIBEGL)$(BR2_PACKAGE_LIBEPOXY),yy) > > XWAYLAND_CONF_OPTS += -Dglamor=true > > -XWAYLAND_DEPENDENCIES += libepoxy > > +XWAYLAND_DEPENDENCIES += libepoxy mesa3d > > else > > XWAYLAND_CONF_OPTS += -Dglamor=false > > endif > > -- > > 2.53.0 > > > > dri is needed by glx, not glamor, when checking the code. Did you test > build this? > > (it's easier to reproduce with a per-package build where this becomes > a persistent failure rather than random) > I test-built this with per-package enabled, and it worked, but I only have Mesa providing libgl as in my build scenario, I am using either Intel or AMDGPU. > > It should go to the LIBGL check rather than the LIB*E*GL check. But as > I mentioned on v1, you probably need to check how this works when > nvidia-driver is the LIBGL provider. I'll dig a bit further into this. adam > > Best regards // John Ernberg > _______________________________________________ buildroot mailing list [email protected] https://lists.buildroot.org/mailman/listinfo/buildroot