android / ndk

The Android Native Development Kit
1.93k stars 253 forks source link

M1 yasm #1549

Open DanAlbert opened 2 years ago

DanAlbert commented 2 years ago

Subtask for https://github.com/android/ndk/issues/1299

rprichard commented 2 years ago

Rather than upgrade yasm with a universal macOS build, it may make more sense for the NDK to switch to shipping nasm instead.

DanAlbert commented 2 years ago

We're still blocked on this, and r23c is being prepped right now, so this won't make r23c. Retriaged.

rbebb commented 1 year ago

Hey @DanAlbert, is this something that's still in the works? I don't personally need it, but I figured I'd check since it's the last item for M1 support: https://github.com/android/ndk/issues/1299

DanAlbert commented 1 year ago

Depends on what you mean by "in the works". We are not actively working on all 143 of our open bugs because we don't have that kind of bandwidth. This is still on the todo list, but is low priority because we don't actually know if anyone needs this.

leleliu008 commented 1 month ago
Users/fpliu/.ndk-pkg/uppm/installed/android-ndk-r26d/toolchains/llvm/prebuilt/darwin-x86_64/bin/yasm: Bad CPU type in executable

Is yasm necessary to be shipped with Android NDK?

DanAlbert commented 1 month ago

For the people that use it, yes. We can't have metrics without making yasm phone home, which I'm not going to do, so we can't know how many people use it. It costs us nothing to keep around, so we don't plan to remove it.