nifty icon indicating copy to clipboard operation
nifty copied to clipboard

Windows continous integration

Open DerThorsten opened this issue 8 years ago • 7 comments

setup continuous integration on windows with appveyor https://www.appveyor.com/

This build should use conda to fetch dependencies.

DerThorsten avatar May 21 '17 22:05 DerThorsten

@bluescarni If you could help here to set this up this would be great =)

DerThorsten avatar May 31 '17 15:05 DerThorsten

PR is up at #83. I guess you'll have to enable the project in the appveyor config page, if you haven't already.

bluescarni avatar Jun 07 '17 11:06 bluescarni

I enabled appveyor on my fork to test it, build is passing:

https://ci.appveyor.com/project/bluescarni/nifty/build/1.0.2

Python is not enabled yet as I am running into undefined symbol errors in conjunction with HDF5.

bluescarni avatar Jun 07 '17 11:06 bluescarni

Thank you very much.

Maybe some linking issues for HDF5? Greetings Thorsten

DerThorsten avatar Jun 08 '17 10:06 DerThorsten

@DerThorsten actually there's more issues when enabling the python bindings on MSVC. In addition to the HDF5 linking issue:

  1. for whatever reason, the usage of size_t instead of std::size_t trips up the compilation of the python bindings (but strangely enough, it's ok in the C++ tests - maybe it has something to do with the fact that pure C headers are being included when compiling the python bindings).
  2. there's a bunch of scary errors when enabling the LP_MP option in the bindings, including an internal compiler error apparently. I am not familiar with the code, and I cannot really tell if it's a genuine coding error or if it is just MSVC being a shitty compiler.

I have fixed all the occurrences of 1) I could identify, which involved also some modifications to marray/graph I believe. I'll open a PR shortly with a link to the appveyor log.

bluescarni avatar Jun 08 '17 11:06 bluescarni

PR is up at #85.

bluescarni avatar Jun 08 '17 12:06 bluescarni

Yay seems like it builds with python. I'll set up the integration of the python unit tests.

Thank you again.

DerThorsten avatar Jun 09 '17 17:06 DerThorsten