01/01: gnu: magit: Do not set 'magit-git-executable' to absolute file name.

[Mark H. Weaver]

mhw pushed a commit to branch master
in repository guix.

commit 5fe9ba59ba1cea12a70d011aacbace52e3bfda18
Author: Mark H Weaver <address@hidden>
Date:   Wed Feb 14 03:48:20 2018 -0500

    gnu: magit: Do not set 'magit-git-executable' to absolute file name.
    
    Fixes <https://bugs.gnu.org/30434>.
    Reported by Ricardo Wurmus <address@hidden>.
    
    * gnu/packages/emacs.scm (magit)[arguments]: Do not modify the default value
    of 'magit-git-executable'.  Remove "#:modules" and "#:imported-modules".
---
 gnu/packages/emacs.scm | 15 +++------------
 1 file changed, 3 insertions(+), 12 deletions(-)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 2ddea64..dd3a6ac 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -1,7 +1,7 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2014 Taylan Ulrich Bayirli/Kammer <address@hidden>
 ;;; Copyright © 2013, 2014, 2015, 2016, 2017, 2018 Ludovic Courtès 
<address@hidden>
-;;; Copyright © 2014, 2015, 2016 Mark H Weaver <address@hidden>
+;;; Copyright © 2014, 2015, 2016, 2018 Mark H Weaver <address@hidden>
 ;;; Copyright © 2014, 2015, 2016, 2017 Alex Kost <address@hidden>
 ;;; Copyright © 2015 Federico Beffa <address@hidden>
 ;;; Copyright © 2015, 2016, 2017, 2018 Ricardo Wurmus <address@hidden>
@@ -449,13 +449,7 @@ on stdout instead of using a socket as the Emacsclient 
does.")
        ;; XXX Add 'magit-popup' dependency for the next release (after 2.11.0).
        ("with-editor" ,emacs-with-editor)))
     (arguments
-     `(#:modules ((guix build gnu-build-system)
-                  (guix build utils)
-                  (guix build emacs-utils))
-       #:imported-modules (,@%gnu-build-system-modules
-                           (guix build emacs-utils))
-
-       #:test-target "test"
+     `(#:test-target "test"
        #:tests? #f               ; tests are not included in the release
 
        #:make-flags
@@ -477,10 +471,7 @@ on stdout instead of using a socket as the Emacsclient 
does.")
          (add-before
           'build 'patch-exec-paths
           (lambda* (#:key inputs #:allow-other-keys)
-            (let ((git  (assoc-ref inputs "git"))
-                  (perl (assoc-ref inputs "perl")))
-              (emacs-substitute-variables "lisp/magit-git.el"
-                ("magit-git-executable" (string-append git "/bin/git")))
+            (let ((perl (assoc-ref inputs "perl")))
               (substitute* "lisp/magit-sequence.el"
                 (("perl") (string-append perl "/bin/perl")))
               #t))))))