extension build issue with PostgreSQL 10 on Centos6

2017-09-30 7:54 GMT+02:00 Pavel Stehule <pavel.stehule@gmail.com>:

Hi

There is new, not well solved dependency. I have not any problem to build
extension with PostgreSQL 9.6 there

[oracle@ora2pg plpgsql_check]$ make
gcc -Wall -Wmissing-prototypes -Wpointer-arith
-Wdeclaration-after-statement -Wendif-labels -Wmissing-format-attribute
-Wformat-security -fno-strict-aliasing -fwrapv -O2 -g -pipe -Wall
-Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector
--param=ssp-buffer-size=4 -m64 -mtune=generic -DLINUX_OOM_SCORE_ADJ=0 -fPIC
-I/usr/pgsql-10/lib/pgxs/src/makefiles/../../src/pl/plpgsql/src -I. -I./
-I/usr/pgsql-10/include/server -I/usr/pgsql-10/include/internal
-I/usr/include -D_GNU_SOURCE -I/usr/include/libxml2 -I/usr/include -c -o
plpgsql_check.oplpgsql_check.c
In file included from /usr/pgsql-10/include/server/tsearch/ts_locale.h:18,
from plpgsql_check.c:88:
/usr/pgsql-10/include/server/utils/pg_locale.h:19:26: error:
unicode/ucol.h: Adresář nebo soubor neexistuje
In file included from /usr/pgsql-10/include/server/tsearch/ts_locale.h:18,
from plpgsql_check.c:88:
/usr/pgsql-10/include/server/utils/pg_locale.h:94: error: expected
specifier-qualifier-list before ‘UCollator’
/usr/pgsql-10/include/server/utils/pg_locale.h:108: error: expected ‘)’
before ‘*’ token
/usr/pgsql-10/include/server/utils/pg_locale.h:109: warning: type
defaults to ‘int’ in declaration of ‘UChar’
/usr/pgsql-10/include/server/utils/pg_locale.h:109: error: expected ‘;’,
‘,’ or ‘)’ before ‘*’ token
make: *** [plpgsql_check.o] Error 1

This mean not optional dependency on ICU.

It was fixed by libicu-devel install - so it should be on dep list of
postgresql10-devel

Pavel Stehule <pavel.stehule@gmail.com> writes:

/usr/pgsql-10/include/server/utils/pg_locale.h:19:26: error:
unicode/ucol.h: Adresář nebo soubor neexistuje

This mean not optional dependency on ICU.

Really? That file has

#ifdef USE_ICU
#include <unicode/ucol.h>
#endif

So if you managed to get that error without having said --with-icu,
I'd be very interested to see how.

regards, tom lane

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

2017-09-30 8:18 GMT+02:00 Tom Lane <tgl@sss.pgh.pa.us>:

Pavel Stehule <pavel.stehule@gmail.com> writes:

/usr/pgsql-10/include/server/utils/pg_locale.h:19:26: error:
unicode/ucol.h: Adresář nebo soubor neexistuje

This mean not optional dependency on ICU.

Really? That file has

#ifdef USE_ICU
#include <unicode/ucol.h>
#endif

So if you managed to get that error without having said --with-icu,
I'd be very interested to see how.

probably rpm is created with --with-icu

Hi,

On Sat, 2017-09-30 at 08:19 +0200, Pavel Stehule wrote:

probably rpm is created with --with-icu

Sure, I compile all v10 RPMs with ICU support.

Regards
--
Devrim Gündüz
EnterpriseDB: https://www.enterprisedb.com
PostgreSQL Consultant, Red Hat Certified Engineer
Twitter: @DevrimGunduz , @DevrimGunduzTR

Devrim =?ISO-8859-1?Q?G=FCnd=FCz?= <devrim@gunduz.org> writes:

On Sat, 2017-09-30 at 08:19 +0200, Pavel Stehule wrote:

probably rpm is created with --with-icu

Sure, I compile all v10 RPMs with ICU support.

So the point is that postgresql-devel now needs to have a dependency
on icu-devel.

regards, tom lane

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Hi,

On Sat, 2017-09-30 at 11:15 -0400, Tom Lane wrote:

So the point is that postgresql-devel now needs to have a dependency
on icu-devel.

Oh, I see. Ack, added. Will appear in 10.0 RPMs.

Regards,
--
Devrim Gündüz
EnterpriseDB: https://www.enterprisedb.com
PostgreSQL Consultant, Red Hat Certified Engineer
Twitter: @DevrimGunduz , @DevrimGunduzTR

2017-10-01 4:55 GMT+02:00 Devrim Gündüz <devrim@gunduz.org>:

Hi,

On Sat, 2017-09-30 at 11:15 -0400, Tom Lane wrote:

So the point is that postgresql-devel now needs to have a dependency
on icu-devel.

Oh, I see. Ack, added. Will appear in 10.0 RPMs.

Thank you

Pavel