Bruce Momjian wrote:
> Win32 buildfarm members are red because of my inet_pton changes. I will
> look into this in the next day, and also improve how we include C files
> from /port for libpq.
OK, I have accomplished both goals with the two attached, applied
patches.
--
Bruce Momjian <br...@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
diff --git a/src/interfaces/libpq/Makefile b/src/interfaces/libpq/Makefile
index 74ae79a..0be6a72 100644
*** /tmp/kBcRnb_Makefile Thu Nov 25 12:48:37 2010
--- src/interfaces/libpq/Makefile Thu Nov 25 12:44:41 2010
*************** override CFLAGS += $(PTHREAD_CFLAGS)
*** 25,48 ****
endif
# Need to recompile any libpgport object files because we need these
! # object files to use the same compile flags as libpq. If we used
! # the object files from libpgport, this would not be true on all
! # platforms. We filter some object files so we only use object
! # files configure says we need.
LIBS := $(LIBS:-lpgport=)
OBJS= fe-auth.o fe-connect.o fe-exec.o fe-misc.o fe-print.o fe-lobj.o \
fe-protocol2.o fe-protocol3.o pqexpbuffer.o pqsignal.o fe-secure.o \
libpq-events.o \
! md5.o ip.o wchar.o encnames.o inet_net_ntop.o noblock.o pgstrcasecmp.o thread.o \
! $(filter crypt.o getaddrinfo.o inet_aton.o open.o snprintf.o strerror.o strlcpy.o win32error.o, $(LIBOBJS))
ifeq ($(PORTNAME), cygwin)
override shlib = cyg$(NAME)$(DLSUFFIX)
endif
ifeq ($(PORTNAME), win32)
! OBJS += win32.o pgsleep.o libpqrc.o
libpqrc.o: libpq.rc
$(WINDRES) -i $< -o $@
--- 25,57 ----
endif
# Need to recompile any libpgport object files because we need these
! # object files to use the same compile flags as libpq; some
! # platforms require special flags for all libpq object files.
LIBS := $(LIBS:-lpgport=)
+ # external object files that are always used by libpq
+ BACKEND_LIBPQ = md5 ip
+ UTILS_MB = encnames wchar
+ PERM_PGPORT = inet_net_ntop noblock pgstrcasecmp thread
+ ifeq ($(PORTNAME), win32)
+ PERM_PGPORT += pgsleep
+ endif
+
+
+ # pgport object files are used by libpq if identified by configure
+ OPT_PGPORT = $(filter $(addsuffix .o, crypt getaddrinfo inet_aton open snprintf strerror strlcpy win32error), $(LIBOBJS))
+
OBJS= fe-auth.o fe-connect.o fe-exec.o fe-misc.o fe-print.o fe-lobj.o \
fe-protocol2.o fe-protocol3.o pqexpbuffer.o pqsignal.o fe-secure.o \
libpq-events.o \
! $(addsuffix .o, $(PERM_PGPORT) $(OPT_PGPORT) $(BACKEND_LIBPQ) $(UTILS_MB))
ifeq ($(PORTNAME), cygwin)
override shlib = cyg$(NAME)$(DLSUFFIX)
endif
ifeq ($(PORTNAME), win32)
! OBJS += win32.o libpqrc.o
libpqrc.o: libpq.rc
$(WINDRES) -i $< -o $@
*************** backend_src = $(top_srcdir)/src/backend
*** 77,93 ****
# We use several backend modules verbatim, but since we need to
# compile with appropriate options to build a shared lib, we can't
# necessarily use the same object files as the backend uses. Instead,
! # symlink the source files in here and build our own object file.
! # For port modules, this only happens if configure decides the module
! # is needed (see filter hack in OBJS, above).
! crypt.c getaddrinfo.c inet_aton.c inet_net_ntop.c noblock.c open.c pgstrcasecmp.c snprintf.c strerror.c strlcpy.c thread.c win32error.c pgsleep.c: % : $(top_srcdir)/src/port/%
rm -f $@ && $(LN_S) $< .
! md5.c ip.c: % : $(backend_src)/libpq/%
rm -f $@ && $(LN_S) $< .
! encnames.c wchar.c : % : $(backend_src)/utils/mb/%
rm -f $@ && $(LN_S) $< .
--- 86,100 ----
# We use several backend modules verbatim, but since we need to
# compile with appropriate options to build a shared lib, we can't
# necessarily use the same object files as the backend uses. Instead,
! # we symlink the source files in here and build our own object files.
! $(addsuffix .c, $(PERM_PGPORT) $(OPT_PGPORT)): % : $(top_srcdir)/src/port/%
rm -f $@ && $(LN_S) $< .
! $(addsuffix .c, $(BACKEND_LIBPQ)): % : $(backend_src)/libpq/%
rm -f $@ && $(LN_S) $< .
! $(addsuffix .c, $(UTILS_MB)): % : $(backend_src)/utils/mb/%
rm -f $@ && $(LN_S) $< .
*************** uninstall: uninstall-lib
*** 124,130 ****
rm -f '$(DESTDIR)$(datadir)/pg_service.conf.sample'
clean distclean: clean-lib
! rm -f $(OBJS) pg_config_paths.h crypt.c getaddrinfo.c inet_aton.c noblock.c open.c pgstrcasecmp.c snprintf.c strerror.c strlcpy.c thread.c md5.c ip.c encnames.c wchar.c win32error.c pgsleep.c pthread.h libpq.rc
# Might be left over from a Win32 client-only build
rm -f pg_config_paths.h
--- 131,137 ----
rm -f '$(DESTDIR)$(datadir)/pg_service.conf.sample'
clean distclean: clean-lib
! rm -f $(OBJS) pg_config_paths.h pthread.h libpq.rc $(addsuffix .c, $(BACKEND_LIBPQ) $(UTILS_MB) $(PERM_PGPORT) $(OPT_PGPORT))
# Might be left over from a Win32 client-only build
rm -f pg_config_paths.h
diff --git a/src/port/getaddrinfo.c b/src/port/getaddrinfo.c
index 807f5bd..3d33706 100644
*** /tmp/d4DKgd_getaddrinfo.c Thu Nov 25 12:52:47 2010
--- src/port/getaddrinfo.c Thu Nov 25 12:52:06 2010
*************** getnameinfo(const struct sockaddr * sa,
*** 390,396 ****
{
if (sa->sa_family == AF_INET)
{
! if (inet_net_ntop(AF_INET, ((struct sockaddr_in *) sa)->sin_addr,
sa->sa_family == AF_INET ? 32 : 128,
node, nodelen) == NULL)
return EAI_MEMORY;
--- 390,396 ----
{
if (sa->sa_family == AF_INET)
{
! if (inet_net_ntop(AF_INET, &((struct sockaddr_in *) sa)->sin_addr,
sa->sa_family == AF_INET ? 32 : 128,
node, nodelen) == NULL)
return EAI_MEMORY;
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
