Re: [Openvpn-devel] [PATCH applied] Re: Add a unit test for functions in cryptoapi.c

2023-02-25 Thread Gert Doering
Hi, On Sat, Feb 25, 2023 at 12:23:42PM -0500, Selva Nair wrote: > > OTOH, there might be a bit of polishing needed - the other tests > > print out what they are doing ("Running 7 test(s)"), while the > > cryptoapi unit test "just succeeds". Is there something missing > > wrt cmocka

Re: [Openvpn-devel] [PATCH applied] Re: Add a unit test for functions in cryptoapi.c

2023-02-25 Thread Selva Nair
Hi On Sat, Feb 25, 2023 at 11:29 AM Gert Doering wrote: > Acked-by: Gert Doering > > Thanks for the v4. This enabled me to just push to GH to have > to build and run the tests, without having to bother myself with > copying binaries around :-) > > OTOH, there might be a bit of polishing

[Openvpn-devel] [PATCH applied] Re: configure: fix formatting of --disable-lz4 and --enable-comp-stub

2023-02-25 Thread Gert Doering
Acked-by: Gert Doering "Makes sense". Before: --disable-lzo disable LZO compression support [default=yes] --disable-lz4 Disable LZ4 compression support --enable-comp-stub Don't compile compression support but still allow limited interoperability with

[Openvpn-devel] [PATCH applied] Re: Windows: fix signedness errors with recv/send

2023-02-25 Thread Gert Doering
Acked-by: Gert Doering I'm not a big fan of adding (cast) to silence compiler warnings, but this signed/unsigned char * nonsense is really hard to avoid - casting to (void *) is the right fix here. I have only test compiled on MinGW - and my MinGW did not warn about these anyway (maybe not

[Openvpn-devel] [PATCH applied] Re: Add a unit test for functions in cryptoapi.c

2023-02-25 Thread Gert Doering
Acked-by: Gert Doering Thanks for the v4. This enabled me to just push to GH to have to build and run the tests, without having to bother myself with copying binaries around :-) OTOH, there might be a bit of polishing needed - the other tests print out what they are doing ("Running 7

[Openvpn-devel] [PATCH applied] Re: Add logging for windows driver selection process

2023-02-25 Thread Gert Doering
Acked-by: Gert Doering While I do have no idea about windows drivers and such, having some helpful debug output for "adapter not found" issues is good. I have stared at the code for a bit, and it's all fairly trivial msg()'ing of strings and integers (plus an M_ERRNO added in one of the error

[Openvpn-devel] [PATCH applied] Re: configure: enable DCO by default on FreeBSD/Linux

2023-02-25 Thread Gert Doering
This patch is identical to v3, except that it modifies the github actions to avoid GHA explosions on the linux workers. So recording Selva's and Antonio's ACKs of v3 here on v6. Pushed to GHA and buildbots, all of them are happy now. (We need to remember to add this to Changes.rst, as it might