https://bugzilla.novell.com/show_bug.cgi?id=255686 Summary: gdm-gdmsetup.patch needs a review Product: openSUSE 10.3 Version: Alpha 2 Platform: All OS/Version: All Status: NEW Severity: Normal Priority: P5 - None Component: GNOME AssignedTo: bnc-team-gnome@forge.provo.novell.com ReportedBy: andreas.hanke@gmx-topmail.de QAContact: qa@suse.de gdm-gdmsetup.patch will not apply to the next gdm version (2.18 final) and needs to be reviewed. Some background info: In the past, gdmsetup.desktop had Exec=gdmsetup and TryExec=gdmsetup without full paths. Therefore the .desktop file was only shown to root because gdmsetup is $(sbindir)/gdmsetup and non-root users don't have $(sbindir) in their path. Then, upstream decided to change to TryExec=@sbindir@/gdmsetup, but keep Exec=gdmsetup. This way, the .desktop file is shown to all users, but works only for root. Other users just get a "no such file or directory" message (bug 210943). We fixed this by effectively reverting the upstream change. In gdm 2.18.0, this bug is fixed, but in a different way. Now there is TryExec=@sbindir@/gdmsetup and Exec=@sbindir@/gdmsetup. The patch will no longer apply and the .desktop file will be shown to all users. Instead of a "no such file or directory" message, non-root users will get a "you have to log in as root" message. Someone has to decide what should happen: - Don't show it to non-root users => The patch needs to be extended to revert *both* upstream changes (Exec *and* TryExec). - Show it to non-root users and live with the "you have to log in as root" message => Drop the patch entirely. - Show it to non-root users, but make it also ask for the root password => Replace the patch with a different one so that gdmsetup is launched with gnomesu. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is.