ddemidov / vexcl

VexCL is a C++ vector expression template library for OpenCL/CUDA/OpenMP

http://vexcl.readthedocs.org

MIT License

699 stars 81 forks source link

Fix appveyor #266

Closed Zialus closed 5 years ago

Zialus commented 5 years ago

I'm back with CI fixes.

This should finally make everything build fine on windows.

ddemidov commented 5 years ago

Thank you! I've left a couple of notes in the code.

codecov[bot] commented 5 years ago

Codecov Report

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

@@            Coverage Diff             @@
##           master     #266      +/-   ##
==========================================
+ Coverage   94.81%   94.82%   +<.01%     
==========================================
  Files         122      122              
  Lines       14862    14862              
==========================================
+ Hits        14092    14093       +1     
+ Misses        770      769       -1

Impacted Files	Coverage Δ
tests/scan_by_key.cpp	`96.82% <0%> (+1.58%)`	:arrow_up:

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 6f7641e...b0e07e5. Read the comment docs.

codecov[bot] commented 5 years ago

Codecov Report

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

@@            Coverage Diff             @@
##           master     #266      +/-   ##
==========================================
+ Coverage   94.81%   94.82%   +<.01%     
==========================================
  Files         122      122              
  Lines       14862    14862              
==========================================
+ Hits        14092    14093       +1     
+ Misses        770      769       -1

Impacted Files	Coverage Δ
tests/scan_by_key.cpp	`96.82% <0%> (+1.58%)`	:arrow_up:

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 6f7641e...b0e07e5. Read the comment docs.

Zialus commented 5 years ago

VS2013 just doesn't seem to work

https://ci.appveyor.com/project/Zialus/vexcl/builds/21238879/job/io39ts666vbrestm

I've added Ubuntu back and removed VS2013

ddemidov commented 5 years ago

Thanks again!