Not removing the duplicative nacl-vms-pexe target fixes Unvanquished/Unvanquished#3245 (comment)

diff --git a/cmake/DaemonGame.cmake b/cmake/DaemonGame.cmake
index 0a2f70784..c50bb6309 100644
--- a/cmake/DaemonGame.cmake
+++ b/cmake/DaemonGame.cmake
@@ -129,8 +129,9 @@ function(GAMEMODULE)
                     )
                 endforeach()
             else()
-                set(NACL_VMS_PROJECT nacl-vms)
+                set(NACL_VMS_PROJECT nacl-vms-pexe)
                 set(NACL_VMS_PROJECTS ${NACL_VMS_PROJECT})
+                add_dependencies(nacl-vms ${NACL_VMS_PROJECT})
 
                 # Workaround a bug where CMake ExternalProject lists-as-args are cut on first “;”
                 string(REPLACE ";" "," NACL_TARGETS_STRING "${NACL_TARGETS}")

With that patch re-adding the separate generic nacl-vms target and toolchain-specific nacl-vms-pexe target, LGTM.

And actually I like the fact the nacl-vms target just exists to depend on the toolchain-specific targets anyway (and we may rename to no be nacl-centric, so it can be reused for wasm later).

Not removing the duplicative nacl-vms-pexe target fixes Unvanquished/Unvanquished#3245 (comment)

Fixed. The Saigo-specific line creating a dummy target was mistakenly not in its if block.

Not adding back nacl-vms-pexe since there is no reason to have two names for the same target as long as the generic nacl-vms one is available.

It works, you can merge it, but I will add a generic vms target that is independent to the underlying format and toolchain anyway.