On 08/09/17 17:17, Rafael Sadowski wrote:
> On Wed Aug 09, 2017 at 01:03:28PM +0200, Giovanni Bechis wrote:
>> Hi,
>> enabled yubikey support, tested with challenge-response on yubikey-neo.
>> Ok or too many dependencies ?
>> Cheers
>> Giovanni
>
> Thanks for the diff and keepassxc tests with yubikey.
> In my opinion there are too many dependencies for a special use-case.
>
> What do you think about a FLAVOR like this:
>
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/security/keepassxc/Makefile,v
> retrieving revision 1.5
> diff -u -p -u -p -r1.5 Makefile
> --- Makefile 30 Jul 2017 14:42:01 -0000 1.5
> +++ Makefile 9 Aug 2017 14:12:18 -0000
> @@ -13,7 +13,7 @@ HOMEPAGE = https://keepassxc.org
>
> MAINTAINER = Rafael Sadowski <rsadow...@openbsd.org>
>
> -# GPLv2
> +# GPLv2 or GPLv3
> PERMIT_PACKAGE_CDROM = Yes
>
> WANTLIB += Qt5Concurrent Qt5Core Qt5DBus Qt5Gui Qt5Network Qt5Widgets
> @@ -35,6 +35,16 @@ CONFIGURE_ARGS= -DWITH_GUI_TESTS=ON \
> -DWITH_XC_AUTOTYPE=ON
>
> TEST_IS_INTERACTIVE = X11
> +
> +FLAVORS = yubikey
> +FLAVOR ?=
> +
> +.if ${FLAVOR:Myubikey}
> +LIB_DEPENDS += security/yubico/yubico-c \
> + security/yubico/yubikey-personalization
> +CONFIGURE_ARGS += -DWITH_XC_YUBIKEY=ON
> +WANTLIB += yubikey ykpers-1
> +.endif
>
> post-patch:
> cd ${WRKSRC}/src/; sed -i -e 's/Q_OS_LINUX/Q_OS_UNIX/g' \
>
ok for me.
Cheers
Giovanni