To be tested on Windows machines

x64 build is still broken - shouldn't the $ARCH actually be x64 instead of x86_64 here? That's what process.arch is on x64 Windows.

https://github.com/Jai-JAP/tabby/actions/runs/2141322690
@Eugeny can you test these new build artifacts

GitHub

Update build.yml · Jai-JAP/tabby@31f549f

A terminal for a more modern age. Contribute to Jai-JAP/tabby development by creating an account on GitHub.

https://github.com/Jai-JAP/tabby/actions/runs/2141391555

Here are the latest builds

GitHub

Update build-native.js · Jai-JAP/tabby@579b602

A terminal for a more modern age. Contribute to Jai-JAP/tabby development by creating an account on GitHub.

31f549f works on Windows x64

579b602 broke it again, the new condition is equivalent to

process.env.ARCH = (process.env.ARCH || (process.arch === 'arm')) ? 'armv7l' : process.arch``` and always returns `armv7l`

Shouldn't it be just process.env.ARCH = (process.env.ARCH || process.arch) === 'arm' ? 'armv7l' : process.arch ?

Shouldn't it be just process.env.ARCH = (process.env.ARCH || process.arch) === 'arm' ? 'armv7l' : process.arch ?

I will commit this and again can you try the build as my windows system is broken

Also fixed the same in build-linux.js

Aftifacts
Fix build-native.js
Fix build-linux.js

@Eugeny Have you tested the Mac OS x86_64 builds as they may also have the same issue as mentioned in #6106 (comment)

Windows x64 build is tested working on my friend's PC
Windows arm64 has to be tested however

macOS x64 confirmed working

Ok then...
Ready to merge