Only warn about INPUT_REQUIREMENTS if a package exports requirements …

…to begin with
locusrobotics · May 27, 2020 · 488fa2b · 488fa2b
1 parent fc4379c
commit 488fa2b
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/catkin_virtualenv/cmake/catkin_generate_virtualenv.cmake b/catkin_virtualenv/cmake/catkin_generate_virtualenv.cmake
@@ -42,10 +42,6 @@ function(catkin_generate_virtualenv)
     set(lock_args "${lock_args} --no-deps")
   endif()
 
-  if(NOT DEFINED ARG_INPUT_REQUIREMENTS)
-    message(WARNING "Please define an INPUT_REQUIREMENTS and generate a lock file - see https://github.com/locusrobotics/catkin_virtualenv/blob/master/README.md#locking-dependencies")
-  endif()
-
   if (NOT DEFINED ARG_EXTRA_PIP_ARGS)
     set(ARG_EXTRA_PIP_ARGS "-qq" "--retries 10" "--timeout 30")
   endif()
@@ -102,6 +98,10 @@ function(catkin_generate_virtualenv)
         ${CMAKE_BINARY_DIR}/${venv_dir}/bin/python
         ${CMAKE_SOURCE_DIR}/${ARG_INPUT_REQUIREMENTS}
     )
+
+  elseif(NOT DEFINED ARG_INPUT_REQUIREMENTS AND NOT package_requirements STREQUAL "")
+    message(WARNING "Please define an INPUT_REQUIREMENTS and generate a lock file - see https://github.com/locusrobotics/catkin_virtualenv/blob/master/README.md#locking-dependencies")
+
   endif()
 
   add_custom_command(COMMENT "Install requirements to ${CMAKE_BINARY_DIR}/${venv_dir}"