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

cmake: set CMP0074 policy #751

Merged
merged 1 commit into from
Dec 6, 2021
Merged

cmake: set CMP0074 policy #751

merged 1 commit into from
Dec 6, 2021

Conversation

sergiud
Copy link
Collaborator

@sergiud sergiud commented Dec 6, 2021

This enables the use of <PackageName>_ROOT (environment) variables.

@sergiud sergiud added this to the 0.6 milestone Dec 6, 2021
@google-cla google-cla bot added the cla: yes label Dec 6, 2021
This enables the use of `<PackageName>_ROOT` (environment) variables.
@codecov-commenter
Copy link

codecov-commenter commented Dec 6, 2021

Codecov Report

Merging #751 (75cfad3) into master (f4dd77a) will increase coverage by 1.27%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #751      +/-   ##
==========================================
+ Coverage   71.38%   72.65%   +1.27%     
==========================================
  Files          17       17              
  Lines        3208     3211       +3     
==========================================
+ Hits         2290     2333      +43     
+ Misses        918      878      -40     
Impacted Files Coverage Δ
src/logging.cc 73.68% <0.00%> (+0.92%) ⬆️
src/symbolize.cc 62.82% <0.00%> (+1.72%) ⬆️
src/glog/logging.h.in 82.27% <0.00%> (+4.64%) ⬆️
src/utilities.cc 67.64% <0.00%> (+13.97%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f4dd77a...75cfad3. Read the comment docs.

@sergiud sergiud merged commit 6e9b2f1 into master Dec 6, 2021
@sergiud sergiud deleted the cmake-cmp0074 branch December 6, 2021 20:12
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants