Remap iOS simulator target names to fix the build for the simulator

[plaidfinch]

This adds additional logic to the build.rs script which handles the mismatch between the -sim and -simulator suffixes — Rust uses the former, and Clang (at least, Apple clang version 17.0.0 (clang-1700.0.13.3) Target: arm64-apple-darwin24.3.0) expects the latter.

This patch is motivated by a need to fix the build of cpal for the iOS simulator — at present, it fails (at least on my machine) due to this error:

error: failed to run custom build command for `coreaudio-sys v0.2.16`

Caused by:
  process didn't exit successfully: `/target/release/build/coreaudio-sys-a853612bd3562b3f/build-script-build` (exit status: 101)
  --- stdout
  cargo:rerun-if-env-changed=COREAUDIO_SDK_PATH
  cargo:rustc-link-lib=framework=AudioToolbox
  cargo:rustc-link-lib=framework=AudioToolbox
  cargo:rustc-link-lib=framework=CoreAudio
  cargo:rerun-if-env-changed=BINDGEN_EXTRA_CLANG_ARGS

  --- stderr

  thread 'main' panicked at /.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/coreaudio-sys-0.2.16/build.rs:162:39:
  unable to generate bindings: ClangDiagnostic("error: version 'sim' in target triple 'aarch64-apple-ios-sim' is invalid\n")

I can confirm that with a workspace patch pointed at this fork, cpal is able to build successfully for iOS simulator.

[plaidfinch]

It appears this may be duplicative of #112; I'm not sure if these overrides are necessary in all environments, but the fix above works in mine.

[simlay]

It appears this may be duplicative of #112; I'm not sure if these overrides are necessary in all environments, but the fix above works in mine.

I believe it is. I'm currently waiting on a new release of rust-bindgen to include rust-lang/rust-bindgen#3182.

[plaidfinch]

It appears this may be duplicative of #112; I'm not sure if these overrides are necessary in all environments, but the fix above works in mine.

I believe it is. I'm currently waiting on a new release of rust-bindgen to include rust-lang/rust-bindgen#3182.

Ah, great! Thanks for following up on this — should I raise this with cpal, so we can get that dependency bumped once you've been able to incorporate the new bindgen release?

[simlay]

Ah, great! Thanks for following up on this — should I raise this with cpal, so we can get that dependency bumped once you've been able to incorporate the new bindgen release?

Nah. This should be a patch release on coreaudio-sys and won't require updating cpal or coreaudio-rs.

[erikjara]

Is there something missing here to merge? I'm having the same issue with cpal.

[simlay]

Is there something missing here to merge? I'm having the same issue with cpal.

Sorry. Still waiting on a crate.io release for rust-bindgen. I've also been testing out a cpal PR that removes the dependency on rust-bindgen. The testing is documented here RustAudio/coreaudio-rs#132 for RustAudio/cpal#943. This is similarly gonna take longer than you'll like as it will require a cpal release (and probably a minor version bump).

Depending on how much you like to patch your project, you might be able to get away with:

[patch.crates-io]
cpal = { git = "https://github.com/madsmtm/cpal", branch = "coreaudio-update" }

I've noticed that cpal's HEAD removes an oboe feature flag depending on the project so you might end up with that error.

[simlay]

@plaidfinch This should now be resolved with #114 (and is now published to crates.io) so you should be able to cargo update and this should be fixed.

[plaidfinch]

Thank you!