From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Sat, 5 Sep 2009 21:14:04 +0000 (+0000)
Subject: Put back "ifeq ($(PORTNAME), solaris)", this time with some documentation
X-Git-Url: http://git.postgresql.org/gitweb/?a=commitdiff_plain;h=4ea8d1ffa0ec251119b291216db9d12c99687456;p=users%2Fsimon%2Fpostgres.git

Put back "ifeq ($(PORTNAME), solaris)", this time with some documentation
of why it's not as broken as it appears on first glance.
---

diff --git a/src/backend/Makefile b/src/backend/Makefile
index 1ac5123a8d..34b4a8b504 100644
--- a/src/backend/Makefile
+++ b/src/backend/Makefile
@@ -20,9 +20,17 @@ SUBDIRS = access bootstrap catalog parser commands executor foreign lib libpq \
 
 include $(srcdir)/common.mk
 
+# As of 9/2009:
+# * The probes.o file is necessary for dtrace support on Solaris.
+# * OS X's dtrace doesn't use it and doesn't even recognize the -G option.
+# * Systemtap's dtrace will take -G, but it produces a useless empty file.
+# So, build probes.o only on Solaris.
+# This will likely need adjustment as other platforms add dtrace support.
+ifeq ($(PORTNAME), solaris)
 ifeq ($(enable_dtrace), yes)
 LOCALOBJS += utils/probes.o
 endif
+endif
 
 OBJS = $(SUBDIROBJS) $(LOCALOBJS) $(top_builddir)/src/port/libpgport_srv.a