openfl / lime

A foundational Haxe framework for cross-platform development
https://lime.openfl.org/
MIT License
749 stars 362 forks source link

Remove armv7 from default iOS target architectures #1734

Closed tobil4sk closed 6 months ago

tobil4sk commented 7 months ago

Fixes #1733. Nevermind.

Avoids the error no such file or directory: 'arm64' due to multiple architectures being passed in.

player-03 commented 7 months ago

I mean, we could remove it from the defaults for other reasons, but that's unrelated to the rebuild error.

tobil4sk commented 7 months ago

The original error was caused by a bug in hxcpp, but maybe it makes sense to remove armv7 from the defaults anyway: https://community.openfl.org/t/ios-armv7-architecture-is-deprecated/13698

player-03 commented 7 months ago

That thread seems to be talking about the build step, not rebuild, and I think they already removed it by default for that step. So rebuilding for armv7 isn't breaking anything, it just performs a bunch of unnecessary work.

If you undelete the branch, I can reopen the pull request.

tobil4sk commented 7 months ago

The branch is restored.