Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix CMake 3.20 warning in curl http client #752

Merged
merged 1 commit into from
May 14, 2021

Conversation

maxgolov
Copy link
Contributor

@maxgolov maxgolov commented May 14, 2021

Cleaning up build warning. No code changes.

Fix CMake 3.20+ warning. If you are building with fresh CMake that comes in latest Visual Studio 2019 update, then it complains about filenames in CMakeLists.txt not having extensions. This is happening due to new policy CMP0115 described here - Source file extensions must be explicit.. This logically makes sense when target library name matches that one of a source file name, to clearly disambiguate between the two: what is target name, and what is source file name.

image

Fix CMake 3.20+ warning
@maxgolov maxgolov requested a review from a team May 14, 2021 18:15
@codecov
Copy link

codecov bot commented May 14, 2021

Codecov Report

Merging #752 (7567280) into main (b696e11) will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #752      +/-   ##
==========================================
+ Coverage   95.95%   95.97%   +0.01%     
==========================================
  Files         174      174              
  Lines        7127     7127              
==========================================
+ Hits         6839     6840       +1     
+ Misses        288      287       -1     
Impacted Files Coverage Δ
sdk/src/logs/batch_log_processor.cc 95.00% <0.00%> (+1.25%) ⬆️

@lalitb lalitb merged commit 3bfbc93 into main May 14, 2021
@lalitb lalitb deleted the maxgolov/fix_cmake320_warning branch May 14, 2021 18:42
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.

3 participants