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 1ac5123a8d5e2e2c4ade38e546f6e399c1be7dc6..34b4a8b504f70d57d21c4bf907cc1190f03fb40b 100644 (file)
--- 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