Am 29.09.2007 um 09:03 schrieb Adrian Robert:

> Thanks for the patch, I'll incorporate for next release.

This patch is partly of no use and partly dangerous!

• It's of no big use in case Emacs.app gets built as an application  
bundle. Then other rules I don't know determine what the load-path  
will be. It works perfectly in the case of a lean application, when  
the Elisp from /usr/local is used.

• The danger arises from the fact that the compile script is never  
cleaning the directory tree. So every time compile is executed  
another DOC file is created and installed in the bundle and src/ 
epaths.h is augmented once more with the local Lisp path extension.

Actually there is one more bug in my patch: only one case is  
executed, the one that comes first. The if clause has of course to be  
changed to become a while loop:

--- compile.orig        2007-09-18 16:20:43.000000000 +0200
+++ compile     2007-10-02 14:27:42.000000000 +0200
@@ -22,15 +22,22 @@

  # Set up path and config variables.

-if [ $# -gt 0 ]; then
-    if [ $# -ne 2 ] || [ $1 != "--shared-lisp" ]; then
-       echo "Usage: $0 [--shared-lisp 'install_root']"
-       exit
-    fi
-    PREFIX=${2}
-else
-    PREFIX=""
-fi
+PREFIX=""
+while [ $# -gt 0 ]; do
+    case "$1" in
+       --shared-lisp)
+           PREFIX="${2}"
+           shift ; shift
+           ;;
+       --enable-locallisppath=*)
+           locallisppath=`echo "$1" | sed s/--enable-locallisppath=//`
+           shift
+           ;;
+       *)
+           echo "Usage: $0 [--shared-lisp 'install_root' --enable- 
locallisppath='some path(s)']"
+           exit
+    esac
+done

  DISTDIR=`pwd`/..

@@ -116,6 +123,12 @@
      fi
  fi

+# Pete's addition for local lisp load-path
+    echo "Local lisp path is being enabled"
+    (cd src; mv epaths.h epaths.h-orig; \
+       printf "s,\(#define PATH_LOADSEARCH \"\),\\\1%s:,\n" "$ 
{locallisppath}" > locallisppath.sed; \
+       cat epaths.h-orig | sed -f locallisppath.sed > epaths.h; rm  
locallisppath.sed)
+
  # Go (installs binaries to ./build/Emacs.app, lisp to there or PREFIX)
  echo "make install"
  $MAKE


And what is the meaning of the empty font.font file in src?


--
Greetings

   Pete

This is a signature virus.  Add me to your signature and help me to live



-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Emacs-app-dev- mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/emacs-app-dev-

Reply via email to