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

fix build errors #62

Closed
wants to merge 9 commits into from
Closed

Conversation

danielk117
Copy link
Contributor

No description provided.

@danielk117 danielk117 closed this Jan 28, 2024
@danielk117
Copy link
Contributor Author

sorry, this belongs to my master master branch 😉

@danielk117 danielk117 deleted the danielk117-patch-1 branch January 28, 2024 13:23
@duhow
Copy link
Owner

duhow commented Jan 28, 2024

no problem, but actually I will add your commits as well and try it out 🙂
feel free to push for any improvements! 💪🏻

@danielk117
Copy link
Contributor Author

of course 🙂

BTW, I got an build error for blue-alsa package with your latest updates. thats why I removed the flags --enable-aptx --enable-aptx-hd for blue-alsa and build is successful now. https://github.com/duhow/xiaoai-patch/pull/62/files#diff-65ee44e09ce70af0debdfdfa34d0605a91e35ee4970e50dd67888d5aa1fa0c44

any idea how to fix this?

@duhow
Copy link
Owner

duhow commented Jan 28, 2024

blue-alsa uses latest from master, so very likely there's a recent change that may be breaking.
https://github.com/arkq/bluez-alsa
I'll have a look and post a patch if needed.

@duhow
Copy link
Owner

duhow commented Jan 28, 2024

Assuming this is the issue: arkq/bluez-alsa@c0378e5

@danielk117
Copy link
Contributor Author

ah nice, you already fixed it 👍

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

Successfully merging this pull request may close these issues.

2 participants