6
95%
main: 95%

Ran 13 Jul 2023 06:53PM UTC

Files 42

Run time 1s

Badge

Embed ▾

Committed 13 Jul 2023 06:24PM UTC coverage: 90.425%. First build

Job # 5546583804-1.6

Build Type

push

github

Committed by

web-flow

Commit Message

fix(proxy-wasm) expect the 'NGX_WASM_HOST_PROPERTY_NAMESPACE' macro unquoted

This is not really a bug on our end, but Nginx's build system
doesn't like `--with-cc-opt` entries that feature double-quoted variables
as in `"--with-cc-opt=\"-DNGX_WASM_HOST_PROPERTY_NAMESPACE=\\\"kong\\\"\""`.

This _does_ work when expanding the flag and building our module,
but the compilation of Nginx itself later fails with:

```
src/core/nginx.c:456:49: note: in expansion of macro 'NGX_CONFIGURE'
  456 |         ngx_write_stderr("configure arguments:" NGX_CONFIGURE NGX_LINEFEED);
      |                                                 ^~~~~~~~~~~~~
src/core/nginx.c:456:25: note: to match this '('
  456 |         ngx_write_stderr("configure arguments:" NGX_CONFIGURE NGX_LINEFEED);
      |                         ^
```

So let's just play nice and drop the quotes on our end.

From #349

Run Details

6639 of 7342 relevant lines covered (90.42%)

2212.46 hits per line

Kong / ngx_wasm_module / 5546583804-1 / 6
95%
main: 95%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job unit-ngx_1.21.6-wasmer-ssl-debug-no_hup-static - 5546583804-1.6

Kong / ngx_wasm_module / 5546583804-1 / 6 95% main: 95%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job unit-ngx_1.21.6-wasmer-ssl-debug-no_hup-static - 5546583804-1.6

Kong / ngx_wasm_module / 5546583804-1 / 6
95%
main: 95%

README BADGES
x