Browse code

fix building of openvpnsrvmsg.dll from eventmsg.mc in mingw builds

commit 06919a60ae61 introduces .mc files that need to be compiled to
.h and .bin by the windows "mc.exe" tool, and from there into a new
.dll. This worked for MSVC builds, did nothing for cmake/mingw builds,
and broke compilation on autoconf/mingw builds.

This patch consists of two parts:

1. add building of openvpnsrvmsg.dll to autoconf/mingw builds

Add logic to configure.ac to find the "windmc" binary in the linux or
mingw variants, add rules to src/openvpnserv/Makefile.am so make knows
what to do.

Libtool is getting in the way when "openvpnsrvmsg.dll" is created as
anything listed in ...BIN or ...LIB, so decare it as "DATA" and make
the necessary rules explicit.

2. fix building of openvpnsrvmsg.dll on cmake/mingw builds

Fix "find_program()" invocation to avoid using "midnight commander"
binary (mc) on Linux (called "windmc" there).

Change from "-Wl,--noentry" to linker invocation that works.

See also:
https://learn.microsoft.com/en-us/cpp/build/creating-a-resource-only-dll?view=msvc-170

Change-Id: I071e8190dac28f429257b8af1c6f9e68f8896bc0
Signed-off-by: Gert Doering <gert@greenie.muc.de>
Acked-by: Frank Lichtenheld <frank@lichtenheld.com>
Gerrit URL: https://gerrit.openvpn.net/c/openvpn/+/1197
Message-Id: <20250919112424.24728-1-gert@greenie.muc.de>
URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg33083.html
Signed-off-by: Gert Doering <gert@greenie.muc.de>

Gert Doering authored on 2025/09/19 20:24:19
Showing 3 changed files
... ...
@@ -441,6 +441,7 @@ AC_DEFINE_UNQUOTED([IFCONFIG_PATH], ["$IFCONFIG"], [Path to ifconfig tool])
441 441
 AC_DEFINE_UNQUOTED([IPROUTE_PATH], ["$IPROUTE"], [Path to iproute tool])
442 442
 AC_DEFINE_UNQUOTED([ROUTE_PATH], ["$ROUTE"], [Path to route tool])
443 443
 AC_DEFINE_UNQUOTED([SYSTEMD_ASK_PASSWORD_PATH], ["$SYSTEMD_ASK_PASSWORD"], [Path to systemd-ask-password tool])
444
+AC_CHECK_TOOLS([WINDMC], [windmc mc.exe],[no])
444 445
 
445 446
 #
446 447
 #  man page generation - based on python-docutils
... ...
@@ -41,7 +41,7 @@ endif ()
41 41
 file(MAKE_DIRECTORY ${MC_GEN_DIR})
42 42
 set(MC_FILE ${CMAKE_CURRENT_SOURCE_DIR}/eventmsg.mc)
43 43
 
44
-find_program(MC_COMPILER NAMES mc mc.exe x86_64-w64-mingw32-windmc i686-w64-mingw32-windmc windmc)
44
+find_program(MC_COMPILER NAMES mc.exe x86_64-w64-mingw32-windmc i686-w64-mingw32-windmc windmc)
45 45
 
46 46
 if (NOT MC_COMPILER)
47 47
     message(FATAL_ERROR "No message compiler found.")
... ...
@@ -58,12 +58,17 @@ add_custom_command(
58 58
 add_custom_target(msg_mc_gen ALL DEPENDS ${MC_GEN_DIR}/eventmsg.rc ${MC_GEN_DIR}/eventmsg.h)
59 59
 
60 60
 add_library(openvpnservmsg SHARED ${MC_GEN_DIR}/eventmsg.rc)
61
+add_dependencies(openvpnservmsg msg_mc_gen)
61 62
 
62 63
 if (MSVC)
63 64
     set_target_properties(openvpnservmsg PROPERTIES LINK_FLAGS "/NOENTRY")
64 65
 else()
65
-    target_link_options(openvpnservmsg PRIVATE "-Wl,--no-entry")
66
+    set_target_properties(openvpnservmsg PROPERTIES LINKER_LANGUAGE C OUTPUT_NAME "openvpnservmsg")
67
+    target_link_options(openvpnservmsg PRIVATE
68
+        -Wl,--entry=0
69
+        -nostdlib
70
+        -nostartfiles
71
+    )
66 72
 endif()
67 73
 
68
-add_dependencies(openvpnservmsg msg_mc_gen)
69 74
 add_dependencies(openvpnserv msg_mc_gen)
... ...
@@ -28,6 +28,10 @@ openvpnserv_CFLAGS = \
28 28
 openvpnserv_LDADD = \
29 29
 	-ladvapi32 -luserenv -liphlpapi -lfwpuclnt -lrpcrt4 \
30 30
 	-lshlwapi -lnetapi32 -lws2_32 -lntdll -lole32
31
+noinst_DATA = \
32
+	MSG00409.bin eventmsg.h eventmsg.rc openvpnservmsg.dll
33
+BUILT_SOURCES = \
34
+	eventmsg.h
31 35
 endif
32 36
 
33 37
 openvpnserv_SOURCES = \
... ...
@@ -37,3 +41,11 @@ openvpnserv_SOURCES = \
37 37
 	validate.c validate.h \
38 38
 	$(top_srcdir)/src/openvpn/wfp_block.c $(top_srcdir)/src/openvpn/wfp_block.h \
39 39
 	openvpnserv_resources.rc
40
+
41
+openvpnservmsg.dll: eventmsg.o
42
+	$(CC) -shared -Wl,--entry=0 -nostdlib -nostartfiles -o $@ $<
43
+
44
+eventmsg.o: eventmsg.rc
45
+
46
+eventmsg.h: eventmsg.mc
47
+	$(WINDMC) -U $<