Summary: | <xfce-base/libxfce4ui-4.19: does not have wayland USE flag despite potentially using Wayland-only symbols from GTK | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Kacper Słomiński <kacper.slominski72> |
Component: | Current packages | Assignee: | XFCE Team <xfce> |
Status: | UNCONFIRMED --- | ||
Severity: | normal | CC: | freedesktop-bugs, gnome, kacper.slominski72, mgorny, Xeha |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | AMD64 | ||
OS: | Linux | ||
See Also: | https://bugs.gentoo.org/show_bug.cgi?id=624960 | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- |
Description
Kacper Słomiński
2022-09-29 20:35:17 UTC
I don't see any configure option that would be relevant to this. This is an upstream bug then, isn't it? There is no configure option, but it does check for the define that Gdk defines (GDK_WINDOWING_WAYLAND), which seems good enough to me in this case. (In reply to Kacper Słomiński from comment #2) > There is no configure option, but it does check for the define that Gdk > defines (GDK_WINDOWING_WAYLAND), which seems good enough to me in this case. I don't see how to use that to prevent the breakage if libxfce4ui[-wayland] is built against gtk+[wayland]. (In reply to Michał Górny from comment #3) > I don't see how to use that to prevent the breakage if libxfce4ui[-wayland] > is built against gtk+[wayland]. Hmm, good point. I was thinking of having libfxce4ui depend on gtk+ having the same state of the wayland USE flag. Also probably worth noting, the Wayland detection is only there to disable some functionality that's broken under Wayland: #ifdef GDK_WINDOWING_WAYLAND if (GDK_IS_WAYLAND_DISPLAY (gdk_display_get_default ())) { if (startup_notify == TRUE) { /* 'sn_display_new' crashes when used via wayland, so no startup notification support here */ g_warning ("startup notification not supported for wayland sessions"); startup_notify = FALSE; } } #endif Sounds like they're working around some bug in startup-notification then. Any idea if this bug is still valid? I suppose it kinda sucks to add a dep on conditional symbols over something like this. Not sure what do to about this. Looking at the source code for libsn, sn_display_new crashes because it unconditionally uses the given X11 display (and immediately goes to work querying atoms without checking if it's valid), so it's not easy to fix there probably, considering the whole lib looks pretty X11-specific. Just for some further reading: there's all sorts of problems similar to this, unfortunately, see bug 624960 (although the XFCE issue here is actually something concrete instead of gcr etc) This also affects the new official binpkgs. They have wayland symbols. (In reply to Xeha from comment #8) > This also affects the new official binpkgs. They have wayland symbols. Which ones, specifically? The same one Kacper reported, or bug 624960 in general? I suppose this no longer applies to 4.19+ that no longer does automagic Wayland symbols. I suppose the easy way out would be to add IUSE='wayland X' and dep on gtk+[wayland=,X=], though it would be inconvenient for users. The hard way would be to hack it to control the code depending on the flags, and then do gtk+[wayland?,X?]. (In reply to Sam James from comment #9) > (In reply to Xeha from comment #8) > > This also affects the new official binpkgs. They have wayland symbols. > > Which ones, specifically? The same one Kacper reported, or bug 624960 in > general? The same as this one (by Kacper). Updated a node which had the new binpkgs enabled and it broke in the exact same way. |