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 prerequisite packages #402

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

cartheur
Copy link

While building for RaspberryPi OS, a set of packages were recommended during the process. This change request adds the packages by name, relative to Debian Bookworm.

@dhdaines
Copy link
Contributor

Thanks! Very good idea to document this!

@dhdaines
Copy link
Contributor

(note though that ffmpeg is not, I hope, a prerequisite for pocketsphinx, just useful to have)

@cartheur
Copy link
Author

ffmpeg doesn't seem to be a requisite for pocketsphinx, Although a part of the base packages for RaspbianOS.

@dhdaines
Copy link
Contributor

I'll fire up the Raspberry Pi and add some more documentation around this on Friday!

Install prerequisite packages particular to RaspberryOS:

```
sudo apt install libpulse-dev libasound2-dev portaudio19-dev libportaudio2 libportaudiocpp0 ffmpeg
Copy link
Contributor

Choose a reason for hiding this comment

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

Please keep the list sorted to make it easy to spot missing dependencies and nearly impossible to add duplicates.

Copy link
Author

@cartheur cartheur Dec 21, 2024

Choose a reason for hiding this comment

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

I've changes to the PR based on your review (and the removal of ffmpeg) but have not the permissions for git push origin pr-402: "remote: Permission to cmusphinx/pocketsphinx.git denied to cartheur".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants