Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] keepassxc: update to 2.7.1.
@ 2022-04-12 23:18 Piraty
  2022-04-12 23:19 ` [PR PATCH] [Updated] " Piraty
                   ` (9 more replies)
  0 siblings, 10 replies; 11+ messages in thread
From: Piraty @ 2022-04-12 23:18 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1249 bytes --]

There is a new pull request by Piraty against master on the void-packages repository

https://github.com/Piraty/void-packages keepassxc-2.7.1
https://github.com/void-linux/void-packages/pull/36662

keepassxc: update to 2.7.1.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**
@lemmi 

<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!-- 
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


A patch file from https://github.com/void-linux/void-packages/pull/36662.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-keepassxc-2.7.1-36662.patch --]
[-- Type: text/x-diff, Size: 4961 bytes --]

From 22bccf87032d8a973b4d10525ed31621bbe4fa90 Mon Sep 17 00:00:00 2001
From: Piraty <piraty1@inbox.ru>
Date: Wed, 13 Apr 2022 00:50:31 +0200
Subject: [PATCH] keepassxc: update to 2.7.1.

---
 .../keepassxc/patches/ykcore-pthread.patch    | 23 ++++++++++++++
 srcpkgs/keepassxc/template                    | 31 ++++++++++++-------
 2 files changed, 43 insertions(+), 11 deletions(-)
 create mode 100644 srcpkgs/keepassxc/patches/ykcore-pthread.patch

diff --git a/srcpkgs/keepassxc/patches/ykcore-pthread.patch b/srcpkgs/keepassxc/patches/ykcore-pthread.patch
new file mode 100644
index 000000000000..220e207db0c0
--- /dev/null
+++ b/srcpkgs/keepassxc/patches/ykcore-pthread.patch
@@ -0,0 +1,23 @@
+thirdparty/ykcore: make cmake find libpthread
+
+diff --git a/src/thirdparty/ykcore/CMakeLists.txt b/src/thirdparty/ykcore/CMakeLists.txt
+index 47191ea7..103349f2 100644
+--- a/src/thirdparty/ykcore/CMakeLists.txt
++++ b/src/thirdparty/ykcore/CMakeLists.txt
+@@ -27,13 +27,15 @@ if(WIN32)
+ elseif(UNIX AND NOT APPLE)
+     target_sources(ykcore PRIVATE ykcore_libusb-1.0.c)
+ 
++    find_package(Threads REQUIRED)
++
+     find_library(LIBUSB_LIBRARY NAMES usb-1.0)
+     find_path(LIBUSB_INCLUDE_DIR NAMES libusb.h PATH_SUFFIXES "libusb-1.0" "libusb")
+     if(NOT LIBUSB_LIBRARY OR NOT LIBUSB_INCLUDE_DIR)
+         message(FATAL_ERROR "libusb-1.0 dev package required, but not found")
+     endif()
+ 
+-    target_link_libraries(ykcore PRIVATE ${LIBUSB_LIBRARY})
++    target_link_libraries(ykcore PRIVATE Threads::Threads ${LIBUSB_LIBRARY})
+     target_include_directories(ykcore PRIVATE ${LIBUSB_INCLUDE_DIR})
+     target_compile_definitions(ykcore PRIVATE _GNU_SOURCE)
+ elseif(APPLE)
diff --git a/srcpkgs/keepassxc/template b/srcpkgs/keepassxc/template
index f3543633d6da..25e21c706918 100644
--- a/srcpkgs/keepassxc/template
+++ b/srcpkgs/keepassxc/template
@@ -1,6 +1,6 @@
 # Template file for 'keepassxc'
 pkgname=keepassxc
-version=2.6.6
+version=2.7.1
 revision=1
 build_style=cmake
 configure_args="-DWITH_TESTS=ON -DWITH_XC_UPDATECHECK=OFF -DWITH_XC_DOCS=ON
@@ -8,26 +8,26 @@ configure_args="-DWITH_TESTS=ON -DWITH_XC_UPDATECHECK=OFF -DWITH_XC_DOCS=ON
  -DWITH_XC_BROWSER=$(vopt_if browser ON OFF)
  -DWITH_XC_FDOSECRETS=$(vopt_if fdosecrets ON OFF)
  -DWITH_XC_KEESHARE=$(vopt_if keeshare ON OFF)
- -DWITH_XC_KEESHARE_SECURE=$(vopt_if keeshare ON OFF)
  -DWITH_XC_NETWORKING=$(vopt_if network ON OFF)
  -DWITH_XC_SSHAGENT=$(vopt_if sshagent ON OFF)
  -DWITH_XC_YUBIKEY=$(vopt_if yubikey ON OFF)"
-hostmakedepends="qt5-qmake qt5-host-tools ruby-asciidoctor"
-makedepends="qt5-tools-devel qt5-svg-devel libgcrypt-devel libargon2-devel
- qrencode-devel readline-devel libsodium-devel
- $(vopt_if autotype 'qt5-x11extras-devel libXtst-devel libXi-devel')
- $(vopt_if keeshare quazip-devel)
- $(vopt_if yubikey 'libykpers-devel libyubikey-devel')"
+hostmakedepends="qt5-devel ruby-asciidoctor"
+makedepends="qt5-tools-devel qt5-svg-devel botan-devel libargon2-devel
+ qrencode-devel readline-devel pcsclite-devel
+ qt5-x11extras-devel 
+ $(vopt_if autotype 'libXtst-devel libXi-devel')
+ $(vopt_if keeshare minizip-devel)
+ $(vopt_if yubikey 'libusb-devel libyubikey-devel')"
+checkdepends="xvfb-run"
 short_desc="KeePassXC is a cross-platform port of “Keepass Password Safe”"
 maintainer="Piraty <piraty1@inbox.ru>"
 license="GPL-3.0-or-later, BSD-3-Clause, CC0-1.0, LGPL-2.0-only, LGPL-2.1-only,
  LGPL-3.0-or-later, Nokia-Qt-exception-1.1, MIT, BSL-1.0"
 homepage="https://keepassxc.org/"
 changelog="https://raw.githubusercontent.com/keepassxreboot/keepassxc/${version}/CHANGELOG.md"
-distfiles="https://github.com/keepassxreboot/keepassxc/releases/download/${version}/keepassxc-${version}-src.tar.xz"
-checksum=3603b11ac39b289c47fac77fa150e05fd64b393d8cfdf5732dc3ef106650a4e2
+distfiles="https://github.com/keepassxreboot/keepassxc/archive/refs/tags/${version}.tar.gz"
+checksum=c391d7315737758c8e820fa1995ba487f66427be9626a71811a0506d17e6f55e
 
-# https://github.com/keepassxreboot/keepassxc/blob/a775031fe9471310e50232d1861d4991e2803aff/CMakeLists.txt#L46
 build_options="autotype browser fdosecrets keeshare network sshagent yubikey"
 desc_option_autotype="Include auto-type"
 desc_option_browser="Include browser integration with keepassxc-browser-plugin"
@@ -38,8 +38,17 @@ desc_option_sshagent="Include SSH agent support"
 desc_option_yubikey="Include YubiKey support"
 build_options_default="autotype browser fdosecrets keeshare network sshagent yubikey"
 
+# > KeePassXC bundles the ykcore code due to lack of support from Yubico and
+# > stratification of version across various operating system distros.
+# > Additionally, KeePassXC has modified the API ok ykcore to make it more
+# > functional when using non-YubiKey keys (ie, OnlyKey)
+
 post_install() {
 	vlicense COPYING
 	vlicense LICENSE.BSD
 	vlicense LICENSE.MIT
 }
+
+do_check() {
+	xvfb-run ninja -C build test || : #XXX: TestCli::testClip() fails
+}

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PR PATCH] [Updated] keepassxc: update to 2.7.1.
  2022-04-12 23:18 [PR PATCH] keepassxc: update to 2.7.1 Piraty
@ 2022-04-12 23:19 ` Piraty
  2022-04-13  4:58 ` lemmi
                   ` (8 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: Piraty @ 2022-04-12 23:19 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1254 bytes --]

There is an updated pull request by Piraty against master on the void-packages repository

https://github.com/Piraty/void-packages keepassxc-2.7.1
https://github.com/void-linux/void-packages/pull/36662

keepassxc: update to 2.7.1.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**
@lemmi 

<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!-- 
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


A patch file from https://github.com/void-linux/void-packages/pull/36662.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-keepassxc-2.7.1-36662.patch --]
[-- Type: text/x-diff, Size: 4960 bytes --]

From 9e6e9b005c72a9913c26fa1c103e014c0ccccbd4 Mon Sep 17 00:00:00 2001
From: Piraty <piraty1@inbox.ru>
Date: Wed, 13 Apr 2022 00:50:31 +0200
Subject: [PATCH] keepassxc: update to 2.7.1.

---
 .../keepassxc/patches/ykcore-pthread.patch    | 23 ++++++++++++++
 srcpkgs/keepassxc/template                    | 31 ++++++++++++-------
 2 files changed, 43 insertions(+), 11 deletions(-)
 create mode 100644 srcpkgs/keepassxc/patches/ykcore-pthread.patch

diff --git a/srcpkgs/keepassxc/patches/ykcore-pthread.patch b/srcpkgs/keepassxc/patches/ykcore-pthread.patch
new file mode 100644
index 000000000000..220e207db0c0
--- /dev/null
+++ b/srcpkgs/keepassxc/patches/ykcore-pthread.patch
@@ -0,0 +1,23 @@
+thirdparty/ykcore: make cmake find libpthread
+
+diff --git a/src/thirdparty/ykcore/CMakeLists.txt b/src/thirdparty/ykcore/CMakeLists.txt
+index 47191ea7..103349f2 100644
+--- a/src/thirdparty/ykcore/CMakeLists.txt
++++ b/src/thirdparty/ykcore/CMakeLists.txt
+@@ -27,13 +27,15 @@ if(WIN32)
+ elseif(UNIX AND NOT APPLE)
+     target_sources(ykcore PRIVATE ykcore_libusb-1.0.c)
+ 
++    find_package(Threads REQUIRED)
++
+     find_library(LIBUSB_LIBRARY NAMES usb-1.0)
+     find_path(LIBUSB_INCLUDE_DIR NAMES libusb.h PATH_SUFFIXES "libusb-1.0" "libusb")
+     if(NOT LIBUSB_LIBRARY OR NOT LIBUSB_INCLUDE_DIR)
+         message(FATAL_ERROR "libusb-1.0 dev package required, but not found")
+     endif()
+ 
+-    target_link_libraries(ykcore PRIVATE ${LIBUSB_LIBRARY})
++    target_link_libraries(ykcore PRIVATE Threads::Threads ${LIBUSB_LIBRARY})
+     target_include_directories(ykcore PRIVATE ${LIBUSB_INCLUDE_DIR})
+     target_compile_definitions(ykcore PRIVATE _GNU_SOURCE)
+ elseif(APPLE)
diff --git a/srcpkgs/keepassxc/template b/srcpkgs/keepassxc/template
index f3543633d6da..4306972381ac 100644
--- a/srcpkgs/keepassxc/template
+++ b/srcpkgs/keepassxc/template
@@ -1,6 +1,6 @@
 # Template file for 'keepassxc'
 pkgname=keepassxc
-version=2.6.6
+version=2.7.1
 revision=1
 build_style=cmake
 configure_args="-DWITH_TESTS=ON -DWITH_XC_UPDATECHECK=OFF -DWITH_XC_DOCS=ON
@@ -8,26 +8,26 @@ configure_args="-DWITH_TESTS=ON -DWITH_XC_UPDATECHECK=OFF -DWITH_XC_DOCS=ON
  -DWITH_XC_BROWSER=$(vopt_if browser ON OFF)
  -DWITH_XC_FDOSECRETS=$(vopt_if fdosecrets ON OFF)
  -DWITH_XC_KEESHARE=$(vopt_if keeshare ON OFF)
- -DWITH_XC_KEESHARE_SECURE=$(vopt_if keeshare ON OFF)
  -DWITH_XC_NETWORKING=$(vopt_if network ON OFF)
  -DWITH_XC_SSHAGENT=$(vopt_if sshagent ON OFF)
  -DWITH_XC_YUBIKEY=$(vopt_if yubikey ON OFF)"
-hostmakedepends="qt5-qmake qt5-host-tools ruby-asciidoctor"
-makedepends="qt5-tools-devel qt5-svg-devel libgcrypt-devel libargon2-devel
- qrencode-devel readline-devel libsodium-devel
- $(vopt_if autotype 'qt5-x11extras-devel libXtst-devel libXi-devel')
- $(vopt_if keeshare quazip-devel)
- $(vopt_if yubikey 'libykpers-devel libyubikey-devel')"
+hostmakedepends="qt5-devel ruby-asciidoctor"
+makedepends="qt5-tools-devel qt5-svg-devel botan-devel libargon2-devel
+ qrencode-devel readline-devel pcsclite-devel
+ qt5-x11extras-devel
+ $(vopt_if autotype 'libXtst-devel libXi-devel')
+ $(vopt_if keeshare minizip-devel)
+ $(vopt_if yubikey 'libusb-devel libyubikey-devel')"
+checkdepends="xvfb-run"
 short_desc="KeePassXC is a cross-platform port of “Keepass Password Safe”"
 maintainer="Piraty <piraty1@inbox.ru>"
 license="GPL-3.0-or-later, BSD-3-Clause, CC0-1.0, LGPL-2.0-only, LGPL-2.1-only,
  LGPL-3.0-or-later, Nokia-Qt-exception-1.1, MIT, BSL-1.0"
 homepage="https://keepassxc.org/"
 changelog="https://raw.githubusercontent.com/keepassxreboot/keepassxc/${version}/CHANGELOG.md"
-distfiles="https://github.com/keepassxreboot/keepassxc/releases/download/${version}/keepassxc-${version}-src.tar.xz"
-checksum=3603b11ac39b289c47fac77fa150e05fd64b393d8cfdf5732dc3ef106650a4e2
+distfiles="https://github.com/keepassxreboot/keepassxc/archive/refs/tags/${version}.tar.gz"
+checksum=c391d7315737758c8e820fa1995ba487f66427be9626a71811a0506d17e6f55e
 
-# https://github.com/keepassxreboot/keepassxc/blob/a775031fe9471310e50232d1861d4991e2803aff/CMakeLists.txt#L46
 build_options="autotype browser fdosecrets keeshare network sshagent yubikey"
 desc_option_autotype="Include auto-type"
 desc_option_browser="Include browser integration with keepassxc-browser-plugin"
@@ -38,8 +38,17 @@ desc_option_sshagent="Include SSH agent support"
 desc_option_yubikey="Include YubiKey support"
 build_options_default="autotype browser fdosecrets keeshare network sshagent yubikey"
 
+# > KeePassXC bundles the ykcore code due to lack of support from Yubico and
+# > stratification of version across various operating system distros.
+# > Additionally, KeePassXC has modified the API ok ykcore to make it more
+# > functional when using non-YubiKey keys (ie, OnlyKey)
+
 post_install() {
 	vlicense COPYING
 	vlicense LICENSE.BSD
 	vlicense LICENSE.MIT
 }
+
+do_check() {
+	xvfb-run ninja -C build test || : #XXX: TestCli::testClip() fails
+}

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: keepassxc: update to 2.7.1.
  2022-04-12 23:18 [PR PATCH] keepassxc: update to 2.7.1 Piraty
  2022-04-12 23:19 ` [PR PATCH] [Updated] " Piraty
@ 2022-04-13  4:58 ` lemmi
  2022-04-13  8:16 ` Piraty
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: lemmi @ 2022-04-13  4:58 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 5373 bytes --]

New comment by lemmi on void-packages repository

https://github.com/void-linux/void-packages/pull/36662#issuecomment-1097558683

Comment:
Odd... for some reason I can't build it locally. So I rerun the CI and they seem to run fine. I even zapped my masterdir and ended up with the same result.
```
=> xbps-src: updating repositories for host (x86_64)...
[*] Updating repository `https://alpha.de.repo.voidlinux.org/current/x86_64-repodata' ...
[*] Updating repository `https://alpha.de.repo.voidlinux.org/current/nonfree/x86_64-repodata' ...
[*] Updating repository `https://alpha.de.repo.voidlinux.org/current/debug/x86_64-repodata' ...
[*] Updating repository `https://alpha.de.repo.voidlinux.org/current/multilib/x86_64-repodata' ...
[*] Updating repository `https://alpha.de.repo.voidlinux.org/current/multilib/nonfree/x86_64-repodata' ...
=> xbps-src: updating software in / masterdir...
=> xbps-src: cleaning up / masterdir...
=> keepassxc-2.7.1_1: removing autodeps, please wait...
=> keepassxc-2.7.1_1: the following build options are set:
   autotype: Include auto-type (ON)
   browser: Include browser integration with keepassxc-browser-plugin (ON)
   fdosecrets: Include Secret Storage Spec server side API (ON)
   keeshare: Include sharing integration with KeeShare (ON)
   network: Include networking code (favicon download) (ON)
   sshagent: Include SSH agent support (ON)
   yubikey: Include YubiKey support (ON)
=> keepassxc-2.7.1_1: building [cmake] for x86_64...
   [host] qt5-devel-5.15.3+20220222_1: found (https://alpha.de.repo.voidlinux.org/current)
   [host] ruby-asciidoctor-2.0.15_3: found (https://alpha.de.repo.voidlinux.org/current)
   [host] cmake-3.22.2_1: found (https://alpha.de.repo.voidlinux.org/current)
   [host] ninja-1.10.2_2: found (https://alpha.de.repo.voidlinux.org/current)
   [target] qt5-tools-devel-5.15.3+20220222_1: found (https://alpha.de.repo.voidlinux.org/current)
   [target] qt5-svg-devel-5.15.3+20220222_1: found (https://alpha.de.repo.voidlinux.org/current)
   [target] botan-devel-2.19.1_1: found (https://alpha.de.repo.voidlinux.org/current)
   [target] libargon2-devel-20190702_3: found (https://alpha.de.repo.voidlinux.org/current)
   [target] qrencode-devel-4.1.1_1: found (https://alpha.de.repo.voidlinux.org/current)
   [target] readline-devel-8.1.000_1: found (https://alpha.de.repo.voidlinux.org/current)
   [target] pcsclite-devel-1.9.5_1: found (https://alpha.de.repo.voidlinux.org/current)
   [target] qt5-x11extras-devel-5.15.3+20220222_1: found (https://alpha.de.repo.voidlinux.org/current)
   [target] libXtst-devel-1.2.3_2: found (https://alpha.de.repo.voidlinux.org/current)
   [target] libXi-devel-1.8_1: found (https://alpha.de.repo.voidlinux.org/current)
   [target] minizip-devel-1.2.11_3: found (https://alpha.de.repo.voidlinux.org/current)
   [target] libusb-devel-1.0.26_1: found (https://alpha.de.repo.voidlinux.org/current)
   [target] libyubikey-devel-1.13_2: found (https://alpha.de.repo.voidlinux.org/current)
=> keepassxc-2.7.1_1: installing host dependencies: qt5-devel-5.15.3+20220222_1 ruby-asciidoctor-2.0.15_3 cmake-3.22.2_1 ninja-1.10.2_2 ...
=> keepassxc-2.7.1_1: installing target dependencies: qt5-tools-devel-5.15.3+20220222_1 qt5-svg-devel-5.15.3+20220222_1 botan-devel-2.19.1_1 libargon2-devel-20190702_3 qrencode-devel-4.1.1_1 readline-devel-8.1.000_1 pcsclite-devel-1.9.5_1 qt5-x11extras-devel-5.15.3+20220222_1 libXtst-devel-1.2.3_2 libXi-devel-1.8_1 minizip-devel-1.2.11_3 libusb-devel-1.0.26_1 libyubikey-devel-1.13_2 ...
=> keepassxc-2.7.1_1: running do-fetch hook: 00-distfiles ...
=> keepassxc-2.7.1_1: running do-extract hook: 00-distfiles ...
=> keepassxc-2.7.1_1: extracting distfile(s), please wait...
=> keepassxc-2.7.1_1: running do-patch hook: 00-patches ...
=> keepassxc-2.7.1_1: patching: ykcore-pthread.patch.
1 out of 1 hunk FAILED -- saving rejects to file src/thirdparty/ykcore/CMakeLists.txt.rej
=> ERROR: keepassxc-2.7.1_1: do-patch_00-patches: 'patch -s ${_args} < ${_patch} 2> /dev/null' exited with 1
=> ERROR:   in _process_patch() at common/hooks/do-patch/00-patches.sh:34
=> ERROR:   in hook() at common/hooks/do-patch/00-patches.sh:51
=> ERROR:   in run_func() at common/xbps-src/shutils/common.sh:21
=> ERROR:   in run_pkg_hooks() at common/xbps-src/shutils/common.sh:245
=> ERROR:   in run_step() at common/xbps-src/shutils/common.sh:71
=> ERROR:   in main() at common/xbps-src/libexec/xbps-src-dopatch.sh:33
```
`cat src/thirdparty/ykcore/CMakeLists.txt.rej`:
```
--- src/thirdparty/ykcore/CMakeLists.txt
+++ src/thirdparty/ykcore/CMakeLists.txt
@@ -27,13 +27,15 @@ if(WIN32)
 elseif(UNIX AND NOT APPLE)
     target_sources(ykcore PRIVATE ykcore_libusb-1.0.c)

+    find_package(Threads REQUIRED)
+
     find_library(LIBUSB_LIBRARY NAMES usb-1.0)
     find_path(LIBUSB_INCLUDE_DIR NAMES libusb.h PATH_SUFFIXES "libusb-1.0" "libusb")
     if(NOT LIBUSB_LIBRARY OR NOT LIBUSB_INCLUDE_DIR)
         message(FATAL_ERROR "libusb-1.0 dev package required, but not found")
     endif()

-    target_link_libraries(ykcore PRIVATE ${LIBUSB_LIBRARY})
+    target_link_libraries(ykcore PRIVATE Threads::Threads ${LIBUSB_LIBRARY})
     target_include_directories(ykcore PRIVATE ${LIBUSB_INCLUDE_DIR})
     target_compile_definitions(ykcore PRIVATE _GNU_SOURCE)
 elseif(APPLE)
```

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: keepassxc: update to 2.7.1.
  2022-04-12 23:18 [PR PATCH] keepassxc: update to 2.7.1 Piraty
  2022-04-12 23:19 ` [PR PATCH] [Updated] " Piraty
  2022-04-13  4:58 ` lemmi
@ 2022-04-13  8:16 ` Piraty
  2022-04-15  8:07 ` [PR PATCH] [Updated] " Piraty
                   ` (6 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: Piraty @ 2022-04-13  8:16 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 236 bytes --]

New comment by Piraty on void-packages repository

https://github.com/void-linux/void-packages/pull/36662#issuecomment-1097694799

Comment:
it shows "WARNING: You are using an unstable build of KeePassXC." in GUI, so... nomerge for now

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PR PATCH] [Updated] keepassxc: update to 2.7.1.
  2022-04-12 23:18 [PR PATCH] keepassxc: update to 2.7.1 Piraty
                   ` (2 preceding siblings ...)
  2022-04-13  8:16 ` Piraty
@ 2022-04-15  8:07 ` Piraty
  2022-04-15  8:08 ` Piraty
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: Piraty @ 2022-04-15  8:07 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1254 bytes --]

There is an updated pull request by Piraty against master on the void-packages repository

https://github.com/Piraty/void-packages keepassxc-2.7.1
https://github.com/void-linux/void-packages/pull/36662

keepassxc: update to 2.7.1.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**
@lemmi 

<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!-- 
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


A patch file from https://github.com/void-linux/void-packages/pull/36662.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-keepassxc-2.7.1-36662.patch --]
[-- Type: text/x-diff, Size: 7124 bytes --]

From 85c4a61658d90a65c6b77d01bf7bb5f6df607196 Mon Sep 17 00:00:00 2001
From: Piraty <piraty1@inbox.ru>
Date: Wed, 13 Apr 2022 00:50:31 +0200
Subject: [PATCH] keepassxc: update to 2.7.1.

---
 .../keepassxc/patches/ykcore-pthread.patch    | 43 +++++++++++++++++++
 srcpkgs/keepassxc/template                    | 30 ++++++++-----
 2 files changed, 63 insertions(+), 10 deletions(-)
 create mode 100644 srcpkgs/keepassxc/patches/ykcore-pthread.patch

diff --git a/srcpkgs/keepassxc/patches/ykcore-pthread.patch b/srcpkgs/keepassxc/patches/ykcore-pthread.patch
new file mode 100644
index 000000000000..1c7c861061d8
--- /dev/null
+++ b/srcpkgs/keepassxc/patches/ykcore-pthread.patch
@@ -0,0 +1,43 @@
+https://patch-diff.githubusercontent.com/raw/keepassxreboot/keepassxc/pull/7807
+
+
+From 6abc55ceea72da81d323e9133905a9c0f0a376f5 Mon Sep 17 00:00:00 2001
+From: Piraty <piraty1@inbox.ru>
+Date: Wed, 6 Apr 2022 16:51:11 +0200
+Subject: [PATCH] thirdparty/ykcore: make cmake find libpthread
+
+it was missing the -pthread flag
+
+```
+[382/549] Linking CXX executable src/keepassxc
+FAILED: src/keepassxc
+: && /usr/lib/ccache/bin/g++ -DNDEBUG -fstack-clash-protection -D_FORTIFY_SOURCE=2 -mtune=generic -Og -ggdb3    -fdebug-prefix-map=/builddir/keepassxc-2.7.1=. -fno-common -Wall -Wextra -Wundef -Wpointer-arith -Wno-long-long -Wformat=2 -Wmissing-format-attribute -fvisibility=hidden -fvisibility-inlines-hidden -fstack-protector-strong -Wnon-virtual-dtor -Wold-style-cast -Woverloaded-virtual -Werror=format-security -Wcast-align -fsized-deallocation -Wno-deprecated-declarations -Wl,-z,relro -Wl,-z,now -Wl,--as-needed -Wl,--no-add-needed -Wl,--as-needed -Wl,--no-undefined -Wl,-z,relro,-z,now -pie   -Wl,--export-dynamic -rdynamic  -fPIE -pie src/CMakeFiles/keepassxc.dir/keepassxc_autogen/mocs_compilation.cpp.o src/CMakeFiles/keepassxc.dir/main.cpp.o -o src/keepassxc  src/libkeepassx_core.a  src/libautotype.a  src/browser/libkeepassxcbrowser.a  src/qrcode/libqrcode.a  /usr/lib/libQt5Svg.so.5.15.2  /usr/lib/libqrencode.so  src/fdosecrets/libfdosecrets.a  /usr/lib/libQt5Concurrent.so.5.15.2  /usr/lib64/libpcsclite.so  src/libzxcvbn.a  /usr/lib/libargon2.so  src/thirdparty/ykcore/libykcore.a  /usr/lib/libusb-1.0.so  src/sshagent/libsshagent.a  /usr/lib/libQt5Network.so.5.15.2  src/keeshare/libkeeshare.a  /usr/lib/libQt5Widgets.so.5.15.2  /usr/lib/libbotan-2.so  /usr/lib/libz.so  /usr/lib/libminizip.so  /usr/lib/libQt5DBus.so.5.15.2  /usr/lib/libQt5X11Extras.so.5.15.2  /usr/lib/libQt5Gui.so.5.15.2  /usr/lib/libQt5Core.so.5.15.2  -lX11 && :
+/usr/bin/ld: src/thirdparty/ykcore/libykcore.a(ykcore.c.o): undefined reference to symbol 'pthread_setspecific@@GLIBC_2.2.5'
+/usr/bin/ld: /lib64/libpthread.so.0: error adding symbols: DSO missing from command line
+collect2: error: ld returned 1 exit status
+```
+---
+ src/thirdparty/ykcore/CMakeLists.txt | 4 +++-
+ 1 file changed, 3 insertions(+), 1 deletion(-)
+
+diff --git a/src/thirdparty/ykcore/CMakeLists.txt b/src/thirdparty/ykcore/CMakeLists.txt
+index 47191ea741..103349f228 100644
+--- a/src/thirdparty/ykcore/CMakeLists.txt
++++ b/src/thirdparty/ykcore/CMakeLists.txt
+@@ -27,13 +27,15 @@ if(WIN32)
+ elseif(UNIX AND NOT APPLE)
+     target_sources(ykcore PRIVATE ykcore_libusb-1.0.c)
+ 
++    find_package(Threads REQUIRED)
++
+     find_library(LIBUSB_LIBRARY NAMES usb-1.0)
+     find_path(LIBUSB_INCLUDE_DIR NAMES libusb.h PATH_SUFFIXES "libusb-1.0" "libusb")
+     if(NOT LIBUSB_LIBRARY OR NOT LIBUSB_INCLUDE_DIR)
+         message(FATAL_ERROR "libusb-1.0 dev package required, but not found")
+     endif()
+ 
+-    target_link_libraries(ykcore PRIVATE ${LIBUSB_LIBRARY})
++    target_link_libraries(ykcore PRIVATE Threads::Threads ${LIBUSB_LIBRARY})
+     target_include_directories(ykcore PRIVATE ${LIBUSB_INCLUDE_DIR})
+     target_compile_definitions(ykcore PRIVATE _GNU_SOURCE)
+ elseif(APPLE)
diff --git a/srcpkgs/keepassxc/template b/srcpkgs/keepassxc/template
index f3543633d6da..e5105cc50d56 100644
--- a/srcpkgs/keepassxc/template
+++ b/srcpkgs/keepassxc/template
@@ -1,6 +1,6 @@
 # Template file for 'keepassxc'
 pkgname=keepassxc
-version=2.6.6
+version=2.7.1
 revision=1
 build_style=cmake
 configure_args="-DWITH_TESTS=ON -DWITH_XC_UPDATECHECK=OFF -DWITH_XC_DOCS=ON
@@ -8,16 +8,17 @@ configure_args="-DWITH_TESTS=ON -DWITH_XC_UPDATECHECK=OFF -DWITH_XC_DOCS=ON
  -DWITH_XC_BROWSER=$(vopt_if browser ON OFF)
  -DWITH_XC_FDOSECRETS=$(vopt_if fdosecrets ON OFF)
  -DWITH_XC_KEESHARE=$(vopt_if keeshare ON OFF)
- -DWITH_XC_KEESHARE_SECURE=$(vopt_if keeshare ON OFF)
  -DWITH_XC_NETWORKING=$(vopt_if network ON OFF)
  -DWITH_XC_SSHAGENT=$(vopt_if sshagent ON OFF)
  -DWITH_XC_YUBIKEY=$(vopt_if yubikey ON OFF)"
-hostmakedepends="qt5-qmake qt5-host-tools ruby-asciidoctor"
-makedepends="qt5-tools-devel qt5-svg-devel libgcrypt-devel libargon2-devel
- qrencode-devel readline-devel libsodium-devel
- $(vopt_if autotype 'qt5-x11extras-devel libXtst-devel libXi-devel')
- $(vopt_if keeshare quazip-devel)
- $(vopt_if yubikey 'libykpers-devel libyubikey-devel')"
+hostmakedepends="qt5-devel ruby-asciidoctor"
+makedepends="qt5-tools-devel qt5-svg-devel botan-devel libargon2-devel
+ qrencode-devel readline-devel pcsclite-devel
+ qt5-x11extras-devel
+ $(vopt_if autotype 'libXtst-devel libXi-devel')
+ $(vopt_if keeshare minizip-devel)
+ $(vopt_if yubikey 'libusb-devel')"
+checkdepends="xvfb-run"
 short_desc="KeePassXC is a cross-platform port of “Keepass Password Safe”"
 maintainer="Piraty <piraty1@inbox.ru>"
 license="GPL-3.0-or-later, BSD-3-Clause, CC0-1.0, LGPL-2.0-only, LGPL-2.1-only,
@@ -25,9 +26,8 @@ license="GPL-3.0-or-later, BSD-3-Clause, CC0-1.0, LGPL-2.0-only, LGPL-2.1-only,
 homepage="https://keepassxc.org/"
 changelog="https://raw.githubusercontent.com/keepassxreboot/keepassxc/${version}/CHANGELOG.md"
 distfiles="https://github.com/keepassxreboot/keepassxc/releases/download/${version}/keepassxc-${version}-src.tar.xz"
-checksum=3603b11ac39b289c47fac77fa150e05fd64b393d8cfdf5732dc3ef106650a4e2
+checksum=6001ba626c35c316dbda6de35736f012a2264f95139fcb4a094b8eb49b15d3e7
 
-# https://github.com/keepassxreboot/keepassxc/blob/a775031fe9471310e50232d1861d4991e2803aff/CMakeLists.txt#L46
 build_options="autotype browser fdosecrets keeshare network sshagent yubikey"
 desc_option_autotype="Include auto-type"
 desc_option_browser="Include browser integration with keepassxc-browser-plugin"
@@ -38,8 +38,18 @@ desc_option_sshagent="Include SSH agent support"
 desc_option_yubikey="Include YubiKey support"
 build_options_default="autotype browser fdosecrets keeshare network sshagent yubikey"
 
+# quote droidmonkey from IRC (keepassxc developer)
+# > KeePassXC bundles the ykcore code due to lack of support from Yubico and
+# > stratification of version across various operating system distros.
+# > Additionally, KeePassXC has modified the API of ykcore to make it more
+# > functional when using non-YubiKey keys (ie, OnlyKey)
+
 post_install() {
 	vlicense COPYING
 	vlicense LICENSE.BSD
 	vlicense LICENSE.MIT
 }
+
+do_check() {
+	xvfb-run ninja -C build test || : #XXX: TestCli::testClip() fails
+}

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: keepassxc: update to 2.7.1.
  2022-04-12 23:18 [PR PATCH] keepassxc: update to 2.7.1 Piraty
                   ` (3 preceding siblings ...)
  2022-04-15  8:07 ` [PR PATCH] [Updated] " Piraty
@ 2022-04-15  8:08 ` Piraty
  2022-04-15  8:09 ` [PR PATCH] [Updated] " Piraty
                   ` (4 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: Piraty @ 2022-04-15  8:08 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 349 bytes --]

New comment by Piraty on void-packages repository

https://github.com/void-linux/void-packages/pull/36662#issuecomment-1097694799

Comment:
it shows "WARNING: You are using an unstable build of KeePassXC." in GUI, so... nomerge for now
[EDIT] fixed, building from generated release source tarball , which ships `.version` to overwrite git-describe

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PR PATCH] [Updated] keepassxc: update to 2.7.1.
  2022-04-12 23:18 [PR PATCH] keepassxc: update to 2.7.1 Piraty
                   ` (4 preceding siblings ...)
  2022-04-15  8:08 ` Piraty
@ 2022-04-15  8:09 ` Piraty
  2022-04-15  8:21 ` Piraty
                   ` (3 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: Piraty @ 2022-04-15  8:09 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1254 bytes --]

There is an updated pull request by Piraty against master on the void-packages repository

https://github.com/Piraty/void-packages keepassxc-2.7.1
https://github.com/void-linux/void-packages/pull/36662

keepassxc: update to 2.7.1.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**
@lemmi 

<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!-- 
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


A patch file from https://github.com/void-linux/void-packages/pull/36662.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-keepassxc-2.7.1-36662.patch --]
[-- Type: text/x-diff, Size: 7122 bytes --]

From 30c923204dbcaf443e8172bc5d72fde317fd5742 Mon Sep 17 00:00:00 2001
From: Piraty <piraty1@inbox.ru>
Date: Wed, 13 Apr 2022 00:50:31 +0200
Subject: [PATCH] keepassxc: update to 2.7.1.

---
 .../keepassxc/patches/ykcore-pthread.patch    | 42 +++++++++++++++++++
 srcpkgs/keepassxc/template                    | 30 ++++++++-----
 2 files changed, 62 insertions(+), 10 deletions(-)
 create mode 100644 srcpkgs/keepassxc/patches/ykcore-pthread.patch

diff --git a/srcpkgs/keepassxc/patches/ykcore-pthread.patch b/srcpkgs/keepassxc/patches/ykcore-pthread.patch
new file mode 100644
index 000000000000..224e8faa0fbf
--- /dev/null
+++ b/srcpkgs/keepassxc/patches/ykcore-pthread.patch
@@ -0,0 +1,42 @@
+https://patch-diff.githubusercontent.com/raw/keepassxreboot/keepassxc/pull/7807
+
+From 6abc55ceea72da81d323e9133905a9c0f0a376f5 Mon Sep 17 00:00:00 2001
+From: Piraty <piraty1@inbox.ru>
+Date: Wed, 6 Apr 2022 16:51:11 +0200
+Subject: [PATCH] thirdparty/ykcore: make cmake find libpthread
+
+it was missing the -pthread flag
+
+```
+[382/549] Linking CXX executable src/keepassxc
+FAILED: src/keepassxc
+: && /usr/lib/ccache/bin/g++ -DNDEBUG -fstack-clash-protection -D_FORTIFY_SOURCE=2 -mtune=generic -Og -ggdb3    -fdebug-prefix-map=/builddir/keepassxc-2.7.1=. -fno-common -Wall -Wextra -Wundef -Wpointer-arith -Wno-long-long -Wformat=2 -Wmissing-format-attribute -fvisibility=hidden -fvisibility-inlines-hidden -fstack-protector-strong -Wnon-virtual-dtor -Wold-style-cast -Woverloaded-virtual -Werror=format-security -Wcast-align -fsized-deallocation -Wno-deprecated-declarations -Wl,-z,relro -Wl,-z,now -Wl,--as-needed -Wl,--no-add-needed -Wl,--as-needed -Wl,--no-undefined -Wl,-z,relro,-z,now -pie   -Wl,--export-dynamic -rdynamic  -fPIE -pie src/CMakeFiles/keepassxc.dir/keepassxc_autogen/mocs_compilation.cpp.o src/CMakeFiles/keepassxc.dir/main.cpp.o -o src/keepassxc  src/libkeepassx_core.a  src/libautotype.a  src/browser/libkeepassxcbrowser.a  src/qrcode/libqrcode.a  /usr/lib/libQt5Svg.so.5.15.2  /usr/lib/libqrencode.so  src/fdosecrets/libfdosecrets.a  /usr/lib/libQt5Concurrent.so.5.15.2  /usr/lib64/libpcsclite.so  src/libzxcvbn.a  /usr/lib/libargon2.so  src/thirdparty/ykcore/libykcore.a  /usr/lib/libusb-1.0.so  src/sshagent/libsshagent.a  /usr/lib/libQt5Network.so.5.15.2  src/keeshare/libkeeshare.a  /usr/lib/libQt5Widgets.so.5.15.2  /usr/lib/libbotan-2.so  /usr/lib/libz.so  /usr/lib/libminizip.so  /usr/lib/libQt5DBus.so.5.15.2  /usr/lib/libQt5X11Extras.so.5.15.2  /usr/lib/libQt5Gui.so.5.15.2  /usr/lib/libQt5Core.so.5.15.2  -lX11 && :
+/usr/bin/ld: src/thirdparty/ykcore/libykcore.a(ykcore.c.o): undefined reference to symbol 'pthread_setspecific@@GLIBC_2.2.5'
+/usr/bin/ld: /lib64/libpthread.so.0: error adding symbols: DSO missing from command line
+collect2: error: ld returned 1 exit status
+```
+---
+ src/thirdparty/ykcore/CMakeLists.txt | 4 +++-
+ 1 file changed, 3 insertions(+), 1 deletion(-)
+
+diff --git a/src/thirdparty/ykcore/CMakeLists.txt b/src/thirdparty/ykcore/CMakeLists.txt
+index 47191ea741..103349f228 100644
+--- a/src/thirdparty/ykcore/CMakeLists.txt
++++ b/src/thirdparty/ykcore/CMakeLists.txt
+@@ -27,13 +27,15 @@ if(WIN32)
+ elseif(UNIX AND NOT APPLE)
+     target_sources(ykcore PRIVATE ykcore_libusb-1.0.c)
+ 
++    find_package(Threads REQUIRED)
++
+     find_library(LIBUSB_LIBRARY NAMES usb-1.0)
+     find_path(LIBUSB_INCLUDE_DIR NAMES libusb.h PATH_SUFFIXES "libusb-1.0" "libusb")
+     if(NOT LIBUSB_LIBRARY OR NOT LIBUSB_INCLUDE_DIR)
+         message(FATAL_ERROR "libusb-1.0 dev package required, but not found")
+     endif()
+ 
+-    target_link_libraries(ykcore PRIVATE ${LIBUSB_LIBRARY})
++    target_link_libraries(ykcore PRIVATE Threads::Threads ${LIBUSB_LIBRARY})
+     target_include_directories(ykcore PRIVATE ${LIBUSB_INCLUDE_DIR})
+     target_compile_definitions(ykcore PRIVATE _GNU_SOURCE)
+ elseif(APPLE)
diff --git a/srcpkgs/keepassxc/template b/srcpkgs/keepassxc/template
index f3543633d6da..e5105cc50d56 100644
--- a/srcpkgs/keepassxc/template
+++ b/srcpkgs/keepassxc/template
@@ -1,6 +1,6 @@
 # Template file for 'keepassxc'
 pkgname=keepassxc
-version=2.6.6
+version=2.7.1
 revision=1
 build_style=cmake
 configure_args="-DWITH_TESTS=ON -DWITH_XC_UPDATECHECK=OFF -DWITH_XC_DOCS=ON
@@ -8,16 +8,17 @@ configure_args="-DWITH_TESTS=ON -DWITH_XC_UPDATECHECK=OFF -DWITH_XC_DOCS=ON
  -DWITH_XC_BROWSER=$(vopt_if browser ON OFF)
  -DWITH_XC_FDOSECRETS=$(vopt_if fdosecrets ON OFF)
  -DWITH_XC_KEESHARE=$(vopt_if keeshare ON OFF)
- -DWITH_XC_KEESHARE_SECURE=$(vopt_if keeshare ON OFF)
  -DWITH_XC_NETWORKING=$(vopt_if network ON OFF)
  -DWITH_XC_SSHAGENT=$(vopt_if sshagent ON OFF)
  -DWITH_XC_YUBIKEY=$(vopt_if yubikey ON OFF)"
-hostmakedepends="qt5-qmake qt5-host-tools ruby-asciidoctor"
-makedepends="qt5-tools-devel qt5-svg-devel libgcrypt-devel libargon2-devel
- qrencode-devel readline-devel libsodium-devel
- $(vopt_if autotype 'qt5-x11extras-devel libXtst-devel libXi-devel')
- $(vopt_if keeshare quazip-devel)
- $(vopt_if yubikey 'libykpers-devel libyubikey-devel')"
+hostmakedepends="qt5-devel ruby-asciidoctor"
+makedepends="qt5-tools-devel qt5-svg-devel botan-devel libargon2-devel
+ qrencode-devel readline-devel pcsclite-devel
+ qt5-x11extras-devel
+ $(vopt_if autotype 'libXtst-devel libXi-devel')
+ $(vopt_if keeshare minizip-devel)
+ $(vopt_if yubikey 'libusb-devel')"
+checkdepends="xvfb-run"
 short_desc="KeePassXC is a cross-platform port of “Keepass Password Safe”"
 maintainer="Piraty <piraty1@inbox.ru>"
 license="GPL-3.0-or-later, BSD-3-Clause, CC0-1.0, LGPL-2.0-only, LGPL-2.1-only,
@@ -25,9 +26,8 @@ license="GPL-3.0-or-later, BSD-3-Clause, CC0-1.0, LGPL-2.0-only, LGPL-2.1-only,
 homepage="https://keepassxc.org/"
 changelog="https://raw.githubusercontent.com/keepassxreboot/keepassxc/${version}/CHANGELOG.md"
 distfiles="https://github.com/keepassxreboot/keepassxc/releases/download/${version}/keepassxc-${version}-src.tar.xz"
-checksum=3603b11ac39b289c47fac77fa150e05fd64b393d8cfdf5732dc3ef106650a4e2
+checksum=6001ba626c35c316dbda6de35736f012a2264f95139fcb4a094b8eb49b15d3e7
 
-# https://github.com/keepassxreboot/keepassxc/blob/a775031fe9471310e50232d1861d4991e2803aff/CMakeLists.txt#L46
 build_options="autotype browser fdosecrets keeshare network sshagent yubikey"
 desc_option_autotype="Include auto-type"
 desc_option_browser="Include browser integration with keepassxc-browser-plugin"
@@ -38,8 +38,18 @@ desc_option_sshagent="Include SSH agent support"
 desc_option_yubikey="Include YubiKey support"
 build_options_default="autotype browser fdosecrets keeshare network sshagent yubikey"
 
+# quote droidmonkey from IRC (keepassxc developer)
+# > KeePassXC bundles the ykcore code due to lack of support from Yubico and
+# > stratification of version across various operating system distros.
+# > Additionally, KeePassXC has modified the API of ykcore to make it more
+# > functional when using non-YubiKey keys (ie, OnlyKey)
+
 post_install() {
 	vlicense COPYING
 	vlicense LICENSE.BSD
 	vlicense LICENSE.MIT
 }
+
+do_check() {
+	xvfb-run ninja -C build test || : #XXX: TestCli::testClip() fails
+}

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PR PATCH] [Updated] keepassxc: update to 2.7.1.
  2022-04-12 23:18 [PR PATCH] keepassxc: update to 2.7.1 Piraty
                   ` (5 preceding siblings ...)
  2022-04-15  8:09 ` [PR PATCH] [Updated] " Piraty
@ 2022-04-15  8:21 ` Piraty
  2022-04-19  9:10 ` Piraty
                   ` (2 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: Piraty @ 2022-04-15  8:21 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1254 bytes --]

There is an updated pull request by Piraty against master on the void-packages repository

https://github.com/Piraty/void-packages keepassxc-2.7.1
https://github.com/void-linux/void-packages/pull/36662

keepassxc: update to 2.7.1.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**
@lemmi 

<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!-- 
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


A patch file from https://github.com/void-linux/void-packages/pull/36662.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-keepassxc-2.7.1-36662.patch --]
[-- Type: text/x-diff, Size: 4851 bytes --]

From 7c65b7ff3293cb321f265d2314ee362244aa0b2f Mon Sep 17 00:00:00 2001
From: Piraty <piraty1@inbox.ru>
Date: Wed, 13 Apr 2022 00:50:31 +0200
Subject: [PATCH] keepassxc: update to 2.7.1.

---
 .../keepassxc/patches/ykcore-pthread.patch    | 21 +++++++++++++
 srcpkgs/keepassxc/template                    | 30 ++++++++++++-------
 2 files changed, 41 insertions(+), 10 deletions(-)
 create mode 100644 srcpkgs/keepassxc/patches/ykcore-pthread.patch

diff --git a/srcpkgs/keepassxc/patches/ykcore-pthread.patch b/srcpkgs/keepassxc/patches/ykcore-pthread.patch
new file mode 100644
index 000000000000..cc9398f65474
--- /dev/null
+++ b/srcpkgs/keepassxc/patches/ykcore-pthread.patch
@@ -0,0 +1,21 @@
+https://patch-diff.githubusercontent.com/raw/keepassxreboot/keepassxc/pull/7807
+
+--- a/src/thirdparty/ykcore/CMakeLists.txt
++++ b/src/thirdparty/ykcore/CMakeLists.txt
+@@ -27,13 +27,15 @@ if(WIN32)
+ elseif(UNIX AND NOT APPLE)
+     target_sources(ykcore PRIVATE ykcore_libusb-1.0.c)
+ 
++    find_package(Threads REQUIRED)
++
+     find_library(LIBUSB_LIBRARY NAMES usb-1.0)
+     find_path(LIBUSB_INCLUDE_DIR NAMES libusb.h PATH_SUFFIXES "libusb-1.0" "libusb")
+     if(NOT LIBUSB_LIBRARY OR NOT LIBUSB_INCLUDE_DIR)
+         message(FATAL_ERROR "libusb-1.0 dev package required, but not found")
+     endif()
+ 
+-    target_link_libraries(ykcore PRIVATE ${LIBUSB_LIBRARY})
++    target_link_libraries(ykcore PRIVATE Threads::Threads ${LIBUSB_LIBRARY})
+     target_include_directories(ykcore PRIVATE ${LIBUSB_INCLUDE_DIR})
+     target_compile_definitions(ykcore PRIVATE _GNU_SOURCE)
+ elseif(APPLE)
diff --git a/srcpkgs/keepassxc/template b/srcpkgs/keepassxc/template
index f3543633d6da..e5105cc50d56 100644
--- a/srcpkgs/keepassxc/template
+++ b/srcpkgs/keepassxc/template
@@ -1,6 +1,6 @@
 # Template file for 'keepassxc'
 pkgname=keepassxc
-version=2.6.6
+version=2.7.1
 revision=1
 build_style=cmake
 configure_args="-DWITH_TESTS=ON -DWITH_XC_UPDATECHECK=OFF -DWITH_XC_DOCS=ON
@@ -8,16 +8,17 @@ configure_args="-DWITH_TESTS=ON -DWITH_XC_UPDATECHECK=OFF -DWITH_XC_DOCS=ON
  -DWITH_XC_BROWSER=$(vopt_if browser ON OFF)
  -DWITH_XC_FDOSECRETS=$(vopt_if fdosecrets ON OFF)
  -DWITH_XC_KEESHARE=$(vopt_if keeshare ON OFF)
- -DWITH_XC_KEESHARE_SECURE=$(vopt_if keeshare ON OFF)
  -DWITH_XC_NETWORKING=$(vopt_if network ON OFF)
  -DWITH_XC_SSHAGENT=$(vopt_if sshagent ON OFF)
  -DWITH_XC_YUBIKEY=$(vopt_if yubikey ON OFF)"
-hostmakedepends="qt5-qmake qt5-host-tools ruby-asciidoctor"
-makedepends="qt5-tools-devel qt5-svg-devel libgcrypt-devel libargon2-devel
- qrencode-devel readline-devel libsodium-devel
- $(vopt_if autotype 'qt5-x11extras-devel libXtst-devel libXi-devel')
- $(vopt_if keeshare quazip-devel)
- $(vopt_if yubikey 'libykpers-devel libyubikey-devel')"
+hostmakedepends="qt5-devel ruby-asciidoctor"
+makedepends="qt5-tools-devel qt5-svg-devel botan-devel libargon2-devel
+ qrencode-devel readline-devel pcsclite-devel
+ qt5-x11extras-devel
+ $(vopt_if autotype 'libXtst-devel libXi-devel')
+ $(vopt_if keeshare minizip-devel)
+ $(vopt_if yubikey 'libusb-devel')"
+checkdepends="xvfb-run"
 short_desc="KeePassXC is a cross-platform port of “Keepass Password Safe”"
 maintainer="Piraty <piraty1@inbox.ru>"
 license="GPL-3.0-or-later, BSD-3-Clause, CC0-1.0, LGPL-2.0-only, LGPL-2.1-only,
@@ -25,9 +26,8 @@ license="GPL-3.0-or-later, BSD-3-Clause, CC0-1.0, LGPL-2.0-only, LGPL-2.1-only,
 homepage="https://keepassxc.org/"
 changelog="https://raw.githubusercontent.com/keepassxreboot/keepassxc/${version}/CHANGELOG.md"
 distfiles="https://github.com/keepassxreboot/keepassxc/releases/download/${version}/keepassxc-${version}-src.tar.xz"
-checksum=3603b11ac39b289c47fac77fa150e05fd64b393d8cfdf5732dc3ef106650a4e2
+checksum=6001ba626c35c316dbda6de35736f012a2264f95139fcb4a094b8eb49b15d3e7
 
-# https://github.com/keepassxreboot/keepassxc/blob/a775031fe9471310e50232d1861d4991e2803aff/CMakeLists.txt#L46
 build_options="autotype browser fdosecrets keeshare network sshagent yubikey"
 desc_option_autotype="Include auto-type"
 desc_option_browser="Include browser integration with keepassxc-browser-plugin"
@@ -38,8 +38,18 @@ desc_option_sshagent="Include SSH agent support"
 desc_option_yubikey="Include YubiKey support"
 build_options_default="autotype browser fdosecrets keeshare network sshagent yubikey"
 
+# quote droidmonkey from IRC (keepassxc developer)
+# > KeePassXC bundles the ykcore code due to lack of support from Yubico and
+# > stratification of version across various operating system distros.
+# > Additionally, KeePassXC has modified the API of ykcore to make it more
+# > functional when using non-YubiKey keys (ie, OnlyKey)
+
 post_install() {
 	vlicense COPYING
 	vlicense LICENSE.BSD
 	vlicense LICENSE.MIT
 }
+
+do_check() {
+	xvfb-run ninja -C build test || : #XXX: TestCli::testClip() fails
+}

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PR PATCH] [Updated] keepassxc: update to 2.7.1.
  2022-04-12 23:18 [PR PATCH] keepassxc: update to 2.7.1 Piraty
                   ` (6 preceding siblings ...)
  2022-04-15  8:21 ` Piraty
@ 2022-04-19  9:10 ` Piraty
  2022-04-19 19:45 ` Piraty
  2022-04-19 21:01 ` [PR PATCH] [Closed]: " Piraty
  9 siblings, 0 replies; 11+ messages in thread
From: Piraty @ 2022-04-19  9:10 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1254 bytes --]

There is an updated pull request by Piraty against master on the void-packages repository

https://github.com/Piraty/void-packages keepassxc-2.7.1
https://github.com/void-linux/void-packages/pull/36662

keepassxc: update to 2.7.1.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**
@lemmi 

<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!-- 
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


A patch file from https://github.com/void-linux/void-packages/pull/36662.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-keepassxc-2.7.1-36662.patch --]
[-- Type: text/x-diff, Size: 4851 bytes --]

From 1d26c5b3ee62348b49b62ba57d1e3b219d3fa3b7 Mon Sep 17 00:00:00 2001
From: Piraty <piraty1@inbox.ru>
Date: Wed, 13 Apr 2022 00:50:31 +0200
Subject: [PATCH] keepassxc: update to 2.7.1.

---
 .../keepassxc/patches/ykcore-pthread.patch    | 21 +++++++++++++
 srcpkgs/keepassxc/template                    | 30 ++++++++++++-------
 2 files changed, 41 insertions(+), 10 deletions(-)
 create mode 100644 srcpkgs/keepassxc/patches/ykcore-pthread.patch

diff --git a/srcpkgs/keepassxc/patches/ykcore-pthread.patch b/srcpkgs/keepassxc/patches/ykcore-pthread.patch
new file mode 100644
index 000000000000..cc9398f65474
--- /dev/null
+++ b/srcpkgs/keepassxc/patches/ykcore-pthread.patch
@@ -0,0 +1,21 @@
+https://patch-diff.githubusercontent.com/raw/keepassxreboot/keepassxc/pull/7807
+
+--- a/src/thirdparty/ykcore/CMakeLists.txt
++++ b/src/thirdparty/ykcore/CMakeLists.txt
+@@ -27,13 +27,15 @@ if(WIN32)
+ elseif(UNIX AND NOT APPLE)
+     target_sources(ykcore PRIVATE ykcore_libusb-1.0.c)
+ 
++    find_package(Threads REQUIRED)
++
+     find_library(LIBUSB_LIBRARY NAMES usb-1.0)
+     find_path(LIBUSB_INCLUDE_DIR NAMES libusb.h PATH_SUFFIXES "libusb-1.0" "libusb")
+     if(NOT LIBUSB_LIBRARY OR NOT LIBUSB_INCLUDE_DIR)
+         message(FATAL_ERROR "libusb-1.0 dev package required, but not found")
+     endif()
+ 
+-    target_link_libraries(ykcore PRIVATE ${LIBUSB_LIBRARY})
++    target_link_libraries(ykcore PRIVATE Threads::Threads ${LIBUSB_LIBRARY})
+     target_include_directories(ykcore PRIVATE ${LIBUSB_INCLUDE_DIR})
+     target_compile_definitions(ykcore PRIVATE _GNU_SOURCE)
+ elseif(APPLE)
diff --git a/srcpkgs/keepassxc/template b/srcpkgs/keepassxc/template
index f3543633d6da..e5105cc50d56 100644
--- a/srcpkgs/keepassxc/template
+++ b/srcpkgs/keepassxc/template
@@ -1,6 +1,6 @@
 # Template file for 'keepassxc'
 pkgname=keepassxc
-version=2.6.6
+version=2.7.1
 revision=1
 build_style=cmake
 configure_args="-DWITH_TESTS=ON -DWITH_XC_UPDATECHECK=OFF -DWITH_XC_DOCS=ON
@@ -8,16 +8,17 @@ configure_args="-DWITH_TESTS=ON -DWITH_XC_UPDATECHECK=OFF -DWITH_XC_DOCS=ON
  -DWITH_XC_BROWSER=$(vopt_if browser ON OFF)
  -DWITH_XC_FDOSECRETS=$(vopt_if fdosecrets ON OFF)
  -DWITH_XC_KEESHARE=$(vopt_if keeshare ON OFF)
- -DWITH_XC_KEESHARE_SECURE=$(vopt_if keeshare ON OFF)
  -DWITH_XC_NETWORKING=$(vopt_if network ON OFF)
  -DWITH_XC_SSHAGENT=$(vopt_if sshagent ON OFF)
  -DWITH_XC_YUBIKEY=$(vopt_if yubikey ON OFF)"
-hostmakedepends="qt5-qmake qt5-host-tools ruby-asciidoctor"
-makedepends="qt5-tools-devel qt5-svg-devel libgcrypt-devel libargon2-devel
- qrencode-devel readline-devel libsodium-devel
- $(vopt_if autotype 'qt5-x11extras-devel libXtst-devel libXi-devel')
- $(vopt_if keeshare quazip-devel)
- $(vopt_if yubikey 'libykpers-devel libyubikey-devel')"
+hostmakedepends="qt5-devel ruby-asciidoctor"
+makedepends="qt5-tools-devel qt5-svg-devel botan-devel libargon2-devel
+ qrencode-devel readline-devel pcsclite-devel
+ qt5-x11extras-devel
+ $(vopt_if autotype 'libXtst-devel libXi-devel')
+ $(vopt_if keeshare minizip-devel)
+ $(vopt_if yubikey 'libusb-devel')"
+checkdepends="xvfb-run"
 short_desc="KeePassXC is a cross-platform port of “Keepass Password Safe”"
 maintainer="Piraty <piraty1@inbox.ru>"
 license="GPL-3.0-or-later, BSD-3-Clause, CC0-1.0, LGPL-2.0-only, LGPL-2.1-only,
@@ -25,9 +26,8 @@ license="GPL-3.0-or-later, BSD-3-Clause, CC0-1.0, LGPL-2.0-only, LGPL-2.1-only,
 homepage="https://keepassxc.org/"
 changelog="https://raw.githubusercontent.com/keepassxreboot/keepassxc/${version}/CHANGELOG.md"
 distfiles="https://github.com/keepassxreboot/keepassxc/releases/download/${version}/keepassxc-${version}-src.tar.xz"
-checksum=3603b11ac39b289c47fac77fa150e05fd64b393d8cfdf5732dc3ef106650a4e2
+checksum=6001ba626c35c316dbda6de35736f012a2264f95139fcb4a094b8eb49b15d3e7
 
-# https://github.com/keepassxreboot/keepassxc/blob/a775031fe9471310e50232d1861d4991e2803aff/CMakeLists.txt#L46
 build_options="autotype browser fdosecrets keeshare network sshagent yubikey"
 desc_option_autotype="Include auto-type"
 desc_option_browser="Include browser integration with keepassxc-browser-plugin"
@@ -38,8 +38,18 @@ desc_option_sshagent="Include SSH agent support"
 desc_option_yubikey="Include YubiKey support"
 build_options_default="autotype browser fdosecrets keeshare network sshagent yubikey"
 
+# quote droidmonkey from IRC (keepassxc developer)
+# > KeePassXC bundles the ykcore code due to lack of support from Yubico and
+# > stratification of version across various operating system distros.
+# > Additionally, KeePassXC has modified the API of ykcore to make it more
+# > functional when using non-YubiKey keys (ie, OnlyKey)
+
 post_install() {
 	vlicense COPYING
 	vlicense LICENSE.BSD
 	vlicense LICENSE.MIT
 }
+
+do_check() {
+	xvfb-run ninja -C build test || : #XXX: TestCli::testClip() fails
+}

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: keepassxc: update to 2.7.1.
  2022-04-12 23:18 [PR PATCH] keepassxc: update to 2.7.1 Piraty
                   ` (7 preceding siblings ...)
  2022-04-19  9:10 ` Piraty
@ 2022-04-19 19:45 ` Piraty
  2022-04-19 21:01 ` [PR PATCH] [Closed]: " Piraty
  9 siblings, 0 replies; 11+ messages in thread
From: Piraty @ 2022-04-19 19:45 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 195 bytes --]

New comment by Piraty on void-packages repository

https://github.com/void-linux/void-packages/pull/36662#issuecomment-1103038501

Comment:
patch applies fine here, same for CI, so ... \*shrug\*

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PR PATCH] [Closed]: keepassxc: update to 2.7.1.
  2022-04-12 23:18 [PR PATCH] keepassxc: update to 2.7.1 Piraty
                   ` (8 preceding siblings ...)
  2022-04-19 19:45 ` Piraty
@ 2022-04-19 21:01 ` Piraty
  9 siblings, 0 replies; 11+ messages in thread
From: Piraty @ 2022-04-19 21:01 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1092 bytes --]

There's a closed pull request on the void-packages repository

keepassxc: update to 2.7.1.
https://github.com/void-linux/void-packages/pull/36662

Description:
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**
@lemmi 

<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!-- 
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl
-->


^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2022-04-19 21:01 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-12 23:18 [PR PATCH] keepassxc: update to 2.7.1 Piraty
2022-04-12 23:19 ` [PR PATCH] [Updated] " Piraty
2022-04-13  4:58 ` lemmi
2022-04-13  8:16 ` Piraty
2022-04-15  8:07 ` [PR PATCH] [Updated] " Piraty
2022-04-15  8:08 ` Piraty
2022-04-15  8:09 ` [PR PATCH] [Updated] " Piraty
2022-04-15  8:21 ` Piraty
2022-04-19  9:10 ` Piraty
2022-04-19 19:45 ` Piraty
2022-04-19 21:01 ` [PR PATCH] [Closed]: " Piraty

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).