[postgis-tickets] [PostGIS] #4527: Travis: error: unrecognized command line option '-Wno-unknown-warning-option' [-Werror]

[PostGIS]

#4527: Travis: error: unrecognized command line option '-Wno-unknown-warning-
option' [-Werror]
------------------------+---------------------------
  Reporter:  strk       |      Owner:  robe
      Type:  defect     |     Status:  new
  Priority:  blocker    |  Milestone:  PostGIS 3.0.0
 Component:  buildbots  |    Version:  2.5.x
Resolution:             |   Keywords:
------------------------+---------------------------

Comment (by Algunenano):

 The true error is right on top of that:
 {{{
 lwgeom_functions_basic.c: In function 'LWGEOM_makepoly':
 /usr/local/pgsql/include/server/utils/array.h:305:21: error: operand of ?:
 changes signedness from 'int32' {aka 'int'} to 'long unsigned int' due to
 unsignedness of other operand [-Werror=sign-compare]
   305 |   (ARR_HASNULL(a) ? (a)->dataoffset :
 ARR_OVERHEAD_NONULLS(ARR_NDIM(a)))
       |                     ^~~~~~~~~~~~~~~
 /usr/local/pgsql/include/server/utils/array.h:311:21: note: in expansion
 of macro 'ARR_DATA_OFFSET'
   311 |   (((char *) (a)) + ARR_DATA_OFFSET(a))
       |                     ^~~~~~~~~~~~~~~
 lwgeom_functions_basic.c:1553:37: note: in expansion of macro
 'ARR_DATA_PTR'
  1553 |    GSERIALIZED *g = (GSERIALIZED *)(ARR_DATA_PTR(array) + offset);
       |                                     ^~~~~~~~~~~~
 In file included from lwgeom_geos.c:33:
 lwgeom_geos.c: In function 'pgis_union_geometry_array':
 /usr/local/pgsql/include/server/utils/array.h:305:21: error: operand of ?:
 changes signedness from 'int32' {aka 'int'} to 'long unsigned int' due to
 unsignedness of other operand [-Werror=sign-compare]
   305 |   (ARR_HASNULL(a) ? (a)->dataoffset :
 ARR_OVERHEAD_NONULLS(ARR_NDIM(a)))
       |                     ^~~~~~~~~~~~~~~
 /usr/local/pgsql/include/server/postgres.h:556:38: note: in definition of
 macro 'PointerGetDatum'
   556 | #define PointerGetDatum(X) ((Datum) (X))
       |                                      ^
 lwgeom_geos.c:397:3: note: in expansion of macro 'PG_RETURN_POINTER'
   397 |   PG_RETURN_POINTER((GSERIALIZED *)(ARR_DATA_PTR(array)));
       |   ^~~~~~~~~~~~~~~~~
 /usr/local/pgsql/include/server/utils/array.h:311:21: note: in expansion
 of macro 'ARR_DATA_OFFSET'
   311 |   (((char *) (a)) + ARR_DATA_OFFSET(a))
       |                     ^~~~~~~~~~~~~~~
 lwgeom_geos.c:397:37: note: in expansion of macro 'ARR_DATA_PTR'
   397 |   PG_RETURN_POINTER((GSERIALIZED *)(ARR_DATA_PTR(array)));
       |                                     ^~~~~~~~~~~~
 lwgeom_geos.c: At top level:
 }}}

 This was changed in r17865

 I don't know where `GCC_VERSION` comes from, but it isn't a predefined
 macro in my installation (gcc version 9.2.0).

-- 
Ticket URL: <https://trac.osgeo.org/postgis/ticket/4527#comment:1>
PostGIS <http://trac.osgeo.org/postgis/>
The PostGIS Trac is used for bug, enhancement & task tracking, a user and developer wiki, and a view into the subversion code repository of PostGIS project.