[njs] CI: removed DUMP_LEAKS flag from QuickJS build.

noreply at nginx.com noreply at nginx.com
Wed Sep 18 01:06:01 UTC 2024


details:   https://github.com/nginx/njs/commit/50e6a1141db346d1396ff7588a4f6e64327e5bdf
branches:  master
commit:    50e6a1141db346d1396ff7588a4f6e64327e5bdf
user:      Dmitry Volyntsev <xeioex at nginx.com>
date:      Wed, 11 Sep 2024 21:53:38 -0700
description:
CI: removed DUMP_LEAKS flag from QuickJS build.

While it is useful for debugging what objects are leaking at the end
this flag hides QuickJS JSString leaks from Address Sanitizer.

---
 .github/workflows/check-pr.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/check-pr.yml b/.github/workflows/check-pr.yml
index 2f151197..e26b1e29 100644
--- a/.github/workflows/check-pr.yml
+++ b/.github/workflows/check-pr.yml
@@ -39,7 +39,7 @@ jobs:
       - name: Check out and build quickjs
         run: |
           git clone https://github.com/bellard/quickjs
-          cd quickjs && curl -OL http://pp.nginx.com/pluknet/quickjs.patch && git apply quickjs.patch
+          cd quickjs
           CFLAGS=$CC_OPT LDFLAGS=$LD_OPT $MAKE_UTILITY -j$(nproc) libquickjs.a
 
       - name: Configure and make njs


More information about the nginx-devel mailing list