Skip to content

Fix a CMake warning when GENERATE_DOC is off#2646

Closed
psychon wants to merge 1 commit intoawesomeWM:masterfrom
psychon:fix_cmake_warning
Closed

Fix a CMake warning when GENERATE_DOC is off#2646
psychon wants to merge 1 commit intoawesomeWM:masterfrom
psychon:fix_cmake_warning

Conversation

@psychon
Copy link
Copy Markdown
Member

@psychon psychon commented Feb 14, 2019

This fixes the following CMake warning that I saw in some NixOS build:

CMake Warning (dev) at CMakeLists.txt:457 (add_dependencies):
Policy CMP0046 is not set: Error on non-existent dependency in
add_dependencies. Run "cmake --help-policy CMP0046" for policy details.
Use the cmake_policy command to set the policy and suppress this warning.
The dependency target "check-examples" of target "check" does not exist.
This warning is for project developers. Use -Wno-dev to suppress it.

Signed-off-by: Uli Schlachter psychon@znc.in

This fixes the following CMake warning that I saw in some NixOS build:

CMake Warning (dev) at CMakeLists.txt:457 (add_dependencies):
  Policy CMP0046 is not set: Error on non-existent dependency in
  add_dependencies.  Run "cmake --help-policy CMP0046" for policy details.
  Use the cmake_policy command to set the policy and suppress this warning.
  The dependency target "check-examples" of target "check" does not exist.
This warning is for project developers.  Use -Wno-dev to suppress it.

Signed-off-by: Uli Schlachter <psychon@znc.in>
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 14, 2019

Codecov Report

Merging #2646 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2646      +/-   ##
==========================================
+ Coverage    85.3%   85.31%   +<.01%     
==========================================
  Files         519      519              
  Lines       35181    35181              
==========================================
+ Hits        30012    30015       +3     
+ Misses       5169     5166       -3
Flag Coverage Δ
#c_code 74.22% <ø> (+0.05%) ⬆️
#functionaltests 70.45% <ø> (ø) ⬆️
#lua53 85.31% <ø> (ø) ⬆️
#samples 72.73% <ø> (-0.02%) ⬇️
#themes 49.04% <ø> (ø) ⬆️
#unittests 58.38% <ø> (ø) ⬆️
Impacted Files Coverage Δ
lib/awful/widget/layoutlist.lua 90.54% <0%> (-0.68%) ⬇️
event.c 72.48% <0%> (+0.87%) ⬆️

blueyed
blueyed previously approved these changes Feb 14, 2019
@blueyed
Copy link
Copy Markdown
Member

blueyed commented Feb 14, 2019

Note that the include of tests/examples/CMakeLists.txt is unconditional with #2545 (and therefore this would not be necessary / get reverted with it).

@blueyed blueyed dismissed their stale review February 14, 2019 14:33

See previous comment, finishing #2545 might be better, I am +0 on this.

@blueyed
Copy link
Copy Markdown
Member

blueyed commented Feb 19, 2019

Since #2545 has been merged this should not be necessary anymore.

@blueyed blueyed closed this Feb 19, 2019
@psychon psychon deleted the fix_cmake_warning branch March 10, 2019 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants