node icon indicating copy to clipboard operation
node copied to clipboard

Re-enable test-ubsan.yml

Open RafaelGSS opened this issue 10 months ago • 0 comments

During the latest Node.js major release (https://github.com/nodejs/node/pull/52505) we have had to disable test-ubsan.yml due to failing tests on a recent V8 update (https://github.com/nodejs/node/pull/52560).

I have tried to make it work, but due to the short time frame I had to fix it, we have decided to simply disable it for now.

Some insights:

  • It seems clang provides a better output and some docs we could use in the suppressions file.
    • Changing from g++ to clang for UBSAN will require changing all the suppression file

RafaelGSS avatar Apr 29 '24 18:04 RafaelGSS