ada-url / ada

WHATWG-compliant and fast URL parser written in modern C++, part of Node.js, Redpanda, Kong, Telegram and Cloudflare Workers.
https://ada-url.com
Apache License 2.0
1.34k stars 84 forks source link

Add workflow file to run arm-based testing with sanitizers #649

Closed CarlosEduR closed 4 months ago

CarlosEduR commented 5 months ago

Fixes: #520

lemire commented 5 months ago

Running tests.

CarlosEduR commented 5 months ago

Seems it failed... I just pushed an update, let's see how it goes.

lemire commented 5 months ago

Running...

CarlosEduR commented 4 months ago

@lemire hmm, it is still failing... Off the top of your head, do you have any idea? image

lemire commented 4 months ago

It looks like a permission problem. I am not sure what the source of the issue is.

lemire commented 4 months ago

I have never seen this error:

CMake Error: Cannot open file for write: /home/runner/work/ada/ada/build/Testing/Temporary/LastTest.log.tmp
CMake Error: : System Error: Permission denied
lemire commented 4 months ago

Let us run the tests...

lemire commented 4 months ago

@anonrig I suspect that the memory sanitizer is still busted. :-(

It finds problems inside the standard library.

(It is clearly unrelated to this PR.)

CarlosEduR commented 4 months ago

hmm, I see the pipeline passed, but what I saw from logs: image

lemire commented 4 months ago

@CarlosEduR Yes. But at least we are now checking that things build, which is good.