436620 – x11-misc/slim-1.3.2-r7 does not set VT for X to use in non-daemon mode

Bug 436620 - x11-misc/slim-1.3.2-r7 does not set VT for X to use in non-daemon mode

Summary: x11-misc/slim-1.3.2-r7 does not set VT for X to use in non-daemon mode

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	Ian Stakenvicius (RETIRED)

URL:
Whiteboard:
Keywords:	PATCH

Depends on:
Blocks:

Reported:	2012-09-29 21:07 UTC by Łukasz Stelmach
Modified:	2012-10-05 17:07 UTC (History)
CC List:	1 user (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
Assign VT7 for X server statically (slim.conf-vt7.patch,440 bytes, patch) 2012-09-30 10:01 UTC, Łukasz Stelmach	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Łukasz Stelmach 2012-09-29 21:07:43 UTC

If slim is to be used with systemd it has to be started in non-daemon mode. In this mode it does not add automatically "vt7" to xserver_arguments and does not set hasVtSet variable to true, like it does in daemonmode. This affects the ConsoleKit patch which creates a session without providing x11-display-device, a session that is not on Seat1, thus it is unable to control some things e.g. Udisks.

To solve problem vt7 must be a part of the xserver_arguments option in the /etc/slim.conf just like in xdm's Xservers file.

Reproducible: Always

Comment 1 Łukasz Stelmach 2012-09-30 10:01:51 UTC

Created attachment 325360 [details, diff]
Assign VT7 for X server statically

Comment 2 Ian Stakenvicius (RETIRED) gentoo-dev

2012-10-02 13:36:23 UTC

Meh, doesn't hurt to explicitly declare the VT by default ; the C code just hard-codes 'vt07' if it's unset anyhow.

Will commit once I fix bug 436556

Comment 3 Ian Stakenvicius (RETIRED) gentoo-dev

2012-10-05 17:07:02 UTC

+*slim-1.3.4-r1 (05 Oct 2012)
+
+  05 Oct 2012; Ian Stakenvicius <axs@gentoo.org> -files/slim-1.3.3-config.diff,
+  -slim-1.3.4.ebuild, +slim-1.3.4-r1.ebuild, +files/slim-1.3.4-config.diff,
+  +files/slim-1.3.4-pam-end.patch:
+  force default vt07 (bug 436620) and ensure pam session isn't erroneously
+  cleaned during child execution (cause for krb5 ticket cache not existing in
+  bug 436556)
+