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

Add notes and troubleshooting for system deps #461

Merged
merged 1 commit into from
Aug 17, 2016
Merged

Conversation

waprin
Copy link
Contributor

@waprin waprin commented Aug 17, 2016

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Aug 17, 2016
@waprin
Copy link
Contributor Author

waprin commented Aug 17, 2016

cc @jonparrott

@waprin waprin changed the title Add notes and troubleshooting for system dips Add notes and troubleshooting for system deps Aug 17, 2016
Try adding the following to your ~/.pydistutils.cfg file,
substituting in your appropriate brew Cellar directory:

include_dirs=/Users/YourUsernamen/homebrew/Cellar/portaudio/19.20140130/include/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

note that most users install homebrew in /usr/local.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

really? I don't recall doing anything special and mine was in home.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Their homepage says /usr/local. (shruggie)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, done.

@theacodes
Copy link
Contributor

LGTM, but @jerjou should be the decider.

library_dirs=/usr/local/YourUsername/homebrew/Cellar/portaudio/19.20140130/lib/



Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Getting a bit newline-happy here, huh :-)


#### PortAudio on OS X

See speech/api/README.md.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd make speech/api/README.md a link for easier access.

@jerjou
Copy link
Contributor

jerjou commented Aug 17, 2016

LGTM after linking

@waprin
Copy link
Contributor Author

waprin commented Aug 17, 2016

@dpebot merge when travis passes

@dpebot
Copy link
Collaborator

dpebot commented Aug 17, 2016

Okay! I'll merge when all statuses are green.

@dpebot dpebot added the automerge Merge the pull request once unit tests and other checks pass. label Aug 17, 2016
@waprin
Copy link
Contributor Author

waprin commented Aug 17, 2016

thanks @dpebot

@dpebot dpebot merged commit 43b1489 into master Aug 17, 2016
@dpebot dpebot deleted the testing_docs branch August 17, 2016 21:02
arbrown pushed a commit that referenced this pull request Nov 14, 2022
* chore(deps): update all dependencies

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* revert

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
dandhlee pushed a commit that referenced this pull request Nov 15, 2022
* chore(deps): update all dependencies

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* revert

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
engelke pushed a commit that referenced this pull request Nov 28, 2022
* chore(deps): update all dependencies

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* revert

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
dandhlee pushed a commit that referenced this pull request Nov 29, 2022
* chore(deps): update all dependencies

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* revert

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
Sita04 pushed a commit that referenced this pull request Feb 7, 2023
…ve] (#461)

Source-Link: https://togithub.com/googleapis/synthtool/commit/6ed3a831cb9ff69ef8a504c353e098ec0192ad93
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:3abfa0f1886adaf0b83f07cb117b24a639ea1cb9cffe56d43280b977033563eb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge the pull request once unit tests and other checks pass. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants