nodejs / snap

Node.js snap source and updater
MIT License
169 stars 38 forks source link

No `20/stable` snap for `ppc64le` #59

Closed simondeziel closed 11 months ago

simondeziel commented 11 months ago

We've been using 18/stable previously and it provided most of the arches we needed (thanks!) but when moving to 20/stable, we lost ppc64le support causing our build to fail:

:: + '[' ppc64le = riscv64 ']'
:: + snap install node --channel=20/stable --classic
:: error: snap "node" is not available on 20/stable but is available to install on
::        the following tracks:
::
::        19/stable  snap install --channel=19 node
::        18/stable  snap install --channel=18 node
::        16/stable  snap install --channel=16 node
::
::        Please be mindful that different tracks may include different features.
::        Get more information with 'snap info node'.

Looking at https://nodejs.org/download/release/index.tab, it looks like upstream is still building for ppc64le so it'd be nice to have the snap for it too, please :)

simondeziel commented 11 months ago

I don't have access but I think this is as easy as checking the box "PowerPC64 Little-Endian (ppc64el)" on https://launchpad.net/~openjs/+snap/node20/+edit

rvagg commented 11 months ago

no, it wasn't, but I've turned it on now and requested new builds 🤞 that it shows up, I'll keep an eye out for failed build notification and come back and update here if it does

simondeziel commented 11 months ago

@rvagg thanks for the quick turnaround! I don't know if I check in the right spot but I'm not seeing any ppc64el build on https://launchpad.net/~openjs/+snap/node20

rvagg commented 11 months ago

Sorry, I just noticed that myself while in there setting up a node21, maybe there was an eventual consistency problem after saving the snap and requesting a new build; I did it again just now and it appears to have produced it. Got one for the new node21 too if that's interesting for you and I'll also turn it on for edge.

simondeziel commented 11 months ago

Thanks Rod!