Skip to content
This repository has been archived by the owner on Apr 21, 2023. It is now read-only.

Compilation fails with Nginx 1.23 #1757

Closed
Kontr3X opened this issue Aug 10, 2022 · 1 comment
Closed

Compilation fails with Nginx 1.23 #1757

Kontr3X opened this issue Aug 10, 2022 · 1 comment

Comments

@Kontr3X
Copy link

Kontr3X commented Aug 10, 2022

./incubator-pagespeed-ngx-1.13.35.2-stable/psol/include/third_party/aprutil/gen/arch/linux/x64/include -I /root/sources/./incubator-pagespeed-ngx-1.13.35.2-stable/psol/include/url -I src/mail \
        -o objs/addon/src/ngx_pagespeed.o \
        /root/sources/./incubator-pagespeed-ngx-1.13.35.2-stable/src/ngx_pagespeed.cc
/root/sources/./incubator-pagespeed-ngx-1.13.35.2-stable/src/ngx_pagespeed.cc: In function ‘ngx_int_t net_instaweb::{anonymous}::ps_set_cache_control(ngx_http_request_t*, char*)’:
/root/sources/./incubator-pagespeed-ngx-1.13.35.2-stable/src/ngx_pagespeed.cc:407:36: error: request for member ‘elts’ in ‘r->ngx_http_request_s::headers_out.ngx_http_headers_out_t::cache_control’, which is of pointer type ‘ngx_table_elt_t*’ {aka ‘ngx_table_elt_s*’} (maybe you meant to use ‘->’ ?)
  407 |   if (r->headers_out.cache_control.elts == NULL) {
      |                                    ^~~~
/root/sources/./incubator-pagespeed-ngx-1.13.35.2-stable/src/ngx_pagespeed.cc:408:35: error: cannot convert ‘ngx_table_elt_t**’ {aka ‘ngx_table_elt_s**’} to ‘ngx_array_t*’
  408 |     ngx_int_t rc = ngx_array_init(&r->headers_out.cache_control, r->pool,
      |                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |                                   |
      |                                   ngx_table_elt_t** {aka ngx_table_elt_s**}
In file included from src/core/ngx_core.h:65,
                 from src/http/ngx_http.h:13,
                 from /root/sources/./incubator-pagespeed-ngx-1.13.35.2-stable/src/ngx_pagespeed.h:33,
                 from /root/sources/./incubator-pagespeed-ngx-1.13.35.2-stable/src/ngx_pagespeed.cc:26:
src/core/ngx_array.h:32:29: note:   initializing argument 1 of ‘ngx_int_t ngx_array_init(ngx_array_t*, ngx_pool_t*, ngx_uint_t, size_t)’
   32 | ngx_array_init(ngx_array_t *array, ngx_pool_t *pool, ngx_uint_t n, size_t size)
      |                ~~~~~~~~~~~~~^~~~~
/root/sources/./incubator-pagespeed-ngx-1.13.35.2-stable/src/ngx_pagespeed.cc:415:22: error: cannot convert ‘ngx_table_elt_t**’ {aka ‘ngx_table_elt_s**’} to ‘ngx_array_t*’
  415 |       ngx_array_push(&r->headers_out.cache_control));
      |                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      |                      |
      |                      ngx_table_elt_t** {aka ngx_table_elt_s**}
In file included from src/core/ngx_core.h:65,
                 from src/http/ngx_http.h:13,
                 from /root/sources/./incubator-pagespeed-ngx-1.13.35.2-stable/src/ngx_pagespeed.h:33,
                 from /root/sources/./incubator-pagespeed-ngx-1.13.35.2-stable/src/ngx_pagespeed.cc:26:
src/core/ngx_array.h:27:35: note:   initializing argument 1 of ‘void* ngx_array_push(ngx_array_t*)’
   27 | void *ngx_array_push(ngx_array_t *a);
      |                      ~~~~~~~~~~~~~^
/root/sources/./incubator-pagespeed-ngx-1.13.35.2-stable/src/ngx_pagespeed.cc: In function ‘bool net_instaweb::{anonymous}::ps_get_cache_control(ngx_http_request_t*, GoogleString*)’:
/root/sources/./incubator-pagespeed-ngx-1.13.35.2-stable/src/ngx_pagespeed.cc:439:74: error: request for member ‘elts’ in ‘r->ngx_http_request_s::headers_out.ngx_http_headers_out_t::cache_control’, which is of pointer type ‘ngx_table_elt_t*’ {aka ‘ngx_table_elt_s*’} (maybe you meant to use ‘->’ ?)
  439 |   auto ccp = static_cast<ngx_table_elt_t**>(r->headers_out.cache_control.elts);
      |                                                                          ^~~~
/root/sources/./incubator-pagespeed-ngx-1.13.35.2-stable/src/ngx_pagespeed.cc:444:59: error: request for member ‘nelts’ in ‘r->ngx_http_request_s::headers_out.ngx_http_headers_out_t::cache_control’, which is of pointer type ‘ngx_table_elt_t*’ {aka ‘ngx_table_elt_s*’} (maybe you meant to use ‘->’ ?)
  444 |   for (ngx_uint_t i = 0; i < r->headers_out.cache_control.nelts; i++) {
      |                                                           ^~~~~
make[1]: *** [objs/Makefile:1722: objs/addon/src/ngx_pagespeed.o] Error 1
make[1]: Leaving directory '/root/sources/nginx-1.23.1'
make: *** [Makefile:10: build] Error 2

NGINX Version: 1.23.1
Debian GNU/Linux 11 (bullseye)

@oschaaf
Copy link
Member

oschaaf commented Aug 10, 2022

Closing as a duplicate of #1749

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants