Skip to content
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

Issue installing gcloud using command "npm install --save gcloud" #817

Closed
ghost opened this issue Aug 22, 2015 · 14 comments
Closed

Issue installing gcloud using command "npm install --save gcloud" #817

ghost opened this issue Aug 22, 2015 · 14 comments

Comments

@ghost
Copy link

ghost commented Aug 22, 2015

HI,

I have an issue installing gcloud package on my machine . I am getting following error. So could you please help me out with this.

F:\Projects\Kiran\Bald-Eagle\mcapi>npm install --save gcloud
npm WARN package.json [email protected] No README data

npm WARN package.json [email protected] No license field.

[email protected] install F:\Projects\Kiran\Bald-Eagle\mcapi\node_modules\gclou
d\node_modules\sse4_crc32
node-gyp rebuild

F:\Projects\Kiran\Bald-Eagle\mcapi\node_modules\gcloud\node_modules\sse4_crc32>i
f not defined npm_config_node_gyp (node "C:\Program Files\nodejs\node_modules\np
else (node rebuild ) ..\node_modules\node-gyp\bin\node-gyp.js" rebuild )
configure error
gyp ERR! stack Error: Can't find Python executable "python", you can set the PYT
HON env variable.
gyp ERR! stack at failNoPython (C:\Program Files\nodejs\node_modules\npm\nod
e_modules\node-gyp\lib\configure.js:114:14)
gyp ERR! stack at C:\Program Files\nodejs\node_modules\npm\node_modules\node
-gyp\lib\configure.js:69:11
gyp ERR! stack at FSReqWrap.oncomplete (evalmachine.:95:15)
gyp ERR! System Windows_NT 6.3.9600
gyp ERR! command "node" "C:\Program Files\nodejs\node_modules\npm\node_modu
les\node-gyp\bin\node-gyp.js" "rebuild"
gyp ERR! cwd F:\Projects\Kiran\Bald-Eagle\mcapi\node_modules\gcloud\node_modules
\sse4_crc32
gyp ERR! node -v v0.12.7
gyp ERR! node-gyp -v v2.0.1
gyp ERR! not ok
npm ERR! Windows_NT 6.3.9600
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs
\node_modules\npm\bin\npm-cli.js" "install" "--save" "gcloud"
npm ERR! node v0.12.7
npm ERR! npm v2.11.3
npm ERR! code ELIFECYCLE

npm ERR! [email protected] install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the [email protected] install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the sse4_crc32 package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR! node-gyp rebuild
npm ERR! You can get their info via:
npm ERR! npm owner ls sse4_crc32
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR! F:\Projects\Kiran\Bald-Eagle\mcapi\npm-debug.log

@stephenplusplus
Copy link
Contributor

This is an issue with a dependency we use. Can you report this over on their repo? https://github.com/voxer/sse4_crc32

Thanks, and sorry you ran into this.

@ghost
Copy link
Author

ghost commented Aug 22, 2015

Hi @stephenplusplus it seems that sse4_ccr32 has released a new build to fix the issue. So do you think you need to update the version you are referring in the gcloud

Thanks,
Pavan

@stephenplusplus
Copy link
Contributor

I don't think 4.0 had anything to do with this. The problem here is: "Can't find Python executable "python", you can set the PYTHON env variable." If https://github.com/Voxer/sse4_crc32/issues/39 turns up something else, and we need to update, we'll definitely do that.

@ghost
Copy link
Author

ghost commented Aug 22, 2015

Python thing is that i regularly get and i am ignoring it since past. But yesterday i was trying to install gcloud and since then i am getting this error related to sse4_crc32. So i don't think python is culprit here.

@ghost
Copy link
Author

ghost commented Aug 22, 2015

Previously it seems that there was an issue from Voxer end and they gave a fix for it

https://github.com/Voxer/sse4_crc32/issues/29

@stephenplusplus
Copy link
Contributor

No problem, we'll just wait to see what they say in https://github.com/Voxer/sse4_crc32/issues/39.

@jgeewax
Copy link
Contributor

jgeewax commented Aug 22, 2015

@stephenplusplus This is far from the first time this library has caused a problem with installing gcloud. Are there really no alternatives that are more mature for doing CRC checks in Node? Or nothing else we can do to make this less of an issue ?

@stephenplusplus
Copy link
Contributor

The thing that's unfortunate about this particular library is that it has native bindings-- that's only unfortunate because all of a sudden, developers get errors they have never seen before and it doesn't make sense why. All they did was install a JavaScript library. These errors will come up any time we use a library with native bindings.

What we can do is look for a software-only equivalent. I've tried in the past without success, but it's probably worth it to try again.

@jgeewax
Copy link
Contributor

jgeewax commented Aug 22, 2015

Doesn't CRC32 (regular) give shitty performance but no installation problems?

And then we can document the whole "If you want things to be super fast, just run npm install sse4_crc32 and we'll use that..." and then things will be faster.... ?

@stephenplusplus
Copy link
Contributor

Working with the storage API is my first experience with crc32/c, but aren't crc32 and crc32c different algorithms that will give different results? The API gives us crc32c -- if crc32 works the same, then we can use something like this and be done with it. I doubt this type of performance tradeoff is anything the Node users will feel.

@jgeewax
Copy link
Contributor

jgeewax commented Aug 22, 2015

We'd need someone more knowledgeable to chime in, however I can say that certain people will definitely notice the difference (the Firebase folks for example definitely will be mad if we regress on performance going from C to JS), so we certainly need the ability to have "the faster way".

@stephenplusplus
Copy link
Contributor

Having both would be pretty tricky since the problem happens at the point
of install. Maybe optionalDependencies can help us, I'll have to play with
it a bit to see how that would behave.

Worst case with only supporting the software for those who want hardware
would be that they would have to run the validation themselves. I (or
we) could make a small wrapper in a different module that makes it as easy
as possible.

On Saturday, August 22, 2015, JJ Geewax [email protected] wrote:

We'd need someone more knowledgeable to chime in, however I can say that
certain people will definitely notice the difference (the Firebase
folks for example definitely will be mad if we regress on performance going
from C to JS), so we certainly need the ability to have "the faster way".


Reply to this email directly or view it on GitHub
#817 (comment)
.

@jgeewax
Copy link
Contributor

jgeewax commented Aug 22, 2015

Wait... we can't set crc32 (software) as the dependency, and then in code:

try {
  crc = require('sse4_crc32');
} catch {
  crc = require('crc32');
}

?

And then anyone who wants "faster crc checks" can run npm install sse4_crc32 ?

@stephenplusplus
Copy link
Contributor

Maybe! It's not a pattern I'm familiar with, I'll ask around if there's a
precedent here. If that is our best option, I'd try to modularize that out
and make a seamless API, otherwise our code would be quite gross.

On Saturday, August 22, 2015, JJ Geewax [email protected] wrote:

Wait... we can't set crc32 (software) as the dependency, and then in code:

try {
crc = require('sse4_crc32');
} catch {
crc = require('crc32');
}

?

And then anyone who wants "faster crc checks" can run npm install
sse4_crc32 ?


Reply to this email directly or view it on GitHub
#817 (comment)
.

sofisl pushed a commit that referenced this issue Nov 10, 2022
🤖 I have created a release *beep* *boop*
---


## [7.0.3](googleapis/nodejs-translate@v7.0.2...v7.0.3) (2022-09-22)


### Bug Fixes

* Allow passing gax instance to client constructor ([#814](googleapis/nodejs-translate#814)) ([ddc93f9](googleapis/nodejs-translate@ddc93f9))
* **deps:** Update dependency @google-cloud/vision to v3 ([#816](googleapis/nodejs-translate#816)) ([331c65b](googleapis/nodejs-translate@331c65b))
* Preserve default values in x-goog-request-params header ([#820](googleapis/nodejs-translate#820)) ([913ac11](googleapis/nodejs-translate@913ac11))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
sofisl pushed a commit that referenced this issue Jan 24, 2023
[![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [@types/node](https://togithub.com/DefinitelyTyped/DefinitelyTyped) | [`^14.0.0` -> `^16.0.0`](https://renovatebot.com/diffs/npm/@types%2fnode/14.17.32/16.11.6) | [![age](https://badges.renovateapi.com/packages/npm/@types%2fnode/16.11.6/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/npm/@types%2fnode/16.11.6/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/npm/@types%2fnode/16.11.6/compatibility-slim/14.17.32)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/npm/@types%2fnode/16.11.6/confidence-slim/14.17.32)](https://docs.renovatebot.com/merge-confidence/) |

---

### Configuration

📅 **Schedule**: "after 9am and before 3pm" (UTC).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/nodejs-speech).
sofisl pushed a commit that referenced this issue Jan 25, 2023
[![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [@types/node](https://togithub.com/DefinitelyTyped/DefinitelyTyped) | [`^14.0.0` -> `^16.0.0`](https://renovatebot.com/diffs/npm/@types%2fnode/14.17.32/16.11.6) | [![age](https://badges.renovateapi.com/packages/npm/@types%2fnode/16.11.6/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/npm/@types%2fnode/16.11.6/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/npm/@types%2fnode/16.11.6/compatibility-slim/14.17.32)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/npm/@types%2fnode/16.11.6/confidence-slim/14.17.32)](https://docs.renovatebot.com/merge-confidence/) |

---

### Configuration

📅 **Schedule**: "after 9am and before 3pm" (UTC).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/nodejs-speech).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants