details: https://github.com/nginx/njs/commit/50e6a1141db346d1396ff7588a4f6e64327e5bdf branches: master commit: 50e6a1141db346d1396ff7588a4f6e64327e5bdf user: Dmitry Volyntsev <xei...@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 _______________________________________________ nginx-devel mailing list nginx-devel@nginx.org https://mailman.nginx.org/mailman/listinfo/nginx-devel