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 683278 - Remove gdm wrapper script
Remove gdm wrapper script
Status: RESOLVED FIXED
Product: gdm
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GDM maintainers
GDM maintainers
Depends on:
Blocks:
 
 
Reported: 2012-09-03 15:38 UTC by Armin K.
Modified: 2013-02-05 04:37 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Remove gdm wrapper script (4.22 KB, patch)
2012-09-03 15:38 UTC, Armin K. 		committed Details | Review

Description Armin K. 2012-09-03 15:38:02 UTC 
Created attachment 223338 [details] [review]
Remove gdm wrapper script

Hello.

As I already said on [1], the gdm wrapper script seems unnecesary these days, especially if PAM (and systemd) are used, the script has no use.

I have attached a patch which will remove wrapper script and rename gdm-binary to gdm, so nothing yet breaks. I've checked if anything uses/mentions gdm-binary, but I just found something in docs and gitignore, which I've also modified.

If any more help on the patch is needed, just say.

If the script is still somehow necesary in a way that I don't know, please say so and close the bug.

[1] https://mail.gnome.org/archives/gdm-list/2012-September/msg00000.html
Comment 1 Ray Strode [halfline] 2013-02-05 04:37:16 UTC 
Attachment 223338 [details] pushed as b6c11aa - Remove gdm wrapper script

Thanks for doing this. Seems like a nice improvement.

Hopefully it won't cause any unexpected surprises, but if it does
we can do follow up fixes.