emacs-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Emacs-diffs] /srv/bzr/emacs/emacs-24 r108239: * Makefile.in (install-ar


From: Glenn Morris
Subject: [Emacs-diffs] /srv/bzr/emacs/emacs-24 r108239: * Makefile.in (install-arch-dep): Do not hard-code version number.
Date: Fri, 02 Nov 2012 01:47:37 -0000
User-agent: Bazaar (2.5.0)

------------------------------------------------------------
revno: 108239
committer: Glenn Morris <address@hidden>
branch nick: trunk
timestamp: Tue 2012-05-15 13:32:42 -0400
message:
  * Makefile.in (install-arch-dep): Do not hard-code version number.
modified:
  ChangeLog
  Makefile.in
=== modified file 'ChangeLog'
--- a/ChangeLog 2012-05-15 17:09:00 +0000
+++ b/ChangeLog 2012-05-15 17:32:42 +0000
@@ -1,5 +1,7 @@
 2012-05-15  Glenn Morris  <address@hidden>
 
+       * Makefile.in (install-arch-dep): Do not hard-code version number.
+
        * Makefile.in (install-arch-dep): NS install no longer needs to
        symlink libexec/* into bin/, since 2012-05-14 ns_init_paths change.
 

=== modified file 'Makefile.in'
--- a/Makefile.in       2012-05-15 17:09:00 +0000
+++ b/Makefile.in       2012-05-15 17:32:42 +0000
@@ -475,14 +475,13 @@
            rm -fr share ) ; \
          ( cd ${ns_appbindir} ; \
            if cd libexec ; then dir=emacs/*/*/* ; $(MV_DIRS); \
-           rm -fr emacs; if cd ../bin; then rm -f emacs emacs-24*; \
-           fi ; fi ) ; \
+           rm -fr emacs; rm -f ../bin/emacs ../bin/$(EMACSFULL) ; \
+           fi ) ; \
        else true ; fi
 
-## FIXME is the emacs-24* bit above really necessary and correct?
-## What if I have 24.1 and 24.2 installed at the same time?
-## In any case, it should use something like echo $version | sed 's/\..*//'
-## instead of hard-coding a version.
+## FIXME NS self-contained: rather than installing emacs and
+## EMACSFULL, then deleting them, obviously it would be better to not
+## install them in the first place.
 
 ## http://lists.gnu.org/archive/html/emacs-devel/2007-10/msg01672.html
 ## Needs to be the user running install, so configure can't set it.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]