Build error with latests change

[Stephen Bannasch]

At 11:43 AM -0800 3/8/11, Gary Meyer wrote:
>On Mar 8, 2011, at 6:05 AM, Henri Gomez wrote:
>
>> Did we are alone to build these sources ?
>>
>> I'm more than worried to see such errors commited in HG ;(
>
>
>No Henri, you are not the only one building these sources.  I build them all the time for Apple, and yes these errors were seen.  Alex only compiled the code on one operating system (a yet to be released one).  I compiled them on SnowLeopard, and saw these problems as well.  Alex has been notified that SnowLeopard seems to see these errors while his OS doesn't.
>Gary.
>~~~~~~~~~~~~~~~~~~~~~~~
>Gary Meyer
>Java Software Build Engineer
>gary.meyer at apple.com

from bsd-port list:

At 2:48 PM -0500 3/8/11, Stephen Bannasch wrote:
>Hi Greg,
>
>The change you pulled from the Mac OS X port for jdk/make/common/shared/Defs-bsd.gmk
>doesn't work -- the result is a unhelpful error about being unable to compile freetype.
>
>Apply these changes and it will:


[jdk]$ hg diff
diff --git a/make/common/shared/Defs-bsd.gmk b/make/common/shared/Defs-bsd.gmk
--- a/make/common/shared/Defs-bsd.gmk
+++ b/make/common/shared/Defs-bsd.gmk
@@ -177,6 +177,7 @@
 
 # Macro to check it's input file for banned dependencies and verify the
 #   binary built properly. Relies on process exit code.
+define binary_file_verification # binary_file
 ifeq ($(OS_VENDOR), Apple)
 ( \
   $(ECHO) "Checking for mapfile use in: $1" && \
@@ -189,7 +190,6 @@
   ( $(OTOOL) -l $1 | $(EGREP) 'path ' ) \
 )
 else
-define binary_file_verification # binary_file
 ( \
   $(ECHO) "Checking for mapfile use in: $1" && \
   if [ "`$(NM) -D -g --defined-only $1 | $(EGREP) 'SUNWprivate'`" = "" ] ; then \
@@ -200,5 +200,5 @@
   $(ECHO) "RUNPATH for: $1" && \
   ( $(READELF) -d $1 | $(EGREP) 'NEEDED|RUNPATH|RPATH' ) \
 )
+endif
 endef
-endif
[jdk]$