Explain in more detail why -pipe and -fno-common are good.

[p5sagit/p5-mst-13.2.git] / hints / darwin.sh

diff --git a/hints/darwin.sh b/hints/darwin.sh
index ff4b4fb..c2e7606 100644 (file)
--- a/hints/darwin.sh
+++ b/hints/darwin.sh
@@ -9,24 +9,33 @@
 
 # BSD paths
 case "$prefix" in
-'')    
+'')
+       # Default install; use non-system directories
        prefix='/usr/local'; # Built-in perl uses /usr
        siteprefix='/usr/local';
        vendorprefix='/usr/local'; usevendorprefix='define';
 
-       # 4BSD uses ${prefix}/share/man, not ${prefix}/man.
-       # Don't put man pages in ${prefix}/lib; that's goofy.
-       man1dir="${prefix}/share/man/man1";
-       man3dir="${prefix}/share/man/man3";
+       # Where to put modules.
+       privlib='/Library/Perl'; # Built-in perl uses /System/Library/Perl
+       sitelib='/Library/Perl';
+       vendorlib='/Network/Library/Perl';
+       ;;
+'/usr')
+       # We are building/replacing the built-in perl
+       siteprefix='/usr/local';
+       vendorprefix='/usr/local'; usevendorprefix='define';
 
        # Where to put modules.
-       # Built-in perl uses /System/Library/Perl
-       privlib='/Library/Perl';
+       privlib='/System/Library/Perl';
        sitelib='/Library/Perl';
        vendorlib='/Network/Library/Perl';
        ;;
 esac
 
+# 4BSD uses ${prefix}/share/man, not ${prefix}/man.
+man1dir="${prefix}/share/man/man1";
+man3dir="${prefix}/share/man/man3";
+
 ##
 # Tool chain settings
 ##
@@ -37,13 +46,17 @@ archname='darwin';
 # nm works.
 usenm='true';
 
-# Libc is in libsystem.
-#libc='/usr/lib/libSystem.dylib';
-
 # Optimize.
-optimize='-O3';
-
-# XXX Unclear why we require -pipe and -fno-common here.
+if [ "x$optimize" = 'x' ]; then
+    optimize='-O3'
+fi
+
+# -pipe: makes compilation go faster.
+# -fno-common: we don't like commons.  Common symbols are not allowed
+# in MH_DYLIB binaries, which is what libperl.dylib is.  You will fail
+# to link without that option, unless you otherwise eliminate all commons
+# by, for example, initializing all globals.
+# --Fred Sánchez
 ccflags="${ccflags} -pipe -fno-common"
 
 # At least on Darwin 1.3.x: