monarch-initiative / monarch-legacy

Monarch web application and API
BSD 3-Clause "New" or "Revised" License
42 stars 37 forks source link

Add g++/c++ compiler requirement to documentation #1375

Open yuanzhou opened 8 years ago

yuanzhou commented 8 years ago

I'm using nvm to manage different versions of nodejs. So today I installed nodejs 4.6.0 via nvm, and then updated the npm to 3.10.8 since nodejs 4.6.0 comes with npm 2.15.9.

And I verified the versions:

[zhy19@fedora23 monarch-app]$ npm version
{ 'monarch-app': '0.0.1',
  npm: '3.10.8',
  ares: '1.10.1-DEV',
  http_parser: '2.7.0',
  icu: '56.1',
  modules: '46',
  node: '4.6.0',
  openssl: '1.0.2j',
  uv: '1.9.1',
  v8: '4.5.103.37',
  zlib: '1.2.8' }

Upon running ./install.sh, I got the following error:

screenshot from 2016-10-10 14-41-20

After searching the issues, I found that Chris encountered the similar issue last year: https://github.com/monarch-initiative/monarch-app/issues/977

Then after the package tree view, I saw another error: screenshot from 2016-10-10 14-45-16

I've also attached my npm-debug.log file as a txt file.

npm-debug.log.txt

I could still run npm run start to bring up the server but I'm not sure if any functionalities will be affected due to these errors.

kshefchek commented 8 years ago

Looks like a C compiler issue, could you try installing g++ (GNU C++ compiler)?

yuanzhou commented 8 years ago

Thanks a lot @kshefchek, I installed gcc-c++ on my fedora23 and the errors are gone.

I thought nodejs and npm are all I need, how come I'll need to install g++ separately?

kshefchek commented 8 years ago

It must be a requirement for one of our dependencies (looks like the sleep module). We should add it to our documentation.

kltm commented 8 years ago

@kshefchek This may be of interest to you: https://github.com/geneontology/noctua/issues/364 Essentially, once the gcc/++ dependencies get in there to a certain version, it starts to get harder (sometimes impossible) to maintain that across platforms. Be wary of library upgrades--security against balancing against whoever has the most recent libs.

kltm commented 8 years ago

Also, upstream of that: https://github.com/geneontology/noctua/pull/319#issuecomment-249338272

harryhoch commented 8 years ago

I'm seeing similar problems on my OS X machine, even with gcc/g++ installed

yuanzhou commented 8 years ago

I just had this issue again on my Mac OS X. I have gcc 4.2.1 installed already. I did a quick search and someone said node v4 updated v8 engine which requires newer version of gcc.

screen shot 2016-10-25 at 10 09 44 am

yuanzhou commented 8 years ago

@harryhoch how did you update the gcc on your mac? I don't seem to find a good solution on my mac. On fedora 23 I simply installed the latest gcc and the problem was solved.

harryhoch commented 8 years ago

@yuanzhou, try homebrew - http://brew.sh/ - and then "brew install gcc"