Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove bad add_subdirectory #435

Closed
wants to merge 1 commit into from

Conversation

LocutusOfBorg
Copy link
Contributor

This isn't actually needed, because there is no CMakeLists.txt file in tests directory
"CMake Warning (dev) at CMakeLists.txt:229 (add_subdirectory):
The source directory
/tmp/build/websocketpp/test
does not contain a CMakeLists.txt file.
CMake does not support this case but it used to work accidentally and is
being allowed for compatibility.
Policy CMP0014 is not set: Input directories must have CMakeLists.txt. Run
"cmake --help-policy CMP0014" for policy details. Use the cmake_policy
command to set the policy and suppress this warning.
This warning is for project developers. Use -Wno-dev to suppress it."

This isn't actually needed, because there is no CMakeLists.txt file in tests directory
"CMake Warning (dev) at CMakeLists.txt:229 (add_subdirectory):
  The source directory
    /tmp/build/websocketpp/test
  does not contain a CMakeLists.txt file.
  CMake does not support this case but it used to work accidentally and is
  being allowed for compatibility.
  Policy CMP0014 is not set: Input directories must have CMakeLists.txt.  Run
  "cmake --help-policy CMP0014" for policy details.  Use the cmake_policy
  command to set the policy and suppress this warning.
This warning is for project developers.  Use -Wno-dev to suppress it."
@thoys
Copy link
Contributor

thoys commented Jul 16, 2015

#449 fixes this

@zaphoyd
Copy link
Owner

zaphoyd commented Jul 29, 2015

A more comprehensive CMake overhaul has been committed to the develop branch that should fix this issue. If it is not fixed, please reopen.

@zaphoyd zaphoyd closed this Jul 29, 2015
@LocutusOfBorg LocutusOfBorg deleted the patch-2 branch July 29, 2015 13:29
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants