gpu.js icon indicating copy to clipboard operation
gpu.js copied to clipboard

Fix for Node v22.8.0

Open weagle08 opened this issue 1 year ago • 1 comments

What is wrong?

I'm attempting to update packages to the latest as install failes for newer versions of Node (v22.8.0). However when I update all dependencies and run yarn make I get the build failure in the image. I'm not familiar enough with the build process to know what the error means. If someone could point me in the right direction I can get a PR in after all tests pass.

Where does it happen?

The gl library needs to be updated to latest. I've attempted to do the full chain with gpu-mock.js and this so I can update brain.js, but fail with the attached error.

How do we replicate the issue?

Update all dependencies and dev dependencies to latest. Then run yarn make while on Node v22.8.0 on Linux. I have an updated fork here: gpu.js fork

How important is this (1-5)?

5, it completely breaks useage of this library on newer versions of Node.

Expected behavior (i.e. solution)

npm install, npm run build, npm run test all succeed.

image

weagle08 avatar Sep 07 '24 02:09 weagle08

Thanks weagle08 for pointing me in the right direction. I was able to update the dependencies as per your fork. The build works with Node v22.14.0. It does not work on 22 due to a bug in vinyl js which gulp depends on - https://github.com/gulpjs/vinyl-fs/issues/350. Node 23 doesn't work due to https://github.com/stackgl/headless-gl/issues/303. Also, the gulp build addon for streamx needed to be changed as per https://github.com/gulpjs/gulp/issues/2802. The PR is https://github.com/gpujs/gpu.js/pull/840. I am using gpu.js in browser and was able to verify that our use case seemed to work as expected with the built dist files. The tests passed fine but am not sure what other functionalities are affected by this update.

JenLyndle avatar Feb 13 '25 21:02 JenLyndle