build: Add missing version.h dependency in vnet 62/24362/1
authorChris Luke <chrisy@flirble.org>
Thu, 16 Jan 2020 01:30:04 +0000 (20:30 -0500)
committerChris Luke <chrisy@flirble.org>
Thu, 16 Jan 2020 01:30:04 +0000 (20:30 -0500)
Two modules in vnet include vpp/app/version.h but there is
no explicit build dependency for this generated file. This
leaves a race condition in the build system that the Coverity
build has recently started triggering.

Change-Id: I8e2bb32feeb16e1bdd8efb0d2633cfdba60f51aa
Type: fix
Signed-off-by: Chris Luke <chrisy@flirble.org>
src/vnet/CMakeLists.txt

index 56db742..c2a0dd5 100644 (file)
@@ -1583,7 +1583,7 @@ add_vpp_library(vnet
   INSTALL_HEADERS ${VNET_HEADERS}
   API_FILES ${VNET_API_FILES}
   LINK_LIBRARIES vppinfra svm vlib ${OPENSSL_LIBRARIES}
-  DEPENDS api_headers
+  DEPENDS vpp_version_h api_headers
 )
 
 ##############################################################################