search

BoostGSoC21 / math

Boost.org math module
http://boost.org/libs/math
Boost Software License 1.0
0 stars 1 forks source link

Set up a simple project specific CI for FFT only #2

Closed ckormanyos closed 3 years ago

ckormanyos commented 3 years ago

Deactivate the normal Math CI in this fork and set up a simple project specific CI for FFT only.

ckormanyos commented 3 years ago

Hi @Lagrang3 and @cosurgi I have set up a realy minimalistic attempt at some CI running GCC/clang for the few files in the Jam in Eduardos preliminary work.

CI is a rather big failure right now as seen here.

Some of the errors look easy, such as not finding a particular benchmark header file. i might have made some simple mistakes, and probably did. But can you guys see any way to improve the preliminary CI? or see what if anything I forgot?

ckormanyos commented 3 years ago

Hi @Lagrang3 and @cosurgi also I should mention in order to get the CI draft you need to merge from the fft_chris branch. I did this into the fft barnch but only performed this merge for the fft branch.

ckormanyos commented 3 years ago

Hi @Lagrang3 and @cosurgi sorry for so many posts.

This preliminary simple CI has gone green. I needed to remember that clang does not have support for libquadmath. Also I needed to remember to compile with -std_gnu++11, 14, 17, 2a in order to activate the non-standard quadmath on GCC.

One problem remains for, maybe, later. I removed the benchmark test from bjam since the header could not be found. If this gets restored, then CI is good to go.

I found a few places needing to be patched for traditional C++11. These are now in CI, so we will detect future breakages there.

Good job guys on a strong start.

ckormanyos commented 3 years ago

Fixed by 94b34322a7b46f8b5e04804d6061728a321a7a7a

cosurgi commented 3 years ago

Thank you. I managed to bring back clang to CI, but had to open https://github.com/BoostGSoC21/math/issues/4

ckormanyos commented 3 years ago

CI in itself for FFT only is working.

Compiler/Domain-specific problems will be handled within their separate issues.