Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] eid-mw: update to 5.0.22
@ 2021-06-02 12:10 PoroCYon
  2021-06-02 12:14 ` ericonr
                   ` (9 more replies)
  0 siblings, 10 replies; 11+ messages in thread
From: PoroCYon @ 2021-06-02 12:10 UTC (permalink / raw)
  To: ml

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

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

https://github.com/PoroCYon/void-packages eid-mw-5.0.22
https://github.com/void-linux/void-packages/pull/31260

eid-mw: update to 5.0.22
* new `libassuan-devel` needed (unless we want to use `pinentry` instead?)
  * for cross-compiling, it's needed both as a host and as a target dependency, because the autoconf stuff needs the macros for generating `./configure` from `configure.ac`... this is not ideal, but it's how I got it to work.
* `libbsd` now needed for `strlcpy`
* `autoconf-archive` [now needed as well](https://github.com/Fedict/eid-mw#code-from-git-repository)
* [patch file needed for this issue](https://github.com/Fedict/eid-mw/pull/142/files), should be removed once that's in a new release

-------

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [x] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->

#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [x] I built this PR locally for my native architecture, (x86_64-glibc)
- [x] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [x] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-eid-mw-5.0.22-31260.patch --]
[-- Type: text/x-diff, Size: 3460 bytes --]

From efc84e5afaec7f662ddf629bf1ad2de455f90e0d Mon Sep 17 00:00:00 2001
From: PoroCYon <porocyon@titandemo.org>
Date: Wed, 2 Jun 2021 13:13:10 +0200
Subject: [PATCH] eid-mw: update to 5.0.22

---
 ...-int8_t-errors-when-compiling-for-aa.patch | 58 +++++++++++++++++++
 srcpkgs/eid-mw/template                       |  9 +--
 2 files changed, 63 insertions(+), 4 deletions(-)
 create mode 100644 srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch

diff --git a/srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch b/srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch
new file mode 100644
index 000000000000..32ab243e4fe9
--- /dev/null
+++ b/srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch
@@ -0,0 +1,58 @@
+From 0da930d0d2bf7c3f840cbcdf8d2eb3a411868fae Mon Sep 17 00:00:00 2001
+From: PoroCYon <porocyon@titandemo.org>
+Date: Wed, 2 Jun 2021 13:57:00 +0200
+Subject: [PATCH] fix 'unknown type int8_t' errors when compiling for
+ aarch64-musl
+
+---
+ plugins_tools/eid-viewer/test/testlib.c | 4 ++--
+ tests/unit/testlib.c                    | 4 ++--
+ 2 files changed, 4 insertions(+), 4 deletions(-)
+
+diff --git a/plugins_tools/eid-viewer/test/testlib.c b/plugins_tools/eid-viewer/test/testlib.c
+index 91b381a8..c1794229 100644
+--- a/plugins_tools/eid-viewer/test/testlib.c
++++ b/plugins_tools/eid-viewer/test/testlib.c
+@@ -23,8 +23,6 @@
+ #pragma pack(push, cryptoki, 1)
+ #include "pkcs11.h"
+ #pragma pack(pop, cryptoki)
+-#include <stdio.h>
+-#include <stdint.h>
+ #include <tchar.h>
+ #else
+ #include <unix.h>
+@@ -38,6 +36,8 @@
+ #include <stdarg.h>
+ #include <stdbool.h>
+ #include <errno.h>
++#include <stdio.h>
++#include <stdint.h>
+ 
+ 
+ #include "serial_io.h"
+diff --git a/tests/unit/testlib.c b/tests/unit/testlib.c
+index 32c77a73..0c1b0fe2 100644
+--- a/tests/unit/testlib.c
++++ b/tests/unit/testlib.c
+@@ -23,8 +23,6 @@
+ #pragma pack(push, cryptoki, 1)
+ #include "pkcs11.h"
+ #pragma pack(pop, cryptoki)
+-#include <stdio.h>
+-#include <stdint.h>
+ #include <tchar.h>
+ #else
+ #include <unix.h>
+@@ -37,6 +35,8 @@
+ #include <stdarg.h>
+ #include <stdbool.h>
+ #include <errno.h>
++#include <stdio.h>
++#include <stdint.h>
+ 
+ #include "testlib.h"
+ 
+-- 
+2.31.1
+
diff --git a/srcpkgs/eid-mw/template b/srcpkgs/eid-mw/template
index 0734bdb13018..de96df486d1a 100644
--- a/srcpkgs/eid-mw/template
+++ b/srcpkgs/eid-mw/template
@@ -1,18 +1,19 @@
 # Template file for 'eid-mw'
 pkgname=eid-mw
-version=4.4.16
+version=5.0.22
 revision=4
 build_style=gnu-configure
 hostmakedepends="pkg-config automake gettext gettext-devel libtool glib-devel
- gdk-pixbuf-devel"
-makedepends="gtk+3-devel libcurl-devel openssl-devel libproxy-devel pcsclite-devel"
+ gdk-pixbuf-devel libassuan-devel autoconf-archive"
+makedepends="gtk+3-devel libcurl-devel openssl-devel libproxy-devel
+ pcsclite-devel libassuan-devel libbsd-devel"
 depends="pcsc-acsccid pcsc-ccid pcsc-tools"
 short_desc="Middleware for Belgian eID"
 maintainer="Andrew Benson <abenson+void@gmail.com>"
 license="LGPL-3.0-or-later"
 homepage="https://eid.belgium.be/en"
 distfiles="https://github.com/Fedict/eid-mw/archive/v${version}.tar.gz"
-checksum=bed50b2d3161767ef73cb23d5ce63d5039eebfcc7ae3a6045a22dd3ee07b0357
+checksum=c4ed06a4146de95b10de5056b9c174a85cd71c7a794f2f0faf010d51c8792a59
 
 pre_configure() {
 	autoreconf -fi

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

* Re: eid-mw: update to 5.0.22
  2021-06-02 12:10 [PR PATCH] eid-mw: update to 5.0.22 PoroCYon
@ 2021-06-02 12:14 ` ericonr
  2021-06-02 12:22 ` [PR PATCH] [Updated] " PoroCYon
                   ` (8 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: ericonr @ 2021-06-02 12:14 UTC (permalink / raw)
  To: ml

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/31260#issuecomment-852976267

Comment:
`libbsd` shouldn't be necessary on musl here.

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

* Re: [PR PATCH] [Updated] eid-mw: update to 5.0.22
  2021-06-02 12:10 [PR PATCH] eid-mw: update to 5.0.22 PoroCYon
  2021-06-02 12:14 ` ericonr
@ 2021-06-02 12:22 ` PoroCYon
  2021-06-02 12:23 ` PoroCYon
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: PoroCYon @ 2021-06-02 12:22 UTC (permalink / raw)
  To: ml

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

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

https://github.com/PoroCYon/void-packages eid-mw-5.0.22
https://github.com/void-linux/void-packages/pull/31260

eid-mw: update to 5.0.22
* new `libassuan-devel` needed (unless we want to use `pinentry` instead?)
  * for cross-compiling, it's needed both as a host and as a target dependency, because the autoconf stuff needs the macros for generating `./configure` from `configure.ac`... this is not ideal, but it's how I got it to work.
* `libbsd` now needed for `strlcpy`
* `autoconf-archive` [now needed as well](https://github.com/Fedict/eid-mw#code-from-git-repository)
* [patch file needed for this issue](https://github.com/Fedict/eid-mw/pull/142/files), should be removed once that's in a new release

-------

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [x] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->

#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [x] I built this PR locally for my native architecture, (x86_64-glibc)
- [x] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [x] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-eid-mw-5.0.22-31260.patch --]
[-- Type: text/x-diff, Size: 3516 bytes --]

From 74736b671a86ae4d25c20effdde49686e2931b3f Mon Sep 17 00:00:00 2001
From: PoroCYon <porocyon@titandemo.org>
Date: Wed, 2 Jun 2021 13:13:10 +0200
Subject: [PATCH] eid-mw: update to 5.0.22

---
 ...-int8_t-errors-when-compiling-for-aa.patch | 58 +++++++++++++++++++
 srcpkgs/eid-mw/template                       | 14 +++--
 2 files changed, 67 insertions(+), 5 deletions(-)
 create mode 100644 srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch

diff --git a/srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch b/srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch
new file mode 100644
index 000000000000..87a21def3d87
--- /dev/null
+++ b/srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch
@@ -0,0 +1,58 @@
+From 0da930d0d2bf7c3f840cbcdf8d2eb3a411868fae Mon Sep 17 00:00:00 2001
+From: PoroCYon <porocyon@titandemo.org>
+Date: Wed, 2 Jun 2021 13:57:00 +0200
+Subject: [PATCH] fix 'unknown type int8_t' errors when compiling for
+ aarch64-musl
+
+---
+ plugins_tools/eid-viewer/test/testlib.c | 4 ++--
+ tests/unit/testlib.c                    | 4 ++--
+ 2 files changed, 4 insertions(+), 4 deletions(-)
+
+diff --git plugins_tools/eid-viewer/test/testlib.c plugins_tools/eid-viewer/test/testlib.c
+index 91b381a8..c1794229 100644
+--- plugins_tools/eid-viewer/test/testlib.c
++++ plugins_tools/eid-viewer/test/testlib.c
+@@ -23,8 +23,6 @@
+ #pragma pack(push, cryptoki, 1)
+ #include "pkcs11.h"
+ #pragma pack(pop, cryptoki)
+-#include <stdio.h>
+-#include <stdint.h>
+ #include <tchar.h>
+ #else
+ #include <unix.h>
+@@ -38,6 +36,8 @@
+ #include <stdarg.h>
+ #include <stdbool.h>
+ #include <errno.h>
++#include <stdio.h>
++#include <stdint.h>
+ 
+ 
+ #include "serial_io.h"
+diff --git tests/unit/testlib.c tests/unit/testlib.c
+index 32c77a73..0c1b0fe2 100644
+--- tests/unit/testlib.c
++++ tests/unit/testlib.c
+@@ -23,8 +23,6 @@
+ #pragma pack(push, cryptoki, 1)
+ #include "pkcs11.h"
+ #pragma pack(pop, cryptoki)
+-#include <stdio.h>
+-#include <stdint.h>
+ #include <tchar.h>
+ #else
+ #include <unix.h>
+@@ -37,6 +35,8 @@
+ #include <stdarg.h>
+ #include <stdbool.h>
+ #include <errno.h>
++#include <stdio.h>
++#include <stdint.h>
+ 
+ #include "testlib.h"
+ 
+-- 
+2.31.1
+
diff --git a/srcpkgs/eid-mw/template b/srcpkgs/eid-mw/template
index 0734bdb13018..ec94fc457619 100644
--- a/srcpkgs/eid-mw/template
+++ b/srcpkgs/eid-mw/template
@@ -1,18 +1,22 @@
 # Template file for 'eid-mw'
 pkgname=eid-mw
-version=4.4.16
-revision=4
+version=5.0.22
+revision=1
 build_style=gnu-configure
 hostmakedepends="pkg-config automake gettext gettext-devel libtool glib-devel
- gdk-pixbuf-devel"
-makedepends="gtk+3-devel libcurl-devel openssl-devel libproxy-devel pcsclite-devel"
+ gdk-pixbuf-devel libassuan-devel autoconf-archive"
+makedepends="gtk+3-devel libcurl-devel openssl-devel libproxy-devel
+ pcsclite-devel libassuan-devel"
+if [ "$XBPS_LIBC" = "glibc" ]; then
+	makedepends+="libbsd-devel"
+fi
 depends="pcsc-acsccid pcsc-ccid pcsc-tools"
 short_desc="Middleware for Belgian eID"
 maintainer="Andrew Benson <abenson+void@gmail.com>"
 license="LGPL-3.0-or-later"
 homepage="https://eid.belgium.be/en"
 distfiles="https://github.com/Fedict/eid-mw/archive/v${version}.tar.gz"
-checksum=bed50b2d3161767ef73cb23d5ce63d5039eebfcc7ae3a6045a22dd3ee07b0357
+checksum=c4ed06a4146de95b10de5056b9c174a85cd71c7a794f2f0faf010d51c8792a59
 
 pre_configure() {
 	autoreconf -fi

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

* Re: [PR PATCH] [Updated] eid-mw: update to 5.0.22
  2021-06-02 12:10 [PR PATCH] eid-mw: update to 5.0.22 PoroCYon
  2021-06-02 12:14 ` ericonr
  2021-06-02 12:22 ` [PR PATCH] [Updated] " PoroCYon
@ 2021-06-02 12:23 ` PoroCYon
  2021-06-02 12:24 ` PoroCYon
                   ` (6 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: PoroCYon @ 2021-06-02 12:23 UTC (permalink / raw)
  To: ml

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

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

https://github.com/PoroCYon/void-packages eid-mw-5.0.22
https://github.com/void-linux/void-packages/pull/31260

eid-mw: update to 5.0.22
* new `libassuan-devel` needed (unless we want to use `pinentry` instead?)
  * for cross-compiling, it's needed both as a host and as a target dependency, because the autoconf stuff needs the macros for generating `./configure` from `configure.ac`... this is not ideal, but it's how I got it to work.
* `libbsd` now needed for `strlcpy`
* `autoconf-archive` [now needed as well](https://github.com/Fedict/eid-mw#code-from-git-repository)
* [patch file needed for this issue](https://github.com/Fedict/eid-mw/pull/142/files), should be removed once that's in a new release

-------

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [x] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->

#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [x] I built this PR locally for my native architecture, (x86_64-glibc)
- [x] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [x] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-eid-mw-5.0.22-31260.patch --]
[-- Type: text/x-diff, Size: 3518 bytes --]

From c338f6cea62e697906bb11879b6618c8aaf152dc Mon Sep 17 00:00:00 2001
From: PoroCYon <porocyon@titandemo.org>
Date: Wed, 2 Jun 2021 13:13:10 +0200
Subject: [PATCH] eid-mw: update to 5.0.22

---
 ...-int8_t-errors-when-compiling-for-aa.patch | 58 +++++++++++++++++++
 srcpkgs/eid-mw/template                       | 15 +++--
 2 files changed, 68 insertions(+), 5 deletions(-)
 create mode 100644 srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch

diff --git a/srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch b/srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch
new file mode 100644
index 000000000000..87a21def3d87
--- /dev/null
+++ b/srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch
@@ -0,0 +1,58 @@
+From 0da930d0d2bf7c3f840cbcdf8d2eb3a411868fae Mon Sep 17 00:00:00 2001
+From: PoroCYon <porocyon@titandemo.org>
+Date: Wed, 2 Jun 2021 13:57:00 +0200
+Subject: [PATCH] fix 'unknown type int8_t' errors when compiling for
+ aarch64-musl
+
+---
+ plugins_tools/eid-viewer/test/testlib.c | 4 ++--
+ tests/unit/testlib.c                    | 4 ++--
+ 2 files changed, 4 insertions(+), 4 deletions(-)
+
+diff --git plugins_tools/eid-viewer/test/testlib.c plugins_tools/eid-viewer/test/testlib.c
+index 91b381a8..c1794229 100644
+--- plugins_tools/eid-viewer/test/testlib.c
++++ plugins_tools/eid-viewer/test/testlib.c
+@@ -23,8 +23,6 @@
+ #pragma pack(push, cryptoki, 1)
+ #include "pkcs11.h"
+ #pragma pack(pop, cryptoki)
+-#include <stdio.h>
+-#include <stdint.h>
+ #include <tchar.h>
+ #else
+ #include <unix.h>
+@@ -38,6 +36,8 @@
+ #include <stdarg.h>
+ #include <stdbool.h>
+ #include <errno.h>
++#include <stdio.h>
++#include <stdint.h>
+ 
+ 
+ #include "serial_io.h"
+diff --git tests/unit/testlib.c tests/unit/testlib.c
+index 32c77a73..0c1b0fe2 100644
+--- tests/unit/testlib.c
++++ tests/unit/testlib.c
+@@ -23,8 +23,6 @@
+ #pragma pack(push, cryptoki, 1)
+ #include "pkcs11.h"
+ #pragma pack(pop, cryptoki)
+-#include <stdio.h>
+-#include <stdint.h>
+ #include <tchar.h>
+ #else
+ #include <unix.h>
+@@ -37,6 +35,8 @@
+ #include <stdarg.h>
+ #include <stdbool.h>
+ #include <errno.h>
++#include <stdio.h>
++#include <stdint.h>
+ 
+ #include "testlib.h"
+ 
+-- 
+2.31.1
+
diff --git a/srcpkgs/eid-mw/template b/srcpkgs/eid-mw/template
index 0734bdb13018..5c1bd4c92c7f 100644
--- a/srcpkgs/eid-mw/template
+++ b/srcpkgs/eid-mw/template
@@ -1,18 +1,23 @@
 # Template file for 'eid-mw'
 pkgname=eid-mw
-version=4.4.16
-revision=4
+version=5.0.22
+revision=1
 build_style=gnu-configure
 hostmakedepends="pkg-config automake gettext gettext-devel libtool glib-devel
- gdk-pixbuf-devel"
-makedepends="gtk+3-devel libcurl-devel openssl-devel libproxy-devel pcsclite-devel"
+ gdk-pixbuf-devel libassuan-devel autoconf-archive"
+makedepends="gtk+3-devel libcurl-devel openssl-devel libproxy-devel
+ pcsclite-devel libassuan-devel"
 depends="pcsc-acsccid pcsc-ccid pcsc-tools"
 short_desc="Middleware for Belgian eID"
 maintainer="Andrew Benson <abenson+void@gmail.com>"
 license="LGPL-3.0-or-later"
 homepage="https://eid.belgium.be/en"
 distfiles="https://github.com/Fedict/eid-mw/archive/v${version}.tar.gz"
-checksum=bed50b2d3161767ef73cb23d5ce63d5039eebfcc7ae3a6045a22dd3ee07b0357
+checksum=c4ed06a4146de95b10de5056b9c174a85cd71c7a794f2f0faf010d51c8792a59
+
+if [ "$XBPS_LIBC" = "glibc" ]; then
+	makedepends+="libbsd-devel"
+fi
 
 pre_configure() {
 	autoreconf -fi

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

* Re: eid-mw: update to 5.0.22
  2021-06-02 12:10 [PR PATCH] eid-mw: update to 5.0.22 PoroCYon
                   ` (2 preceding siblings ...)
  2021-06-02 12:23 ` PoroCYon
@ 2021-06-02 12:24 ` PoroCYon
  2021-06-02 13:29 ` [PR REVIEW] " ericonr
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: PoroCYon @ 2021-06-02 12:24 UTC (permalink / raw)
  To: ml

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

New comment by PoroCYon on void-packages repository

https://github.com/void-linux/void-packages/pull/31260#issuecomment-852982674

Comment:
ok, added a check for that (+ fixed the patch, and made xlint happy)

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

* Re: [PR REVIEW] eid-mw: update to 5.0.22
  2021-06-02 12:10 [PR PATCH] eid-mw: update to 5.0.22 PoroCYon
                   ` (3 preceding siblings ...)
  2021-06-02 12:24 ` PoroCYon
@ 2021-06-02 13:29 ` ericonr
  2021-06-02 13:29 ` ericonr
                   ` (4 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: ericonr @ 2021-06-02 13:29 UTC (permalink / raw)
  To: ml

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

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/31260#discussion_r643962549

Comment:
should be `XBPS_TARGET_LIBC`

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

* Re: [PR REVIEW] eid-mw: update to 5.0.22
  2021-06-02 12:10 [PR PATCH] eid-mw: update to 5.0.22 PoroCYon
                   ` (4 preceding siblings ...)
  2021-06-02 13:29 ` [PR REVIEW] " ericonr
@ 2021-06-02 13:29 ` ericonr
  2021-06-02 13:52 ` [PR PATCH] [Updated] " PoroCYon
                   ` (3 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: ericonr @ 2021-06-02 13:29 UTC (permalink / raw)
  To: ml

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

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/31260#discussion_r643962689

Comment:
```suggestion
	makedepends+=" libbsd-devel"
```

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

* Re: [PR PATCH] [Updated] eid-mw: update to 5.0.22
  2021-06-02 12:10 [PR PATCH] eid-mw: update to 5.0.22 PoroCYon
                   ` (5 preceding siblings ...)
  2021-06-02 13:29 ` ericonr
@ 2021-06-02 13:52 ` PoroCYon
  2021-06-02 13:52 ` PoroCYon
                   ` (2 subsequent siblings)
  9 siblings, 0 replies; 11+ messages in thread
From: PoroCYon @ 2021-06-02 13:52 UTC (permalink / raw)
  To: ml

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

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

https://github.com/PoroCYon/void-packages eid-mw-5.0.22
https://github.com/void-linux/void-packages/pull/31260

eid-mw: update to 5.0.22
* new `libassuan-devel` needed (unless we want to use `pinentry` instead?)
  * for cross-compiling, it's needed both as a host and as a target dependency, because the autoconf stuff needs the macros for generating `./configure` from `configure.ac`... this is not ideal, but it's how I got it to work.
* `libbsd` now needed for `strlcpy`
* `autoconf-archive` [now needed as well](https://github.com/Fedict/eid-mw#code-from-git-repository)
* [patch file needed for this issue](https://github.com/Fedict/eid-mw/pull/142/files), should be removed once that's in a new release

-------

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [x] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->

#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [x] I built this PR locally for my native architecture, (x86_64-glibc)
- [x] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [x] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-eid-mw-5.0.22-31260.patch --]
[-- Type: text/x-diff, Size: 3526 bytes --]

From 0fd3bc2cadfd711747b36ca3fcd7cac4cca837f4 Mon Sep 17 00:00:00 2001
From: PoroCYon <porocyon@titandemo.org>
Date: Wed, 2 Jun 2021 13:13:10 +0200
Subject: [PATCH] eid-mw: update to 5.0.22

---
 ...-int8_t-errors-when-compiling-for-aa.patch | 58 +++++++++++++++++++
 srcpkgs/eid-mw/template                       | 15 +++--
 2 files changed, 68 insertions(+), 5 deletions(-)
 create mode 100644 srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch

diff --git a/srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch b/srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch
new file mode 100644
index 000000000000..87a21def3d87
--- /dev/null
+++ b/srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch
@@ -0,0 +1,58 @@
+From 0da930d0d2bf7c3f840cbcdf8d2eb3a411868fae Mon Sep 17 00:00:00 2001
+From: PoroCYon <porocyon@titandemo.org>
+Date: Wed, 2 Jun 2021 13:57:00 +0200
+Subject: [PATCH] fix 'unknown type int8_t' errors when compiling for
+ aarch64-musl
+
+---
+ plugins_tools/eid-viewer/test/testlib.c | 4 ++--
+ tests/unit/testlib.c                    | 4 ++--
+ 2 files changed, 4 insertions(+), 4 deletions(-)
+
+diff --git plugins_tools/eid-viewer/test/testlib.c plugins_tools/eid-viewer/test/testlib.c
+index 91b381a8..c1794229 100644
+--- plugins_tools/eid-viewer/test/testlib.c
++++ plugins_tools/eid-viewer/test/testlib.c
+@@ -23,8 +23,6 @@
+ #pragma pack(push, cryptoki, 1)
+ #include "pkcs11.h"
+ #pragma pack(pop, cryptoki)
+-#include <stdio.h>
+-#include <stdint.h>
+ #include <tchar.h>
+ #else
+ #include <unix.h>
+@@ -38,6 +36,8 @@
+ #include <stdarg.h>
+ #include <stdbool.h>
+ #include <errno.h>
++#include <stdio.h>
++#include <stdint.h>
+ 
+ 
+ #include "serial_io.h"
+diff --git tests/unit/testlib.c tests/unit/testlib.c
+index 32c77a73..0c1b0fe2 100644
+--- tests/unit/testlib.c
++++ tests/unit/testlib.c
+@@ -23,8 +23,6 @@
+ #pragma pack(push, cryptoki, 1)
+ #include "pkcs11.h"
+ #pragma pack(pop, cryptoki)
+-#include <stdio.h>
+-#include <stdint.h>
+ #include <tchar.h>
+ #else
+ #include <unix.h>
+@@ -37,6 +35,8 @@
+ #include <stdarg.h>
+ #include <stdbool.h>
+ #include <errno.h>
++#include <stdio.h>
++#include <stdint.h>
+ 
+ #include "testlib.h"
+ 
+-- 
+2.31.1
+
diff --git a/srcpkgs/eid-mw/template b/srcpkgs/eid-mw/template
index 0734bdb13018..93504909b24d 100644
--- a/srcpkgs/eid-mw/template
+++ b/srcpkgs/eid-mw/template
@@ -1,18 +1,23 @@
 # Template file for 'eid-mw'
 pkgname=eid-mw
-version=4.4.16
-revision=4
+version=5.0.22
+revision=1
 build_style=gnu-configure
 hostmakedepends="pkg-config automake gettext gettext-devel libtool glib-devel
- gdk-pixbuf-devel"
-makedepends="gtk+3-devel libcurl-devel openssl-devel libproxy-devel pcsclite-devel"
+ gdk-pixbuf-devel libassuan-devel autoconf-archive"
+makedepends="gtk+3-devel libcurl-devel openssl-devel libproxy-devel
+ pcsclite-devel libassuan-devel"
 depends="pcsc-acsccid pcsc-ccid pcsc-tools"
 short_desc="Middleware for Belgian eID"
 maintainer="Andrew Benson <abenson+void@gmail.com>"
 license="LGPL-3.0-or-later"
 homepage="https://eid.belgium.be/en"
 distfiles="https://github.com/Fedict/eid-mw/archive/v${version}.tar.gz"
-checksum=bed50b2d3161767ef73cb23d5ce63d5039eebfcc7ae3a6045a22dd3ee07b0357
+checksum=c4ed06a4146de95b10de5056b9c174a85cd71c7a794f2f0faf010d51c8792a59
+
+if [ "$XBPS_TARGET_LIBC" = "glibc" ]; then
+	makedepends+= "libbsd-devel"
+fi
 
 pre_configure() {
 	autoreconf -fi

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

* Re: [PR PATCH] [Updated] eid-mw: update to 5.0.22
  2021-06-02 12:10 [PR PATCH] eid-mw: update to 5.0.22 PoroCYon
                   ` (6 preceding siblings ...)
  2021-06-02 13:52 ` [PR PATCH] [Updated] " PoroCYon
@ 2021-06-02 13:52 ` PoroCYon
  2021-06-14  3:11 ` [PR PATCH] [Merged]: " ericonr
  2021-06-14  3:11 ` ericonr
  9 siblings, 0 replies; 11+ messages in thread
From: PoroCYon @ 2021-06-02 13:52 UTC (permalink / raw)
  To: ml

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

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

https://github.com/PoroCYon/void-packages eid-mw-5.0.22
https://github.com/void-linux/void-packages/pull/31260

eid-mw: update to 5.0.22
* new `libassuan-devel` needed (unless we want to use `pinentry` instead?)
  * for cross-compiling, it's needed both as a host and as a target dependency, because the autoconf stuff needs the macros for generating `./configure` from `configure.ac`... this is not ideal, but it's how I got it to work.
* `libbsd` now needed for `strlcpy`
* `autoconf-archive` [now needed as well](https://github.com/Fedict/eid-mw#code-from-git-repository)
* [patch file needed for this issue](https://github.com/Fedict/eid-mw/pull/142/files), should be removed once that's in a new release

-------

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [x] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->

#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [x] I built this PR locally for my native architecture, (x86_64-glibc)
- [x] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [x] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-eid-mw-5.0.22-31260.patch --]
[-- Type: text/x-diff, Size: 3526 bytes --]

From 964a327436805c2579e738b434826adf74370ecd Mon Sep 17 00:00:00 2001
From: PoroCYon <porocyon@titandemo.org>
Date: Wed, 2 Jun 2021 13:13:10 +0200
Subject: [PATCH] eid-mw: update to 5.0.22

---
 ...-int8_t-errors-when-compiling-for-aa.patch | 58 +++++++++++++++++++
 srcpkgs/eid-mw/template                       | 15 +++--
 2 files changed, 68 insertions(+), 5 deletions(-)
 create mode 100644 srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch

diff --git a/srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch b/srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch
new file mode 100644
index 000000000000..87a21def3d87
--- /dev/null
+++ b/srcpkgs/eid-mw/patches/0001-fix-unknown-type-int8_t-errors-when-compiling-for-aa.patch
@@ -0,0 +1,58 @@
+From 0da930d0d2bf7c3f840cbcdf8d2eb3a411868fae Mon Sep 17 00:00:00 2001
+From: PoroCYon <porocyon@titandemo.org>
+Date: Wed, 2 Jun 2021 13:57:00 +0200
+Subject: [PATCH] fix 'unknown type int8_t' errors when compiling for
+ aarch64-musl
+
+---
+ plugins_tools/eid-viewer/test/testlib.c | 4 ++--
+ tests/unit/testlib.c                    | 4 ++--
+ 2 files changed, 4 insertions(+), 4 deletions(-)
+
+diff --git plugins_tools/eid-viewer/test/testlib.c plugins_tools/eid-viewer/test/testlib.c
+index 91b381a8..c1794229 100644
+--- plugins_tools/eid-viewer/test/testlib.c
++++ plugins_tools/eid-viewer/test/testlib.c
+@@ -23,8 +23,6 @@
+ #pragma pack(push, cryptoki, 1)
+ #include "pkcs11.h"
+ #pragma pack(pop, cryptoki)
+-#include <stdio.h>
+-#include <stdint.h>
+ #include <tchar.h>
+ #else
+ #include <unix.h>
+@@ -38,6 +36,8 @@
+ #include <stdarg.h>
+ #include <stdbool.h>
+ #include <errno.h>
++#include <stdio.h>
++#include <stdint.h>
+ 
+ 
+ #include "serial_io.h"
+diff --git tests/unit/testlib.c tests/unit/testlib.c
+index 32c77a73..0c1b0fe2 100644
+--- tests/unit/testlib.c
++++ tests/unit/testlib.c
+@@ -23,8 +23,6 @@
+ #pragma pack(push, cryptoki, 1)
+ #include "pkcs11.h"
+ #pragma pack(pop, cryptoki)
+-#include <stdio.h>
+-#include <stdint.h>
+ #include <tchar.h>
+ #else
+ #include <unix.h>
+@@ -37,6 +35,8 @@
+ #include <stdarg.h>
+ #include <stdbool.h>
+ #include <errno.h>
++#include <stdio.h>
++#include <stdint.h>
+ 
+ #include "testlib.h"
+ 
+-- 
+2.31.1
+
diff --git a/srcpkgs/eid-mw/template b/srcpkgs/eid-mw/template
index 0734bdb13018..9c3b471009f3 100644
--- a/srcpkgs/eid-mw/template
+++ b/srcpkgs/eid-mw/template
@@ -1,18 +1,23 @@
 # Template file for 'eid-mw'
 pkgname=eid-mw
-version=4.4.16
-revision=4
+version=5.0.22
+revision=1
 build_style=gnu-configure
 hostmakedepends="pkg-config automake gettext gettext-devel libtool glib-devel
- gdk-pixbuf-devel"
-makedepends="gtk+3-devel libcurl-devel openssl-devel libproxy-devel pcsclite-devel"
+ gdk-pixbuf-devel libassuan-devel autoconf-archive"
+makedepends="gtk+3-devel libcurl-devel openssl-devel libproxy-devel
+ pcsclite-devel libassuan-devel"
 depends="pcsc-acsccid pcsc-ccid pcsc-tools"
 short_desc="Middleware for Belgian eID"
 maintainer="Andrew Benson <abenson+void@gmail.com>"
 license="LGPL-3.0-or-later"
 homepage="https://eid.belgium.be/en"
 distfiles="https://github.com/Fedict/eid-mw/archive/v${version}.tar.gz"
-checksum=bed50b2d3161767ef73cb23d5ce63d5039eebfcc7ae3a6045a22dd3ee07b0357
+checksum=c4ed06a4146de95b10de5056b9c174a85cd71c7a794f2f0faf010d51c8792a59
+
+if [ "$XBPS_TARGET_LIBC" = "glibc" ]; then
+	makedepends+=" libbsd-devel"
+fi
 
 pre_configure() {
 	autoreconf -fi

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

* Re: [PR PATCH] [Merged]: eid-mw: update to 5.0.22
  2021-06-02 12:10 [PR PATCH] eid-mw: update to 5.0.22 PoroCYon
                   ` (7 preceding siblings ...)
  2021-06-02 13:52 ` PoroCYon
@ 2021-06-14  3:11 ` ericonr
  2021-06-14  3:11 ` ericonr
  9 siblings, 0 replies; 11+ messages in thread
From: ericonr @ 2021-06-14  3:11 UTC (permalink / raw)
  To: ml

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

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

eid-mw: update to 5.0.22
https://github.com/void-linux/void-packages/pull/31260

Description:
* new `libassuan-devel` needed (unless we want to use `pinentry` instead?)
  * for cross-compiling, it's needed both as a host and as a target dependency, because the autoconf stuff needs the macros for generating `./configure` from `configure.ac`... this is not ideal, but it's how I got it to work.
* `libbsd` now needed for `strlcpy`
* `autoconf-archive` [now needed as well](https://github.com/Fedict/eid-mw#code-from-git-repository)
* [patch file needed for this issue](https://github.com/Fedict/eid-mw/pull/142/files), should be removed once that's in a new release

-------

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [x] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->

#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [x] I built this PR locally for my native architecture, (x86_64-glibc)
- [x] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [x] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl


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

* Re: eid-mw: update to 5.0.22
  2021-06-02 12:10 [PR PATCH] eid-mw: update to 5.0.22 PoroCYon
                   ` (8 preceding siblings ...)
  2021-06-14  3:11 ` [PR PATCH] [Merged]: " ericonr
@ 2021-06-14  3:11 ` ericonr
  9 siblings, 0 replies; 11+ messages in thread
From: ericonr @ 2021-06-14  3:11 UTC (permalink / raw)
  To: ml

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/31260#issuecomment-860335238

Comment:
Thanks!

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

end of thread, other threads:[~2021-06-14  3:11 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-02 12:10 [PR PATCH] eid-mw: update to 5.0.22 PoroCYon
2021-06-02 12:14 ` ericonr
2021-06-02 12:22 ` [PR PATCH] [Updated] " PoroCYon
2021-06-02 12:23 ` PoroCYon
2021-06-02 12:24 ` PoroCYon
2021-06-02 13:29 ` [PR REVIEW] " ericonr
2021-06-02 13:29 ` ericonr
2021-06-02 13:52 ` [PR PATCH] [Updated] " PoroCYon
2021-06-02 13:52 ` PoroCYon
2021-06-14  3:11 ` [PR PATCH] [Merged]: " ericonr
2021-06-14  3:11 ` ericonr

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).