search

m1dugh / native-sound-mixer

MIT License
25 stars 13 forks source link

32-bit compatibility #14

Closed Giotino closed 2 years ago

Giotino commented 2 years ago

Hi, I wasn't able to get this module (from npm and from the master branch) working on my machine (Node v16.2.0 x86 on Windows). I noted that

const { arch } = require('os');
console.log(arch());

is printing 'ia32' on my machine (instead of the 'x32' expected by this module).

An easy fix would be to replace this line

https://github.com/m1dugh/native-sound-mixer/blob/fede9f91872254c96747315d5d0518253c81fe7a/src/sound-mixer.ts#L8

with

if (arch() === "x32" || arch() === "x64" || arch() === "ia32") {

With this patch the module works fine.

m1dugh commented 2 years ago

This is fixed as of version 3.2.3, thanks for the issue and the resolution.