[Bug 1086614] New: Duplicate / ambiguous desktop session selection entriesin sddm
http://bugzilla.opensuse.org/show_bug.cgi?id=1086614 Bug ID: 1086614 Summary: Duplicate / ambiguous desktop session selection entriesin sddm Classification: openSUSE Product: openSUSE Distribution Version: Leap 15.0 Hardware: Other OS: Other Status: NEW Severity: Normal Priority: P5 - None Component: Other Assignee: bnc-team-screening@forge.provo.novell.com Reporter: okurz@suse.com QA Contact: qa-bugs@suse.de Found By: --- Blocker: --- ## Observation in an installation of openSUSE Leap 15.0 installing many desktop managers, window managers, etc. I see the following entries in sddm: * Enlightenment * GNOME on Xorg * GNOME * IceWM Session * IceWM * LXDE * LXQt Desktop * MATE * Openbox * Plasma * TWM * Xfce Session * Enlightenment (Wayland) * GNOME (Wayland) * Plasma (Wayland) ## Problem I see the following issues: * What is "GNOME" when there is already "GNOME on Xorg", "GNOME (Wayland)"? * Can "GNOME on Xorg" and "GNOME (Wayland)" use a common naming scheme? E.g. "GNOME" (implicit Xorg) and "GNOME (Wayland)" or "GNOME (Xorg)", "GNOME (Wayland)"? * "IceWM Session" and "IceWM" duplicate? * Why the suffix "Desktop" in "LXQt Desktop" and "Session" on "Xfce" and and of the IceWM entry when others do not have that? -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=1086614 Oliver Kurz <okurz@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Summary|Duplicate / ambiguous |Duplicate / ambiguous |desktop session selection |desktop session selection |entriesin sddm |entries in sddm -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=1086614 http://bugzilla.opensuse.org/show_bug.cgi?id=1086614#c3 --- Comment #3 from QK ZHU <qkzhu@suse.com> --- "IceWM Session" and "IceWM" are not duplicate: - icewm.desktop will only lead to the window manager - icewm-session.desktop will load some env variables then starts icewmbg, icewmtray, icewm and some startup script So I think from IceWM's perspective this is not a bug. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=1086614 http://bugzilla.opensuse.org/show_bug.cgi?id=1086614#c6 --- Comment #6 from QK ZHU <qkzhu@suse.com> --- (In reply to Fabian Vogt from comment #5)
icewm-desktop-nodisplay.patch set NoDisplay for icewm.desktop since it only starts the window manager, icewm-session.desktop (the preferred program to start the IceWM system) will be displayed in display manager.
That explains it, it was Hidden before. Why did that get changed?
This patch was introduced by: bsc#1072557 since we agreed to ship only one .desktop, the usable one (icewm-session.desktop)
In GDM only IceWM Session will be displayed(on Leap15, TW and SLE15), so it seems that sddm doesn't handle 'NoDisplay=true' of icewm.desktop properly.
NoDisplay doesn't make much sense (neither does Hidden). It makes it impossible to start the session except by setting it as session for autologin.
I'll fix sddm to treat NoDisplay like Hidden for the menu, but IMO the file should either be removed completely or NoDisplay/Hidden removed.
And in https://bugzilla.suse.com/show_bug.cgi?id=1072557#c8 you suggested that
"Hidden=true" is equivalent to deleting the file, it must be set to false. So I used NoDisplay=true
-- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=1086614 http://bugzilla.opensuse.org/show_bug.cgi?id=1086614#c7 --- Comment #7 from Fabian Vogt <fvogt@suse.com> --- (In reply to QK ZHU from comment #6)
(In reply to Fabian Vogt from comment #5)
icewm-desktop-nodisplay.patch set NoDisplay for icewm.desktop since it only starts the window manager, icewm-session.desktop (the preferred program to start the IceWM system) will be displayed in display manager.
That explains it, it was Hidden before. Why did that get changed?
This patch was introduced by: bsc#1072557 since we agreed to ship only one .desktop, the usable one (icewm-session.desktop)
Sounds good.
In GDM only IceWM Session will be displayed(on Leap15, TW and SLE15), so it seems that sddm doesn't handle 'NoDisplay=true' of icewm.desktop properly.
NoDisplay doesn't make much sense (neither does Hidden). It makes it impossible to start the session except by setting it as session for autologin.
I'll fix sddm to treat NoDisplay like Hidden for the menu, but IMO the file should either be removed completely or NoDisplay/Hidden removed.
And in https://bugzilla.suse.com/show_bug.cgi?id=1072557#c8 you suggested that
"Hidden=true" is equivalent to deleting the file, it must be set to false. So I used NoDisplay=true
Yes, NoDisplay is the right way to hide an entry from the list. IMO doing that doesn't make sense for session files though - either you can use it or you can't. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=1086614 http://bugzilla.opensuse.org/show_bug.cgi?id=1086614#c9 Jonathan Rivrain <jrivrain@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jrivrain@suse.com --- Comment #9 from Jonathan Rivrain <jrivrain@suse.com> --- I had reported a pretty similar issue for 15.1 here: https://bugzilla.opensuse.org/show_bug.cgi?id=1127747 Except in that other bug, some menu entries are starting with even start the wrong desktop environment, as in the following table: NO! Gnome classic : Plasma on Xorg OK Gnome on Xorg : Gnome on Xorg NO! Gnome : Gnome on Xorg OK IceWM : Icewwm on Xorg OK Plasma : Plasma on Xorg NO! SLE classic : Plasma on Xorg OK TWM : TWM on Xorg OK Xfce : Xfce on Xorg OK Gnome : Gnome on wayland OK Plasma wayland : Plasma on wayland OK SLE classic : SLE classic on wayland -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.opensuse.org/show_bug.cgi?id=1086614 Adam Spiers <suse@adamspiers.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |suse@adamspiers.org -- You are receiving this mail because: You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply＠novell.com