]> git.kaiwu.me - njs.git/commitdiff
CI: removed DUMP_LEAKS flag from QuickJS build.
authorDmitry Volyntsev <xeioex@nginx.com>
Thu, 12 Sep 2024 04:53:38 +0000 (21:53 -0700)
committerDmitry Volyntsev <xeioexception@gmail.com>
Wed, 18 Sep 2024 01:05:57 +0000 (18:05 -0700)
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

index 2f1511976494840731bb8a1ca573f3ffa108861d..e26b1e29ec4a4fdb9b56a05b746ff7dfac70b4f0 100644 (file)
@@ -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