Re: pgsql: Fix compiling errors on Windows platform. Fix wrong usage of

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Tatsuo Ishii <ishii(at)postgresql(dot)org>
Cc: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Fix compiling errors on Windows platform. Fix wrong usage of
Date: 2012-10-07 14:53:00
Message-ID: 5071974C.6010708@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers


On 10/07/2012 10:41 AM, Tatsuo Ishii wrote:
> Fix compiling errors on Windows platform. Fix wrong usage of
> INT64CONST macro. Fix lo_hton64 and lo_ntoh64 not to use int32_t and
> uint32_t.

Are you sure this is going to fix the Windows issues?

On the buildfarm, pitta (gcc) reports:

x86_64-w64-mingw32-gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith -Wdeclaration-after-statement -Wendif-labels -Wmissing-format-attribute -Wformat-security -fno-strict-aliasing -fwrapv -fexcess-precision=standard -g -I../../../../src/include -I/home/pgrunner/bf/root/HEAD/pgsql.2948/../pgsql/src/include -I../pgsql/src/include/port/win32 -DEXEC_BACKEND -I/c/prog/mingwdep/include "-I/home/pgrunner/bf/root/HEAD/pgsql.2948/../pgsql/src/include/port/win32" -DBUILDING_DLL -c -o inv_api.o /home/pgrunner/bf/root/HEAD/pgsql.2948/../pgsql/src/backend/storage/large_object/inv_api.c
c:/mingw/msys/1.0/home/pgrunner/bf/root/HEAD/pgsql.2948/../pgsql/src/backend/storage/large_object/inv_api.c: In function 'inv_seek':
c:/mingw/msys/1.0/home/pgrunner/bf/root/HEAD/pgsql.2948/../pgsql/src/backend/storage/large_object/inv_api.c:389:1: error: 'LOBLKSIZELL' undeclared (first use in this function)

and currawong (MSVC) also says:

"d:\bf\root\HEAD\pgsql.2172\pgsql.sln" (default target) (1) ->
(postgres target) ->
.\src\backend\storage\large_object\inv_api.c(389): error C2065: 'LOBLKSIZELL' : undeclared identifier

It doesn't appear that this patch will fix that at all.

cheers

andrew

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2012-10-07 14:57:18 Re: pgsql: Fix compiling errors on Windows platform. Fix wrong usage of
Previous Message Tatsuo Ishii 2012-10-07 14:41:26 pgsql: Fix compiling errors on Windows platform. Fix wrong usage of