CVS tip build failure (win32)

Started by Dave Pageover 20 years ago4 messages
#1Dave Page
dpage@vale-housing.co.uk

I'm seeing the following failure on win32, post roles patch application:

gcc -O2 -Wall -Wmissing-prototypes -Wpointer-arith
-Wdeclaration-after-statement -Wold-style-definition -Wendif-labels
-fno-strict-aliasing -I../../../src/include -I./src/include/port/win32
-DEXEC_BACKEND "-I../../../src/include/port/win32" -DBUILDING_DLL -c
-o namespace.o namespace.c
In file included from namespace.c:38:
../../../src/include/utils/acl.h:214: error: conflicting types for
'InitializeAcl'
c:/mingw/bin/../lib/gcc/mingw32/3.4.2/../../../../include/winbase.h:1571
: error: previous declaration of 'InitializeAcl' was here
../../../src/include/utils/acl.h:214: error: conflicting types for
'InitializeAcl'
c:/mingw/bin/../lib/gcc/mingw32/3.4.2/../../../../include/winbase.h:1571
: error: previous declaration of 'InitializeAcl' was here
make[3]: *** [namespace.o] Error 1
make[3]: Leaving directory `/cvs/pgsql.dbsize/src/backend/catalog'
make[2]: *** [catalog-recursive] Error 2
make[2]: Leaving directory `/cvs/pgsql.dbsize/src/backend'
make[1]: *** [all] Error 2
make[1]: Leaving directory `/cvs/pgsql.dbsize/src'
make: *** [all] Error 2

That's following a cvs update and a make clean. All was fine before I
updated :-(

Regards, Dave

#2Stephen Frost
sfrost@snowman.net
In reply to: Dave Page (#1)
Re: CVS tip build failure (win32)

* Dave Page (dpage@vale-housing.co.uk) wrote:

I'm seeing the following failure on win32, post roles patch application:

[...]

'InitializeAcl'

[...]

That's following a cvs update and a make clean. All was fine before I
updated :-(

Wow. Apparently 'InitializeAcl' is part of the Windows API. My bad,
sorry about that. I guess we should rename it? I don't see any
particular problem with that (it's only used in 3 places), perhpas
'PGInitializeAcl' or 'InitializeRoleCache' or some such.

Thanks,

Stephen

#3Tom Lane
tgl@sss.pgh.pa.us
In reply to: Dave Page (#1)
Re: CVS tip build failure (win32)

"Dave Page" <dpage@vale-housing.co.uk> writes:

../../../src/include/utils/acl.h:214: error: conflicting types for
'InitializeAcl'
c:/mingw/bin/../lib/gcc/mingw32/3.4.2/../../../../include/winbase.h:1571
: error: previous declaration of 'InitializeAcl' was here

Grumble. I'll change the routine name. Thanks...

regards, tom lane

#4Dave Page
dpage@vale-housing.co.uk
In reply to: Tom Lane (#3)
Re: CVS tip build failure (win32)

-----Original Message-----
From: Stephen Frost [mailto:sfrost@snowman.net]
Sent: 28 June 2005 18:30
To: Dave Page
Cc: PostgreSQL-development
Subject: Re: [HACKERS] CVS tip build failure (win32)

* Dave Page (dpage@vale-housing.co.uk) wrote:

I'm seeing the following failure on win32, post roles patch

application:
[...]

'InitializeAcl'

[...]

That's following a cvs update and a make clean. All was

fine before I

updated :-(

Wow. Apparently 'InitializeAcl' is part of the Windows API. My bad,
sorry about that. I guess we should rename it? I don't see any
particular problem with that (it's only used in 3 places), perhpas
'PGInitializeAcl' or 'InitializeRoleCache' or some such.

Yes, it is. Sorry was a bit pushed for time and couldn't investigate
earlier.

PGInitializeAcl sounds OK to me.

/D