Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 686014 - sys-auth/ykpers needs elogind support
Summary: sys-auth/ykpers needs elogind support
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Göktürk Yüksek
URL: https://github.com/Yubico/yubikey-per...
Whiteboard:
Keywords:
Depends on:
Blocks: elogind-support
  Show dependency tree
 
Reported: 2019-05-15 08:57 UTC by Andreas Sturmlechner
Modified: 2019-05-16 08:32 UTC (History)
1 user (show)

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


Attachments
ykpers-1.19.3.ebuild.diff (ykpers-1.19.3.ebuild.diff,1.53 KB, patch)
2019-05-16 07:57 UTC, Lars Wendler (Polynomial-C) (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andreas Sturmlechner gentoo-dev 2019-05-15 08:57:15 UTC
Package currently applies udev rules when USE=consolekit but probably needs to do the same (+adjustment of the rules) with elogind.
Comment 1 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2019-05-15 09:05:14 UTC
See URL. The 70-yubikey.rules contains the following line:

  TEST=="/var/run/ConsoleKit/database", \

which of course doesn't work with elogind. I do not exactly know what purpose this test has but if it's only for checking if consolekit is present/running, we might be able to easily adjust that for elogind.

I have an actualy elogind use issue with this package. I have configured my system to only allow local logins with a YubiKey being connected. But without these udev rules, KDE's kscreenlocker cannot access the YubiKey which results in unlocking a locked KDE session to always fail.
Installing the udev rules despite USE="-consolekit" fixes this issue.

By the way, there's ykpers-1.9.3 available for a couple of days.
Comment 2 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2019-05-16 07:57:06 UTC
Created attachment 576888 [details, diff]
ykpers-1.19.3.ebuild.diff

Possible fix:

- Bump to version 1.19.3
- Bump to EAPI-7
- Added elogind USE flag and all necessary additional handling
- "Beautified" econf call a bit (but that's just personal preference)
Comment 3 Larry the Git Cow gentoo-dev 2019-05-16 08:32:35 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=82a86ba7b8a7e4395cb979869b11c85dcfdcc01c

commit 82a86ba7b8a7e4395cb979869b11c85dcfdcc01c
Author:     Lars Wendler <polynomial-c@gentoo.org>
AuthorDate: 2019-05-16 08:31:15 +0000
Commit:     Lars Wendler <polynomial-c@gentoo.org>
CommitDate: 2019-05-16 08:32:29 +0000

    sys-auth/ykpers: Added elogind USE flag.
    
    Bumped ebuild to EAPI-7
    
    Closes: https://bugs.gentoo.org/686014
    Package-Manager: Portage-2.3.66, Repoman-2.3.12
    Signed-off-by: Lars Wendler <polynomial-c@gentoo.org>

 sys-auth/ykpers/ykpers-1.19.3.ebuild | 32 ++++++++++++++++++++++----------
 1 file changed, 22 insertions(+), 10 deletions(-)