Welcome! Log In Create A New Profile

Advanced

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

Anonymous User
September 17, 2024 09:08PM
details: https://github.com/nginx/njs/commit/50e6a1141db346d1396ff7588a4f6e64327e5bdf
branches: master
commit: 50e6a1141db346d1396ff7588a4f6e64327e5bdf
user: Dmitry Volyntsev <xeioex@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
Subject Author Views Posted

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

Anonymous User 113 September 17, 2024 09:08PM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 123
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 500 on July 15, 2024
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready