Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 620896 - x11-apps/xinit: Add sys-auth/elogind support
Summary: x11-apps/xinit: Add sys-auth/elogind support
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Gentoo X packagers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: elogind-support
  Show dependency tree
 
Reported: 2017-06-05 12:21 UTC by Sven Eden
Modified: 2018-01-06 22:42 UTC (History)
3 users (show)

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


Attachments
Add support for elogind (xinit-add-elogind-support.patch,908 bytes, patch)
2017-06-05 12:21 UTC, Sven Eden
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Sven Eden 2017-06-05 12:21:15 UTC
Created attachment 475234 [details, diff]
Add support for elogind

xinit does not need any explicit elogind support per se.
But since elogind is systemd-login, the xinit-1.3.4-startx-current-vt.patch must not be applied if elogind is used as the systems session tracker, too.
Comment 1 Mike Gilbert gentoo-dev 2017-06-05 14:58:27 UTC
Did you mean to assign this to yourself?

Anyway, we could apply that patch unconditionally; our xserverrc file takes care of launching X on the correct vt, and we don't need the code in startx at all.
Comment 2 Sven Eden 2017-06-05 19:57:44 UTC
(In reply to Mike Gilbert from comment #1)
> Did you mean to assign this to yourself?

That was indeed a mistake. :-/

 
> Anyway, we could apply that patch unconditionally; our xserverrc file takes
> care of launching X on the correct vt, and we don't need the code in startx
> at all.


That would be great! Then bothe elogind and systemd USE flag can go. Simpler is better. ;-)
Comment 3 Matt Turner gentoo-dev 2017-10-21 00:29:13 UTC
I have no idea about this stuff. floppym, if you do, please feel free to handle it.
Comment 4 Mike Gilbert gentoo-dev 2017-10-21 05:27:46 UTC
Ok, it's been a while since I've looked at this, but I will review and see about committing something.
Comment 5 Andreas Sturmlechner gentoo-dev 2018-01-06 22:01:53 UTC
@Mike, any progress here?
Comment 6 Larry the Git Cow gentoo-dev 2018-01-06 22:42:26 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=024a727b2db862ff122216a324c3ca4177e2785b

commit 024a727b2db862ff122216a324c3ca4177e2785b
Author:     Mike Gilbert <floppym@gentoo.org>
AuthorDate: 2018-01-06 22:40:58 +0000
Commit:     Mike Gilbert <floppym@gentoo.org>
CommitDate: 2018-01-06 22:42:21 +0000

    x11-apps/xinit: apply vt patch unconditionally
    
    Our xserverrc file already takes care of passing the correct vt, so we
    don't need startx to do it too.
    
    Closes: https://bugs.gentoo.org/620896
    Package-Manager: Portage-2.3.19_p3, Repoman-2.3.6_p37

 x11-apps/xinit/xinit-1.3.4-r4.ebuild | 67 ++++++++++++++++++++++++++++++++++++
 1 file changed, 67 insertions(+)