mirror of
https://github.com/jpochyla/psst.git
synced 2026-04-27 15:35:56 +03:00
[GH-ISSUE #51] Build fails installing on macOS 11.3.1 (Apple Silicon, M1) #41
Labels
No labels
api
bug
build
documentation
duplicate
enhancement
good first issue
help wanted
idea
invalid
linux
lowprio
macos
pull-request
upstream
windows
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/psst#41
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @lonkelle on GitHub (May 24, 2021).
Original GitHub issue: https://github.com/jpochyla/psst/issues/51
This is the error I get when
cargo build --releaseORcargo build --releaseon my M1 MacBook Air (macOS 11.3.1).@jpochyla commented on GitHub (May 27, 2021):
Do you have the submodules fetched? I don't have any M1 hw and have no way to test, sorry. PRs welcome!
@lonkelle commented on GitHub (Jun 14, 2021):
I think I can fix it, I just need to figure out how to enable
bindgenon your build. I needed to install it I believe so:brew install llvmcargo install bindgenworked for that but your build code still isn't using it.
Any suggestion on how to force it to use
bindgen?@jpochyla commented on GitHub (Jun 15, 2021):
Thanks @lonkle! Can you try adding
features = ["bindgen"]do theminiaudiodependency?@lonkelle commented on GitHub (Jun 15, 2021):
That worked perfectly, here's my PR for Apple Silicon support.
https://github.com/jpochyla/psst/pull/62 (you can choose to adding that "feature" conditional if you want, but
bindgenworks for all platforms it looks like).@jpochyla commented on GitHub (Jun 18, 2021):
Merged #62, thank you!
@jpochyla commented on GitHub (Jun 24, 2021):
@lonkle Hi, I think we were a bit too quick with merging this. Enabling the
bindgenfeature requiresllvminstallation, and I think we can do better. Do you think it would be possible to upstream the pre-generated arm64 bindings tominiaudio-rs, by running this script and opening a PR there?@lonkelle commented on GitHub (Jun 25, 2021):
@jpochyla Good point. Done! https://github.com/ExPixel/miniaudio-rs/pull/65
@jpochyla commented on GitHub (Jul 26, 2021):
This is taking a bit long, let's use our fork of
miniaudio-rs.@jpochyla commented on GitHub (Jul 26, 2021):
I've merged the PR into
jpochyla/miniaudio-rsand included it ina71179f. Please let me know in case of further problems on M1!@naymapl commented on GitHub (Jan 9, 2022):
Hello.
I start to build fresh build on Apple Silicon and got error: