After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 402475 - use gdm socket command to initiate login screen with user name
use gdm socket command to initiate login screen with user name
Status: RESOLVED OBSOLETE
Product: fast-user-switch-applet
Classification: Deprecated
Component: Applet
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Fast User Switch Applet Maintainer
Fast User Switch Applet Maintainer
: 172912 447457 (view as bug list)
Depends on: 335786
Blocks:
 
 
Reported: 2007-01-30 15:50 UTC by Matthias Clasen
Modified: 2008-06-10 22:55 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
use new gdm socket commands (6.56 KB, patch)
2007-01-30 15:53 UTC, Matthias Clasen
none Details | Review

Description Matthias Clasen 2007-01-30 15:50:39 UTC
...since it is a bit ridiculous to pick a user from the menu, only to be
forced to type the user name in the login screen.

The bug tracking the necessary gdm addition is bug 335786
Comment 1 Matthias Clasen 2007-01-30 15:53:27 UTC
Created attachment 81521 [details] [review]
use new gdm socket commands

The patch is untested, but it compiles, at least. It is missing some
form of runtime detection to ensure that the running gdm supports the new
socket commands. That shouldn't be hard to add, using the socket command to
get the gdm version number.
Comment 2 Teppo Turtiainen 2007-01-31 17:35:38 UTC
Isn't this a duplicate of bug 172912?
Comment 3 Matthias Clasen 2007-01-31 17:48:38 UTC
Looks like it, yes.

It would be nice to get some endorsement for the gdm addition required for this from the fusa maintainer side - Brian asked for this on the gdm bug linked below.
Comment 4 William Jon McCann 2007-01-31 17:53:07 UTC
I've tested this quite a bit and it works very well.  Do you think it is really necessary to have gdm version detection since GDM follows the same release schedule?
Comment 5 Matthias Clasen 2007-01-31 17:58:14 UTC
it would be good, but I don't see it as a blocker.
For gnome-screensaver there is at least a build-time dependency to ensure that g-s is built against a new-enough gdm, but the fusa code duplicates the gdm defines
internally. It could be as simple as handling an error return from gdm and falling back to the old command.
Comment 6 Thomas Thurman 2007-01-31 18:33:38 UTC
(In reply to comment #3)
> It would be nice to get some endorsement for the gdm addition required for this
> from the fusa maintainer side - Brian asked for this on the gdm bug linked
> below.

I think this is a marvellous idea (as I think I might have said on bug 172912, but perhaps not). Thanks very much for the patch: I'll try and review it before the end of the week (sorry you've caught me at such a busy time) and if it's good I'll do a special 2.17.x release.

From previous experience, checking the GDM version number all the time makes the code at least half as complex again as it needs to be, especially since (as Matthias said) all we need to do is fall back on error.
Comment 7 Matthias Clasen 2007-09-17 03:20:46 UTC
*** Bug 447457 has been marked as a duplicate of this bug. ***
Comment 8 Thomas Thurman 2007-09-17 04:13:54 UTC
*** Bug 172912 has been marked as a duplicate of this bug. ***
Comment 9 Martin-Éric Racine 2007-09-17 04:46:34 UTC
Exactly what was the point of marking an OLDER bug as a duplicate of this bug? Shouldn't it go the other way around? Marking it as solved at a time when the proposed patch hasn't been merged and the bug that offers the patch is still marked as unconfirmed wasn't particularly smart either.
Comment 10 Matthias Clasen 2007-09-17 06:24:46 UTC
I typically keep the bug with the patch open, regardless of which one is older.
There is no way to mark a bug as duplicate without setting it to resolved, so 
that is not indicative of any (lack of) smartness.
Comment 11 Baptiste Mille-Mathias 2008-06-10 20:44:19 UTC
Hello Mr Maintainer,

I'm pinging you, just to review the patches;

Thanks a lot

--
This message was brought to you by the Bugsquad team ©
Comment 12 Matthias Clasen 2008-06-10 22:55:43 UTC
The patch has been obsoleted by the gdm rewrite.