-
Notifications
You must be signed in to change notification settings - Fork 9k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Failing when trying to run 'npm run build' #3027
Comments
Just my suggestion: I have had the exact same issue, and I grabbed v3.0.7 and ran the build on it, and it worked. Not sure if that really helps too much, but it may help narrow down where the problem lies |
@JimSterling1, it's not clear from your post what OS you're on, can you clarify? @Azotherian, can you share your OS, Node version, and NPM version? Thanks to both of you! |
@shockey I am using cent os 7 |
@shockey I am also using cent os, and my npm version is v3.10.10 and my node version is v6.10.1 |
Facing a similar issue on macOS Sierra using Node v7.5.0 and npm v4.1.2 and swagger-ui v3.0.9 I get a similar set of npm warnings when I run
|
@tylerjames, do you only get the warnings, or does your build fail? FYI, we don't explicitly support Node 7 yet. CentOS people: I'm going to spin up a VM today and try to reproduce the issues you're seeing. |
@shockey |
I pulled the zip right to my vm and unzipped and copied the contents over to my project. Maybe that had something to do with it? |
@Azotherian, I can't think of why that would be off the top of my head, but I'll test the zip and cloning when I get around to this today. If you can try cloning on your end as well, that would be helpful. Thanks for chiming in! |
All: Did you see this error message?
|
Yeah, that looks familiar
…On Thu, May 11, 2017 at 12:53 AM shockey ***@***.***> wrote:
All: Did you see this error message?
Error: Git returned with status 128: fatal: Not a git repository (or any of the parent directories): .git
at handleProcessResult (/root/swagger-ui-3.0.8/node_modules/git-describe/lib/git-describe.js:69:19)
at wrapper (/root/swagger-ui-3.0.8/node_modules/lodash/lodash.js:5259:19)
at gitDescribe (/root/swagger-ui-3.0.8/node_modules/git-describe/lib/git-describe.js:46:16)
at wrapper (/root/swagger-ui-3.0.8/node_modules/lodash/lodash.js:5259:19)
at Object.<anonymous> (/root/swagger-ui-3.0.8/make-webpack-config.js:12:17)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#3027 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABFfjfE3eBFtqDK_tuirg7aW0t3N4yhVks5r4pQxgaJpZM4NQ8jc>
.
|
I just merged something that appears to fix a problem I'm encountering when trying to Can y'all try to do an |
@shockey That seems to have fixed it |
Ok, haven't heard anything new about this here or in any other tickets - also can't reproduce anything on my end. I'm going to assume my PR fixed all the problems here. Please comment here or open a new ticket if any issues persist. Thanks! |
I am using v3.0.8 and I am running into issues when I am trying to build swagger-ui with my changes. I have included the npm-debug log, as well as some of the dependencies that npm had said were deprecated when installing npm.
Modules that NPM says are deprecated:
There was also output saying npm-gyp should be installed globally (not sure if that is something I should do since I am not running npm on windows
and some warning NPM gave me after the install:
I am putting this in the google group as well as here, and either one can be removed if it feels redundant. Thank you for any help you can provide!
The text was updated successfully, but these errors were encountered: