Re: [patch] build issues on Win32 - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [patch] build issues on Win32
Date
Msg-id 16045.1268236283@sss.pgh.pa.us
Whole thread Raw
In response to [patch] build issues on Win32  (Dag-Erling Smørgrav <des@des.no>)
Responses Re: [patch] build issues on Win32
Re: [patch] build issues on Win32
List pgsql-hackers
Dag-Erling Smørgrav <des@des.no> writes:
> I've run across a couple of stumbling blocks when building on Win32
> (specifically, XP + MinGW):

>  1. PostgreSQL's private versions of inet_aton etc. can conflict with
>     similar functions in other libraries (in my case, PostgreSQL's
>     inet_aton conflicts with libavformat's).

So what?  We don't link with those libraries.  The proposed #defines
seem like a completely bad idea, especially since as-presented they
would affect every platform not only yours.  We don't need the
maintenance/debugging headaches of routines that don't have the names
they appear to have.

>  ifeq ($(PORTNAME),win32)
> -LIBS += -lws2_32 -lshfolder
> +LIBS += -lws2_32 -lshfolder -lsecur32
>  endif

Surely this bit would need to be conditional on whether libsecur32
is available?

> -LDFLAGS="-Wl,--allow-multiple-definition"
> +LDFLAGS="${LDFLAGS} -Wl,--allow-multiple-definition"

That bit seems sane.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Dag-Erling Smørgrav
Date:
Subject: [patch] build issues on Win32
Next
From: Robert Haas
Date:
Subject: Re: renameatt() can rename attribute of index, sequence, ...