On Tue, Jun 27, 2023 at 08:35:33PM +0200, Gert Doering wrote:
> I have tested this by pushing to GHA (as there are quite significant
> changes to build.yaml) - it seems to still pass, which is good :-)
> 
> I do wonder why all unit tests now produce differently-named .exe
> files (packet_id_testdriver.exe -> test_packet_id.exe etc)?  Is this
> a side effect of a new cmocka version, or...?

No, there is no technical reason for this. I think we can freely choose
the name. We probably just applied the usual naming we use in all our
CMake code. While the autotools buildsystem uses a different naming
scheme.

The old MSVC build had no support for the UTs at all.

> This patch also adds bundling of the CMake related files to tarballs
> (which I missed in 1/5).  Good :-) - besides this, there should be no
> change to "autoconf based building", and there are also no code changes.
> 
> Your patch has been applied to the master branch.
> 
> commit e8881ec6dd63bd80ce05202573eac54ab8657fcb
> Author: Frank Lichtenheld
> Date:   Tue Jun 20 15:53:07 2023 +0200
> 
>      CMake: Add complete MinGW and MSVC build
> 
>      Signed-off-by: Frank Lichtenheld <[email protected]>
>      Acked-by: Lev Stipakov <[email protected]>
>      Message-Id: <[email protected]>
>      URL: 
> https://www.mail-archive.com/[email protected]/msg26754.html
>      Signed-off-by: Gert Doering <[email protected]>

Regards,
-- 
  Frank Lichtenheld


_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to