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 676138 - Add --enable-internal-pam-console option
Add --enable-internal-pam-console option
Status: RESOLVED NOTGNOME
Product: gdm
Classification: Core
Component: general
unspecified
Other All
: Normal normal
: ---
Assigned To: GDM maintainers
GDM maintainers
Depends on:
Blocks:
 
 
Reported: 2012-05-16 03:27 UTC by Colin Walters
Modified: 2012-05-16 20:22 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add --enable-internal-pam-console option (6.67 KB, patch)
2012-05-16 03:27 UTC, Colin Walters
none Details | Review

Description Colin Walters 2012-05-16 03:27:38 UTC
For OS builders who don't want to ship the full pam_console[1], this
simple option allows GDM to create the file /var/run/console/foo,
which is enough for DBus, which in turn is enough for NetworkManager.

Note there was a --enable-console-helper configure option which
was unused, so I deleted it.

This patch mirrors the Solaris devlogin handling.

[1] Because it's messy, complex code, mostly to implement the "chown
    device files" part which is completely obsoleted by udev.  The
    "make /var/run/console/foo" file isn't though; that's what this
    patch does.
Comment 1 Colin Walters 2012-05-16 03:27:40 UTC
Created attachment 214166 [details] [review]
Add --enable-internal-pam-console option
Comment 2 Ray Strode [halfline] 2012-05-16 18:30:23 UTC
ew ew ew. that file is going the way of the dinosaur, do we have to add new code for it?  distros don't have to use pam_console to get it. debian uses consolekit for instance.
Comment 3 Colin Walters 2012-05-16 20:03:04 UTC
(In reply to comment #2)
> ew ew ew. that file is going the way of the dinosaur, do we have to add new
> code for it?

All I can say is it's a pretty small amount of code to do conceptually what Solaris is doing, and would help me a lot as a temporary thing until I do a big rebase on systemd.

>  debian uses consolekit for instance.

I don't see any code in ConsoleKit to maintain /var/run/console/<username>.  (Ironic, I know...)  Please correct me if I'm wrong...

I suppose I could write a custom PAM module to do it, but...
Comment 4 Colin Walters 2012-05-16 20:22:55 UTC
Looks like the way to do this somewhat better is:

https://projects.archlinux.org/svntogit/packages.git/tree/trunk/pam-foreground-compat.ck?h=packages/consolekit