ndk icon indicating copy to clipboard operation
ndk copied to clipboard

M1 yasm

Open DanAlbert opened this issue 4 years ago • 7 comments

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

DanAlbert avatar Jul 26 '21 20:07 DanAlbert

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

rprichard avatar Mar 15 '22 00:03 rprichard

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

DanAlbert avatar Apr 21 '22 21:04 DanAlbert

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

rbebb avatar Apr 05 '23 14:04 rbebb

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.

DanAlbert avatar Apr 05 '23 19:04 DanAlbert

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?

leleliu008 avatar May 14 '24 07:05 leleliu008

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.

DanAlbert avatar May 14 '24 16:05 DanAlbert

Kicking this to the backlog. We've heard no complaints about it in the years since the rest of the NDK got M1 support, so I suspect no one actually uses this. If that's wrong, please speak up. The "fix" here is not trivial because it likely means replacing yasm with nasm. As long as we think no one is using it, that work's just not going to make it to the top of the triage list.

We don't have any plans to actually remove it though. Status quo has cost nothing in maintenance for the past few years, so I'd rather avoid breaking anyone happily using it without native M1 support :)

DanAlbert avatar Aug 01 '24 20:08 DanAlbert