Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 661650 - sys-apps/kbd-2.0.4 pam flag does nothing
Summary: sys-apps/kbd-2.0.4 pam flag does nothing
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-07-20 09:43 UTC by Garry Filakhtov
Modified: 2019-07-16 09:34 UTC (History)
1 user (show)

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


Attachments
Patch to fix ebuild pam flag (kbd-2.0.4.patch,288 bytes, patch)
2018-07-20 10:25 UTC, Garry Filakhtov
Details | Diff
Patch to fix --enable-vlock/--disable-vlock (kbd-configure-vlock.patch,575 bytes, patch)
2018-07-20 13:42 UTC, Garry Filakhtov
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Garry Filakhtov 2018-07-20 09:43:10 UTC
There seems to be a bug in the configure script for the sys-apps/kbd. pam flag passes a --enabled-vlock or --disable-vlock option, but both result in a vlock not being built.

A vlock is only built if both options are omitted.

./configure --enable-vlock
...
	kbd 2.0.4
	======

	prefix:			/usr/local
	libdir:			${exec_prefix}/lib
	bindir:			${exec_prefix}/bin
	datadir:		${datarootdir}

	compiler:		gcc
	cflags:			-g -O2 -Wall -Wextra -Wmissing-noreturn -Wdisabled-optimization -Wcast-align -Wshadow -Wmissing-format-attribute -Wmissing-prototypes -Wstrict-prototypes -Wmissing-declarations

	{get,set}keycodes:	yes
	resizecons:		yes
	optional progs:		no
	vlock:			no
	libkeymap:		no

./configure --disable-vlock
...
	kbd 2.0.4
	======

	prefix:			/usr/local
	libdir:			${exec_prefix}/lib
	bindir:			${exec_prefix}/bin
	datadir:		${datarootdir}

	compiler:		gcc
	cflags:			-g -O2 -Wall -Wextra -Wmissing-noreturn -Wdisabled-optimization -Wcast-align -Wshadow -Wmissing-format-attribute -Wmissing-prototypes -Wstrict-prototypes -Wmissing-declarations

	{get,set}keycodes:	yes
	resizecons:		yes
	optional progs:		no
	vlock:			no
	libkeymap:		no

./configure
...
	kbd 2.0.4
	======

	prefix:			/usr/local
	libdir:			${exec_prefix}/lib
	bindir:			${exec_prefix}/bin
	datadir:		${datarootdir}

	compiler:		gcc
	cflags:			-g -O2 -Wall -Wextra -Wmissing-noreturn -Wdisabled-optimization -Wcast-align -Wshadow -Wmissing-format-attribute -Wmissing-prototypes -Wstrict-prototypes -Wmissing-declarations

	{get,set}keycodes:	yes
	resizecons:		yes
	optional progs:		no
	vlock:			yes
	libkeymap:		no
Comment 1 Garry Filakhtov 2018-07-20 10:25:41 UTC
Created attachment 540162 [details, diff]
Patch to fix ebuild pam flag

Use a usex helper to pass --disable-vlock when pam flag is set and pass nothing if unset.
Comment 2 Garry Filakhtov 2018-07-20 10:33:44 UTC
Also, because vlock is provided separately by app-misc/vlock package, should kbd conflict when pam flag is enabled? Or, perhaps, it is good idea to drop pam flag for kbd altogether since it is currently broken and does nothing?
Comment 3 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2018-07-20 12:35:09 UTC
Your patch only works around the symptoms. A patch fixing configure would be much better.
Comment 4 Garry Filakhtov 2018-07-20 13:42:47 UTC
Created attachment 540168 [details, diff]
Patch to fix --enable-vlock/--disable-vlock

Patch configure and configure.ac to fix --enable-vlock and --disable-vlock
Comment 5 Garry Filakhtov 2018-07-20 13:43:09 UTC
@Lars Wendler good idea, thank you. Patch attached.
Comment 6 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2018-07-20 14:04:43 UTC
Thanks! I've sent your patch to upstream.
Comment 7 Larry the Git Cow gentoo-dev 2019-07-16 09:34:12 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=0c88fd4d16bd6407fe188e449b6b99549978427f

commit 0c88fd4d16bd6407fe188e449b6b99549978427f
Author:     Lars Wendler <polynomial-c@gentoo.org>
AuthorDate: 2019-07-16 09:30:11 +0000
Commit:     Lars Wendler <polynomial-c@gentoo.org>
CommitDate: 2019-07-16 09:34:06 +0000

    sys-apps/kbd: Bump to version 2.1.0
    
    Closes: https://bugs.gentoo.org/661650
    Package-Manager: Portage-2.3.69, Repoman-2.3.16
    Signed-off-by: Lars Wendler <polynomial-c@gentoo.org>

 sys-apps/kbd/Manifest         |  1 +
 sys-apps/kbd/kbd-2.1.0.ebuild | 70 +++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 71 insertions(+)