-
-
Notifications
You must be signed in to change notification settings - Fork 15.3k
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
google-cloud-speech: init at 0.30.0 #32890
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note the comments I made apply to all expressions.
}; | ||
|
||
propagatedBuildInputs = [ google_auth protobuf googleapis_common_protos requests grpcio ]; | ||
buildInputs = [ setuptools mock pytest ]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
checkInputs
|
||
meta = { | ||
description = "This library is not meant to stand-alone. Instead it defines common helpers used by all Google API clients."; | ||
license = "apache"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
which specifically? Use lib.licences
meta = { | ||
description = "This library is not meant to stand-alone. Instead it defines common helpers used by all Google API clients."; | ||
license = "apache"; | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maintainer?
meta = { | ||
description = "This library is not meant to stand-alone. Instead it defines common helpers used by all Google API clients."; | ||
license = "apache"; | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
homepage?
, google_auth, protobuf, googleapis_common_protos, requests, grpcio, setuptools, mock, pytest }: | ||
|
||
buildPythonPackage rec { | ||
name = "google-api-core-${version}"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pname = "google-api-core";
version = "0.1.2";
drop name
|
||
buildPythonPackage rec { | ||
name = "google-auth-${version}"; | ||
version = "1.2.1"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here
sha256 = "041qpwlvpawggasvbfpkx39mkg4dgvivj831x7kinidayrf46w3i"; | ||
}; | ||
|
||
buildInputs = [ pytest mock oauth2client flask requests urllib3 pytest-localserver ]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
checkInputs
|
||
meta = { | ||
description = "This library simplifies using Google’s various server-to-server authentication mechanisms to access Google APIs."; | ||
license = "apache"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use lib.licenses
Also, a commit per expression is needed. |
4c7de61
to
3a180b0
Compare
@FRidh anything I can do to get this PR accepted? I've implemented the changes you requested. I'd rather have it merged now that it still has no conflicts with master. |
Looks good, thanks! |
Motivation for this change
Google Cloud Speech API enables developers to convert audio to text;
The provided package is the Python client.
Things done
pkgs/development/python-modules/google_cloud_speech/default.nix is the actual goal of the present commit; the other packages are required dependencies.
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)