wabt
wabt copied to clipboard
Inconsistent archive name for macos
I noticed that the macos archive name in the release is inconsistent vs ubuntu/windows. It has the version name, -12 while the others don't. This is because the name is computed by stripping out the GHA runner name of -latest, but for macos the runner is not -latest
https://github.com/WebAssembly/wabt/blob/main/.github/actions/release-archive/action.yml#L16 https://github.com/WebAssembly/wabt/blob/main/.github/workflows/build.yml#L33
Presumably these are all supposed to be symmetric. One option to fix would be to switch to using lowercase of runner.os instead of matrix.os, that word contains Linux, MacOS, Windows. Note that it is linux instead of ubuntu, this would be a relatively significant change to the current name but perhaps it's a more appropriate one?
I guess we transitioned from -latest to -12 for macos but didn't realise that would break the release builder. Simplest fix for now would likely be to just strip the -12 suffix in the same place we strip the -latest suffix.
is this (still) a problem?
It looks like things changed a little since filing, now I see a suffix on macos and ubuntu, and none on Windows. Still not consistent, maybe it doesn't matter too much. But I think with macos-14 added, the confusion has increased since many people won't know it's a difference between x86 and arm.
Following common naming like darwin-amd64, etc could make the release archives easier to use.