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

Support cmake using CURL vendor cmake files #1488

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Chaz6
Copy link
Contributor

@Chaz6 Chaz6 commented Apr 30, 2020

With this change weechat will build using CMake when CURL is installed from the original source. Many distributions often include their own FindCURL.cmake, but the cmake files provided by the CURL project do not set CURL_INCLUDE_DIRS or CURL_LIBRARIES.

With this change weechat will build using CMake when CURL is installed from the original source. Many distributions often include their own FindCURL.cmake, but the cmake files provided by the CURL project do not set CURL_INCLUDE_DIRS or CURL_LIBRARIES.
@codecov-io
Copy link

Codecov Report

Merging #1488 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1488      +/-   ##
==========================================
- Coverage   35.28%   35.28%   -0.01%     
==========================================
  Files         203      203              
  Lines       83131    83131              
==========================================
- Hits        29335    29331       -4     
- Misses      53796    53800       +4     
Impacted Files Coverage Δ
src/gui/gui-line.c 45.57% <0.00%> (-0.46%) ⬇️

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 b3478fa...1a6f357. Read the comment docs.

@flashcode flashcode added the bug Unexpected problem or unintended behavior label May 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants