-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Build Failure on Ubuntu 14.04 #32
Comments
I don't know if this helps since I don't have Ubuntu 14.04, but check this out: |
Thanks for your reply, and it looked promising. Unfortunately, there are other compatibility issues too. The next one we run into is: const Hoek = require('hoek'); Looking around on the web, it seems this might be out of joi and the maintainer says he doesn't support legacy anymore. Are there plans to support nodejs directly? |
Ok, got past the previous error by installing version 8.5 of nodejs. Now I have another issue. The build with gradlew completes successfully. I can run the gerrit_download.sh script. I get an error when running the gerrit_stats.sh script. I've been looking at this most of the day and believe this is turning out to be something with webpack. I'm hoping this looks familiar with someone. [email protected] webpack /home/rschmitt/git/gerritstats/GerritStats
ERROR in getOptions is not defined There are many more of those ERROR in getOptions in the output. This is crossposted to a webpack mailing list as well. Any ideas? |
@bluepeach is this still an issue? I see you have also filed webpack/loader-utils#99 . |
(this is @bluepeach) Well, it probably still is an issue. I worked on it a bit more and then just decided to move on to other things. I was hoping to use it though to get status from our gerrit projects. Figured I'd wait for some future release and try again. |
We could also rephrase the bug to "it doesn't build anymore on most platforms". Also tried RHEL, Fedora and MacOS, broken on all of them. |
@bluepeach @holmari I build with ubuntu 16.04 LTS and node v8.5, it keeps looking for v3.13.1/linux-x64-57_binding.node but this really not exist on github, do you got any idea on it?
|
OK, so I am not using Ubuntu, but here are some instructions on how I got this to build on Linux. A quick warning is that it ain't pretty. First off, a better list of dependencies:
What we are going to do is separate the build into 2 steps. The node build for GerritStats, and the Application build from root. To do that we will disable the Gradle npm dependency, and then just handle it ourselves. A. Node stuff In the Note that I tried several different approaches with inconsistent results, but suffice it to say, if you can get npm/yarn to finish, then you can move on. B Gradle run Edit the file /GerritStats/build.gradlew and comment out that line the says DONE Why does this fix it? Well the older npm version delcared in the package-lock.json, which might make sense for MacOS, actually causes issues on other platforms, as newer versions have fixed a number of threading/dependency issues. @holmari is you are interested, I will submit a PR with an updated package-lock.json |
@james-nesbitt, happy to accept any PRs. I'm sorry I don't have time to actively maintain this project, I'm not using Gerrit anymore at my current workplace. |
I'm trying to build gerritstats by following the readme on ubuntu 14.04.
It first complained that npm wasn't installed, so I apt-get'ed it.
It then complained that node wasn't installed, so I apt-get'ed it too
It then fails with:
npm http GET https://registry.npmjs.org/inherits
npm http 304 https://registry.npmjs.org/inherits
npm http 304 https://registry.npmjs.org/block-stream
npm WARN This failure might be due to the use of legacy binary "node"
npm WARN For further explanations, please read
/usr/share/doc/nodejs/README.Debian
npm ERR! weird error 1
npm ERR! not ok code 0
:GerritStats:npmInstall FAILED
FAILURE: Build failed with an exception.
Execution failed for task ':GerritStats:npmInstall'.
It might be due to legacy node, since it says it's replaced with nodejs. But nodejs is installed. So, I don't know. Does any of this sound familiar?
Rich
The text was updated successfully, but these errors were encountered: