GATB / bcalm

compacted de Bruijn graph construction in low memory
MIT License
99 stars 20 forks source link

Compilation error #22

Closed nithvarma closed 6 years ago

nithvarma commented 7 years ago

Hi,

I am trying to compile bcalm on my Surface Pro. I am using cygwin64 and gcc 5.4. I got a bunch of warnings while running cmake and running the Make command resulted in errors. All the errors were about some functions (stderror_r(..) and fseeko(..)) not being declared in scope. I am also attaching the outputs of cmake and make. Could you please help me out?

Thanks in advance, Nithin

output-of-cmake.txt output-of-make.txt

rchikhi commented 7 years ago

hi Nithin, I tried to fix these issues for cygwin. I'm surprised there aren't more, noone tests our gatb-core module with cygwin.. to update your bcalm version:

cd gatb-core && git pull
cd .. && cd build && make -j 8
nithvarma commented 7 years ago

Hi Rayan,

Thanks a lot for the quick reply. I shall update it and let you know if there are still problems.

Thanks, Nithin

nithvarma commented 7 years ago

I am still getting similar errors. I think I will try installing it on my Mac. Thanks again :)

rchikhi commented 7 years ago

oh, really? i'd be curious to see the new make log, but otherwise, okay

nithvarma commented 7 years ago

Sorry for the delayed reply, Rayan. Attaching the error files. output-of-cmake.txt output-of-make.txt

rchikhi commented 7 years ago

Thanks. It looks like you were still using the old gatb-core submodule version. For instance you see, at the first error in Exception.hpp, line 140 doesn't correspond to the error indicated by the make output anymore (https://github.com/GATB/gatb-core/blob/master/gatb-core/src/gatb/system/api/Exception.hpp#L140).

I've updated bcalm repository, can you try doing a 'git pull' again? It is low priority by the way.

nithvarma commented 7 years ago

Thank you so much Rayan, for helping me out. I had done a git pull after you had made the changes. I am not sure why it still did not get updated.

When I did a `git status' command in the folder /bcalm/gatb-core, it displayed a detached HEAD message with HEAD detached at 2aa4891e.

I later created a new folder /temp/ and did another clone of the repository in that folder. I got the detached HEAD message again, when I did a git status' inside /temp/bcalm/gatb-core. This time, though the message wasHEAD detached 0d8a9203', which I think corresponds to a commit on May 24, 2017. Trying to compile, again gave me error messages. But, this was not the latest commit, right? I am attaching the error log. output-of-make.txt

rchikhi commented 7 years ago

hi Nithin, thanks for the details. git pull does not automatically update the submodules, that would explain it. I misunderstood your latest answer, and now I realize that the problem isn't resolved, right? A possibility is to update gatb-core manually, i.e. go into the gatb-core folder and do a git pull origin master inside it`. This will force to get the very latest version.

nithvarma commented 7 years ago

Thanks for the answer on how to update gatb-core. I started using bcalm on my MAC afterwards and things have been running smoothly.

However, I shall try compiling it again on Cygwin after updating gatb-core the way you suggested.

rchikhi commented 6 years ago

Hi Nithin, should I keep this issue open?

nithvarma commented 6 years ago

Hi Rayan, I shall close it. Sorry for keeping it open for this long. I am no longer trying to use BCALM on Cygwin.

rchikhi commented 6 years ago

ok!