Skip to content
This repository has been archived by the owner on Oct 28, 2021. It is now read-only.

CMake: Use basic compiler settings from Cable #5407

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

chfast
Copy link
Collaborator

@chfast chfast commented Dec 20, 2018

No description provided.

Copy link
Member

@gumb0 gumb0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah we've got warnings in Windows build now

@codecov-io
Copy link

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5407      +/-   ##
==========================================
+ Coverage   61.15%   61.16%   +<.01%     
==========================================
  Files         344      344              
  Lines       28321    28321              
  Branches     3238     3238              
==========================================
+ Hits        17321    17322       +1     
  Misses       9866     9866              
+ Partials     1134     1133       -1

@gumb0
Copy link
Member

gumb0 commented Jan 2, 2019

Windows build still fails

@chfast
Copy link
Collaborator Author

chfast commented Jan 2, 2019

Yes I know. Looks like I have to spin off Windows machine to solve this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants