[Openvpn-devel] [PATCH] GHA: refactor mingw UTs and add missing tls_crypt

2023-07-18 Thread Frank Lichtenheld
I thought instead about moving this into a loop inside powershell. But then error handling becomes annoying. So let's GitHub handle it. Change-Id: I9831f352c07b5c58a4e2d6a3dd5417842e0887d2 Signed-off-by: Frank Lichtenheld --- .github/workflows/build.yaml | 38

[Openvpn-devel] [PATCH] GHA: Add macos-13

2023-07-18 Thread Frank Lichtenheld
Change-Id: Ica2e4b5a6b8da1368b487a33cd4b03ed9fc36011 Signed-off-by: Frank Lichtenheld --- .github/workflows/build.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml index 2ae231ee..1b75a1a7 100644 --- a/.github/work

Re: [Openvpn-devel] [PATCH] GHA: refactor mingw UTs and add missing tls_crypt

2023-07-18 Thread Arne Schwabe
Am 18.07.23 um 12:52 schrieb Frank Lichtenheld: I thought instead about moving this into a loop inside powershell. But then error handling becomes annoying. So let's GitHub handle it. Acked-By: Arne Schwabe I personally do not mind the long list of repeated unit tests in the file but this w

Re: [Openvpn-devel] [PATCH] GHA: Add macos-13

2023-07-18 Thread Arne Schwabe
Am 18.07.23 um 12:52 schrieb Frank Lichtenheld: Change-Id: Ica2e4b5a6b8da1368b487a33cd4b03ed9fc36011 Signed-off-by: Frank Lichtenheld --- .github/workflows/build.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yam

[Openvpn-devel] [PATCH applied] Re: Revert commit 423ced962d

2023-07-18 Thread Gert Doering
Acked-by: Gert Doering This is unfortunate, but the only option if we want to get on with the license change. Not tested individually, but tested together with 2/2, and "everything works as before". Your patch has been applied to the master and release/2.6 branch. commit 370334828659e205941eec

[Openvpn-devel] [PATCH applied] Re: Implement using --peer-fingerprint without CA certificates

2023-07-18 Thread Gert Doering
Acked-by: Gert Doering Tested that things work as before (the t_server testbed has fp tests, both "should succeed" and "should fail", all pass). I did some massaging to the text in the commit message to make the "based on two previous commits" part easier to read. Your patch has been applied to

[Openvpn-devel] [PATCH applied] Re: GHA: refactor mingw UTs and add missing tls_crypt

2023-07-18 Thread Gert Doering
Tested in my GHA instance -> does the job, thanks. (There is lots of individual results in the left column now, but as long as it's all green, it does not really matter... as long as it's not burning 10 times the credits now?) Your patch has been applied to the master branch. commit 26c663f12815

[Openvpn-devel] [PATCH applied] Re: GHA: Add macos-13

2023-07-18 Thread Gert Doering
Also tested in my GH instance, claims to do something, ends up green :-) Your patch has been applied to the master branch. commit 64352467845e7a972a8073d35d41450eea24b905 Author: Frank Lichtenheld Date: Tue Jul 18 12:52:41 2023 +0200 GHA: Add macos-13 Signed-off-by: Frank Lichtenhel