jasperlinthorst / reveal

Graph based multi genome aligner
MIT License
46 stars 3 forks source link

Build fails on MacOS #20

Closed twrightsman closed 6 years ago

twrightsman commented 6 years ago

I'm attempting to update the reveal version in bioconda (https://github.com/bioconda/bioconda-recipes/pull/8406) and one of the automated tests (the MacOS build with llvm) has failed while trying to build reveal.

gcc -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -I/Users/distiller/project/miniconda/conda-bld/reveal_1522068955734/_b_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_pl/include -DNumInsertStates=2 -DVERSION=1.12 -UNDEBUG -I/Users/distiller/project/miniconda/conda-bld/reveal_1522068955734/_b_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_pl/include/python2.7 -c probcons/Probcons.cc -o build/temp.macosx-10.9-x86_64-2.7/probcons/Probcons.o
probcons/Probcons.cc:1611:30: error: address of overloaded function 'align' does not match required type '_object *(_object *, _object *)'
    { "align", (PyCFunction) align, METH_VARARGS|METH_KEYWORDS },
                             ^~~~~
probcons/Probcons.cc:1503:18: note: candidate function has different number of parameters (expected 2 but has 3)
static PyObject *align(Probcons *self, PyObject *args, PyObject *keywds)
                 ^
/Applications/Xcode-8.3.3.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/memory:5603:24: note: candidate function has different number of parameters (expected 2 but has 4)
_LIBCPP_FUNC_VIS void* align(size_t __align, size_t __sz, void*& __ptr, size_t& __space);
                       ^
1 error generated.
error: command 'gcc' failed with exit status 1
jasperlinthorst commented 6 years ago

This should be fixed now. Thanks for pointing it out. Let me know if for some reason it doesn't work...

Cheers, Jasper

twrightsman commented 6 years ago

Build successful; thanks Jasper!