From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mp11.migadu.com ([2001:41d0:2:4a6f::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by ms5.migadu.com with LMTPS id UERvM0OFkWLsCgEAbAwnHQ (envelope-from ) for ; Sat, 28 May 2022 04:13:23 +0200 Received: from aspmx1.migadu.com ([2001:41d0:2:4a6f::]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) by mp11.migadu.com with LMTPS id SI1qM0OFkWKgWAAA9RJhRA (envelope-from ) for ; Sat, 28 May 2022 04:13:23 +0200 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by aspmx1.migadu.com (Postfix) with ESMTPS id 1DF6B3EB6E for ; Sat, 28 May 2022 04:13:23 +0200 (CEST) Received: from localhost ([::1]:50934 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1nulxC-0000ek-8H for larch@yhetil.org; Fri, 27 May 2022 22:13:22 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:57360) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1nulx2-0000eQ-QS for guix-patches@gnu.org; Fri, 27 May 2022 22:13:13 -0400 Received: from debbugs.gnu.org ([209.51.188.43]:42240) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1nulwr-0001Y0-Up for guix-patches@gnu.org; Fri, 27 May 2022 22:13:12 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1nulwr-0003kQ-NG for guix-patches@gnu.org; Fri, 27 May 2022 22:13:01 -0400 X-Loop: help-debbugs@gnu.org Subject: [bug#55517] Acknowledgement ([PATCH]: gnu: emacs-magit: Drop the libgit backend.) Resent-From: Zhu Zihao Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Sat, 28 May 2022 02:13:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 55517 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: Liliana Marie Prikler Cc: 55517@debbugs.gnu.org Received: via spool by 55517-submit@debbugs.gnu.org id=B55517.165370392414339 (code B ref 55517); Sat, 28 May 2022 02:13:01 +0000 Received: (at 55517) by debbugs.gnu.org; 28 May 2022 02:12:04 +0000 Received: from localhost ([127.0.0.1]:36136 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1nulvv-0003jC-QR for submit@debbugs.gnu.org; Fri, 27 May 2022 22:12:04 -0400 Received: from mail-m975.mail.163.com ([123.126.97.5]:44784) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1nulvq-0003ib-Ln for 55517@debbugs.gnu.org; Fri, 27 May 2022 22:12:02 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=163.com; s=s110527; h=From:Subject:Date:Message-ID:MIME-Version; bh=wN3S2 99/bqyjH/HzzafqmQZZmTjdkDQnUnnNIfO43Rg=; b=F048hy9AoWEolQxBBjwn2 hDOxuKnVhWMDD3raPGRyoH3irroRllK0PDe0/RtmnG1ad2eIuJFpk973+gokCKHx 61krE0Rke9m6UppdvlL2SRkJ4eAJa+jFlvTt5/u0ezxbIX8ZvZmpjmhLQnzM6+wr ovqzger462Nm/UhrnyXog0= Received: from asus-laptop (unknown [163.125.202.234]) by smtp5 (Coremail) with SMTP id HdxpCgDncCbkhJFixMiaEg--.9102S2; Sat, 28 May 2022 10:11:49 +0800 (CST) References: <868rqykt19.fsf@163.com> <86sfp6je99.fsf@163.com> <86leuw507n.fsf@163.com> <07286c2df8ffb434e353bdde7bf8d427e9d7d9b2.camel@ist.tugraz.at> <86o7zpaoih.fsf@163.com> User-agent: mu4e 1.6.10; emacs 27.2 From: Zhu Zihao Date: Sat, 28 May 2022 10:11:27 +0800 In-reply-to: Message-ID: <8635gu9zj5.fsf@163.com> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" X-CM-TRANSID: HdxpCgDncCbkhJFixMiaEg--.9102S2 X-Coremail-Antispam: 1Uf129KBjDUn29KB7ZKAUJUUUUU529EdanIXcx71UUUUU7v73 VFW2AGmfu7bjvjm3AaLaJ3UbIYCTnIWIevJa73UjIFyTuYvjxUj18BDUUUU X-Originating-IP: [163.125.202.234] X-CM-SenderInfo: pdoosuxxwbztlvw6il2tof0z/1tbiTwYPr1sGbMr7lQAAs8 X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+larch=yhetil.org@gnu.org Sender: "Guix-patches" X-Migadu-Flow: FLOW_IN X-Migadu-To: larch@yhetil.org X-Migadu-Country: US ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=yhetil.org; s=key1; t=1653704003; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:resent-cc:resent-from:resent-sender: resent-message-id:in-reply-to:in-reply-to:references:references: list-id:list-help:list-unsubscribe:list-subscribe:list-post: dkim-signature; bh=wN3S299/bqyjH/HzzafqmQZZmTjdkDQnUnnNIfO43Rg=; b=punCRZAv2TSncZwGg2gkAS8+UykutQOXEY8p03hWMXpLmoEh8z7gWj6L1qooB7ouaURdQp KfbpDMgEI5V5HNhRuhaKUr/mgP61AR7OTNdyq0LosOoGglNWFa+dv52JTCj2dIs+3ahvH6 4ZjkbsC16uWlFjt128nmixOMM7BhPajugg+FrAnh7jHJjMymoWQy3aYEu2hFQAjvd1PSoS SRVGcrHRHQSEk2OOT9jtarxOgueXVBLrv+LJa3JMX8aEf2YWYvRs2I3kGvvELNDxp6hQUu Y5ejjLdTBnBqpk6nBv3DFgjtVdjCHysCc7s8/RWsaAM8bco5jPp3+/dBeGqyXA== ARC-Seal: i=1; s=key1; d=yhetil.org; t=1653704003; a=rsa-sha256; cv=none; b=GUqoRuWcSBbpXsPZI7MVznTjHVwHm5J1HYpiczUP09OZpnNxoQW9Rf9Etm1Tu9Ae6hotLp ScqPqBBQKS+Ax+/h99WKYlZNjFhP4qr2KQytgCEJX/OpihmCj7ZSUZUpdpkGq1FR8yK3Hl Flk3KUeTnMfbxGnO6QeQSWSCyHvYCCdi1SHlsA9ng67g3GgF0xtwj0XNaLseImCHiruciP x65vPsQxxXiwG2mztjFgEj+Hx6plAVDUV1mZYAYEVOHJyp9v2MHHzD7QjdLD/xlU+ayh8z kBxl1FbYqkeYyyZhmmLdbMwlFUWoRkuf/cbNGEX5Zd7KHRfXfkKBbRNSlzKnew== ARC-Authentication-Results: i=1; aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=163.com header.s=s110527 header.b=F048hy9A; dmarc=fail reason="SPF not aligned (relaxed)" header.from=163.com (policy=none); spf=pass (aspmx1.migadu.com: domain of "guix-patches-bounces+larch=yhetil.org@gnu.org" designates 209.51.188.17 as permitted sender) smtp.mailfrom="guix-patches-bounces+larch=yhetil.org@gnu.org" X-Migadu-Spam-Score: 4.06 Authentication-Results: aspmx1.migadu.com; dkim=fail ("headers rsa verify failed") header.d=163.com header.s=s110527 header.b=F048hy9A; dmarc=fail reason="SPF not aligned (relaxed)" header.from=163.com (policy=none); spf=pass (aspmx1.migadu.com: domain of "guix-patches-bounces+larch=yhetil.org@gnu.org" designates 209.51.188.17 as permitted sender) smtp.mailfrom="guix-patches-bounces+larch=yhetil.org@gnu.org" X-Migadu-Queue-Id: 1DF6B3EB6E X-Spam-Score: 4.06 X-Migadu-Scanner: scn0.migadu.com X-TUID: Q3GntDm0F7fE --=-=-= Content-Type: multipart/signed; boundary="==-=-="; micalg=pgp-sha256; protocol="application/pgp-signature" --==-=-= Content-Type: text/plain --==-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iIsEARYIADMWIQRefA5qkqvnKdl/GTlmOX+E92aT+QUCYpGE3hUcYWxsX2J1dF9s YXN0QDE2My5jb20ACgkQZjl/hPdmk/kx9QD+Kg0Dun5hT+EGxLf/H60ClAbavWkY U1nroWG5oQ7E3bcBAJFp5Uuntz9Jj0EtdMO8tGCPMGkqsZ6YN7us2BpazTsP =JcKD -----END PGP SIGNATURE----- --==-=-=-- --=-=-= Content-Type: text/x-patch Content-Disposition: inline; filename=0001-gnu-emacs-magit-Use-G-expressions.patch >From 74cf2719131d2ebaa23c042fc2d3b5aeddc2be5c Mon Sep 17 00:00:00 2001 From: Zhu Zihao Date: Sun, 22 May 2022 17:43:42 +0800 Subject: [PATCH 1/2] gnu: emacs-magit: Use G-expressions. * gnu/packages/emacs-xyz.scm (emacs-magit)[arguments]: Use G-expressions. : In phase 'patch-exec-paths', use search-input-file for perl executable. --- gnu/packages/emacs-xyz.scm | 91 +++++++++++++++++++------------------- 1 file changed, 46 insertions(+), 45 deletions(-) diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm index eed2f4b71c..a9dc687af4 100644 --- a/gnu/packages/emacs-xyz.scm +++ b/gnu/packages/emacs-xyz.scm @@ -869,51 +869,52 @@ (define-public emacs-magit (base32 "0cxyvp2aav27znc7mf6c83q5pddpdniaqkrxn1r8dbgr540qmnpn")))) (build-system emacs-build-system) (arguments - `(#:emacs ,emacs-no-x ;module support is required - #:tests? #t - #:test-command '("make" "test") - #:phases - (modify-phases %standard-phases - (add-after 'unpack 'build-info-manual - (lambda _ - (invoke "make" "info") - ;; Copy info files to the lisp directory, which acts as - ;; the root of the project for the emacs-build-system. - (for-each (lambda (f) - (install-file f "lisp")) - (find-files "Documentation" "\\.info$")) - (chdir "lisp"))) - (add-after 'build-info-manual 'set-magit-version - (lambda _ - (make-file-writable "magit.el") - (emacs-substitute-variables "magit.el" - ("magit-version" ,version)))) - (add-after 'set-magit-version 'patch-exec-paths - (lambda* (#:key inputs #:allow-other-keys) - (let ((perl (assoc-ref inputs "perl"))) - (make-file-writable "magit-sequence.el") - (emacs-substitute-variables "magit-sequence.el" - ("magit-perl-executable" (string-append perl "/bin/perl")))))) - (add-before 'check 'configure-git - (lambda _ - ;; Otherwise some tests fail with error "unable to auto-detect - ;; email address". - (setenv "HOME" (getcwd)) - (invoke "git" "config" "--global" "user.name" "toto") - (invoke "git" "config" "--global" "user.email" - "toto@toto.com"))) - (add-after 'configure-git 'disable-tramp-test - (lambda _ - ;; There is an issue causing TRAMP to fail in the build - ;; environment. Setting the tramp-remote-shell parameter of - ;; the sudo-method to the file name of the shell didn't help. - (chdir "..") - (substitute* "t/magit-tests.el" - (("^\\(ert-deftest magit-toplevel:tramp.*" all) - (string-append all " (skip-unless nil)"))))) - (add-before 'install 'enter-lisp-directory - (lambda _ - (chdir "lisp")))))) + (list + #:emacs emacs-no-x ;module support is required + #:tests? #t + #:test-command #~(list "make" "test") + #:phases + #~(modify-phases %standard-phases + (add-after 'unpack 'build-info-manual + (lambda _ + (invoke "make" "info") + ;; Copy info files to the lisp directory, which acts as + ;; the root of the project for the emacs-build-system. + (for-each (lambda (f) + (install-file f "lisp")) + (find-files "Documentation" "\\.info$")) + (chdir "lisp"))) + (add-after 'build-info-manual 'set-magit-version + (lambda _ + (make-file-writable "magit.el") + (emacs-substitute-variables "magit.el" + ("magit-version" #$version)))) + (add-after 'set-magit-version 'patch-exec-paths + (lambda* (#:key inputs #:allow-other-keys) + (make-file-writable "magit-sequence.el") + (emacs-substitute-variables "magit-sequence.el" + ("magit-perl-executable" + (search-input-file inputs "/bin/perl"))))) + (add-before 'check 'configure-git + (lambda _ + ;; Otherwise some tests fail with error "unable to auto-detect + ;; email address". + (setenv "HOME" (getcwd)) + (invoke "git" "config" "--global" "user.name" "toto") + (invoke "git" "config" "--global" "user.email" + "toto@toto.com"))) + (add-after 'configure-git 'disable-tramp-test + (lambda _ + ;; There is an issue causing TRAMP to fail in the build + ;; environment. Setting the tramp-remote-shell parameter of + ;; the sudo-method to the file name of the shell didn't help. + (chdir "..") + (substitute* "t/magit-tests.el" + (("^\\(ert-deftest magit-toplevel:tramp.*" all) + (string-append all " (skip-unless nil)"))))) + (add-before 'install 'enter-lisp-directory + (lambda _ + (chdir "lisp")))))) (native-inputs (list texinfo)) (inputs -- 2.36.1 --=-=-= Content-Type: text/x-patch Content-Disposition: inline; filename=0002-gnu-emacs-magit-Drop-the-libgit-backend.patch >From 259d81469127306c4f345598f487122659a55c9c Mon Sep 17 00:00:00 2001 From: Zhu Zihao Date: Sat, 28 May 2022 10:10:31 +0800 Subject: [PATCH 2/2] gnu: emacs-magit: Drop the libgit backend. The libgit backend currently provides next to no features, but demands extra maintenance overhead. It is in fact not a usable backend; thus let's not use it. * gnu/packages/emacs-xyz.scm (emacs-magit)[arguments] : Remove this argument. : Exclude the magit-libgit.el and magit-libgit-pkg.el. [inputs]: Remove emacs-libgit. --- gnu/packages/emacs-xyz.scm | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm index a9dc687af4..21359882ad 100644 --- a/gnu/packages/emacs-xyz.scm +++ b/gnu/packages/emacs-xyz.scm @@ -870,9 +870,11 @@ (define-public emacs-magit (build-system emacs-build-system) (arguments (list - #:emacs emacs-no-x ;module support is required #:tests? #t #:test-command #~(list "make" "test") + #:exclude #~(cons* "magit-libgit.el" + "magit-libgit-pkg.el" + %default-exclude) #:phases #~(modify-phases %standard-phases (add-after 'unpack 'build-info-manual @@ -920,7 +922,7 @@ (define-public emacs-magit (inputs (list git perl)) (propagated-inputs - (list emacs-dash emacs-libgit emacs-transient emacs-with-editor)) + (list emacs-dash emacs-transient emacs-with-editor)) (home-page "https://magit.vc/") (synopsis "Emacs interface for the Git version control system") (description -- 2.36.1 --=-=-= Content-Type: text/plain -- Retrieve my PGP public key: gpg --recv-keys D47A9C8B2AE3905B563D9135BE42B352A9F6821F Zihao --=-=-=--