Check for libpulse-simple.pc#138
Check for libpulse-simple.pc#138mgorse wants to merge 1 commit intoMycroftAI:developmentfrom mgorse:development
Conversation
Needed to pull in libpulse-simple, which contains the pa_simple_* functions that the pulseaudio driver uses.
Codecov Report
@@ Coverage Diff @@
## development #138 +/- ##
==============================================
Coverage ? 40.71%
==============================================
Files ? 78
Lines ? 9031
Branches ? 1558
==============================================
Hits ? 3677
Misses ? 5354
Partials ? 0Continue to review full report at Codecov.
|
|
Sorry; just realized that I pushed this with an invalid email address. Should I submit a new pull request with a correct address? |
|
Thanks @mgorse I can reproduce the error. We are in the process of splitting mimic in several components. This fix would need to be applied to the mimic-core component at these two lines:
I will be happy to merge a pull request to that repository if you want to create it (preferably with a valid email address). Otherwise, just tell me here and I will do the commit myself. Thanks for the report |
I needed this change in order to get mimic to build. (I have version 11.0 installed, but pulseaudio-simple.pc has existed since at least version 0.9.12, so this change looks safe to me.)