unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#74931] [PATCH] gnu: proof-general: Update to 4.5-1.d668946.
@ 2024-12-17 19:32 Morgan Smith
  2024-12-17 20:27 ` Suhail Singh
  0 siblings, 1 reply; 3+ messages in thread
From: Morgan Smith @ 2024-12-17 19:32 UTC (permalink / raw)
  To: 74931; +Cc: Morgan Smith, Julien Lepiller, pukkamustard

* gnu/packages/coq.scm (proof-general): Update to 4.5-1.d668946.
[phases] Remove unneeded phases and fix shebang fix.

Change-Id: I5ce2775cf18775737853d76b5a50d461aa9d14e6
---

Currently the coqtags program included in proof-general does not work as the
shebang is patched incorrectly.

This patch fixes that and updates proof-general.


 gnu/packages/coq.scm | 24 ++++--------------------
 1 file changed, 4 insertions(+), 20 deletions(-)

diff --git a/gnu/packages/coq.scm b/gnu/packages/coq.scm
index ea0868f226..e6df4bf98f 100644
--- a/gnu/packages/coq.scm
+++ b/gnu/packages/coq.scm
@@ -135,7 +135,7 @@ (define-public proof-general
   ;; The latest release is from 2022 and there has been more than 100 commits
   ;; since then.
   ;; Commit from 2024-04-29.
-  (let ((commit "cb23709ad0c9a9ca0ee48b3ee73c29caea243b98")
+  (let ((commit "d6689469298b4140dc1f0f8b0ff7e8f937041ffe")
         (revision "1"))
     (package
       (name "proof-general")
@@ -148,7 +148,7 @@ (define-public proof-general
                 (file-name (git-file-name name version))
                 (sha256
                  (base32
-                  "1spd8rz95s1x91i4lbbb6zabb8014fihx6ai6pgad1nwyr0y9bir"))))
+                  "1d12z41rn5nh15qj4sf0w8xrbd9djxlrz0r6g38fiq63i7krbm4x"))))
       (build-system gnu-build-system)
       (native-inputs
        `(("emacs" ,emacs-minimal)
@@ -168,30 +168,14 @@ (define-public proof-general
            #:phases
            (modify-phases %standard-phases
              (delete 'configure)
-             (add-after 'unpack 'disable-byte-compile-error-on-warn
-               (lambda _
-                 (substitute* "Makefile"
-                   (("\\(setq byte-compile-error-on-warn t\\)")
-                    "(setq byte-compile-error-on-warn nil)"))))
-             (add-after 'unpack 'modify-readme-name
-               ;; The README file is called "README.md", but the Make variable
-               ;; "DOC_FILES" still refers to "README".
-               (lambda _
-                 (substitute* "Makefile"
-                   (("README") "README.md"))))
              (add-after 'unpack 'patch-hardcoded-paths
                (lambda _
                  (substitute* "Makefile"
                    (("/sbin/install-info") "install-info"))))
-             (add-after 'unpack 'remove-which
+             (add-after 'unpack 'do-not-patch-script-shebangs
                (lambda _
                  (substitute* "Makefile"
-                   (("`which perl`") "perl")
-                   (("`which bash`") "bash"))))
-             (add-after 'unpack 'clean
-               (lambda _
-                 ;; Delete the pre-compiled elc files for Emacs 23.
-                 (invoke "make" "clean")))
+                   (("install-bin: scripts") "install-bin:"))))
              (add-after 'install 'install-doc
                (lambda* (#:key make-flags #:allow-other-keys)
                  ;; XXX FIXME avoid building/installing pdf files,

base-commit: 8e71580aae17f0c84896519d63246ab4bbb77fc4
--
2.46.0





^ permalink raw reply related	[flat|nested] 3+ messages in thread

* [bug#74931] [PATCH] gnu: proof-general: Update to 4.5-1.d668946.
  2024-12-17 19:32 [bug#74931] [PATCH] gnu: proof-general: Update to 4.5-1.d668946 Morgan Smith
@ 2024-12-17 20:27 ` Suhail Singh
  2024-12-18  4:07   ` [bug#74931] [PATCH v2] gnu: proof-general: Update to 4.5-2.d668946 Morgan Smith
  0 siblings, 1 reply; 3+ messages in thread
From: Suhail Singh @ 2024-12-17 20:27 UTC (permalink / raw)
  To: Morgan Smith; +Cc: pukkamustard, 74931, Julien Lepiller

Morgan Smith <Morgan.J.Smith@outlook.com> writes:

> * gnu/packages/coq.scm (proof-general): Update to 4.5-1.d668946.
> [phases] Remove unneeded phases and fix shebang fix.
>
> ...
>
> diff --git a/gnu/packages/coq.scm b/gnu/packages/coq.scm
> index ea0868f226..e6df4bf98f 100644
> --- a/gnu/packages/coq.scm
> +++ b/gnu/packages/coq.scm
> @@ -135,7 +135,7 @@ (define-public proof-general
>    ;; The latest release is from 2022 and there has been more than 100 commits
>    ;; since then.
>    ;; Commit from 2024-04-29.
> -  (let ((commit "cb23709ad0c9a9ca0ee48b3ee73c29caea243b98")
> +  (let ((commit "d6689469298b4140dc1f0f8b0ff7e8f937041ffe")
>          (revision "1"))

Shouldn't the revision be incremented as well?

-- 
Suhail




^ permalink raw reply	[flat|nested] 3+ messages in thread

* [bug#74931] [PATCH v2] gnu: proof-general: Update to 4.5-2.d668946.
  2024-12-17 20:27 ` Suhail Singh
@ 2024-12-18  4:07   ` Morgan Smith
  0 siblings, 0 replies; 3+ messages in thread
From: Morgan Smith @ 2024-12-18  4:07 UTC (permalink / raw)
  To: suhailsingh247
  Cc: pukkamustard, 74931, julien, Morgan Smith, Julien Lepiller,
	pukkamustard

* gnu/packages/coq.scm (proof-general): Update to 4.5-2.d668946.
[phases] Remove unneeded phases and fix shebang fix.

Change-Id: I5ce2775cf18775737853d76b5a50d461aa9d14e6
---

Yes I should bump the revision.  Thanks for catching that!

 gnu/packages/coq.scm | 26 +++++---------------------
 1 file changed, 5 insertions(+), 21 deletions(-)

diff --git a/gnu/packages/coq.scm b/gnu/packages/coq.scm
index ea0868f226..3ef91ad78a 100644
--- a/gnu/packages/coq.scm
+++ b/gnu/packages/coq.scm
@@ -135,8 +135,8 @@ (define-public proof-general
   ;; The latest release is from 2022 and there has been more than 100 commits
   ;; since then.
   ;; Commit from 2024-04-29.
-  (let ((commit "cb23709ad0c9a9ca0ee48b3ee73c29caea243b98")
-        (revision "1"))
+  (let ((commit "d6689469298b4140dc1f0f8b0ff7e8f937041ffe")
+        (revision "2"))
     (package
       (name "proof-general")
       (version (git-version "4.5" revision commit))
@@ -148,7 +148,7 @@ (define-public proof-general
                 (file-name (git-file-name name version))
                 (sha256
                  (base32
-                  "1spd8rz95s1x91i4lbbb6zabb8014fihx6ai6pgad1nwyr0y9bir"))))
+                  "1d12z41rn5nh15qj4sf0w8xrbd9djxlrz0r6g38fiq63i7krbm4x"))))
       (build-system gnu-build-system)
       (native-inputs
        `(("emacs" ,emacs-minimal)
@@ -168,30 +168,14 @@ (define-public proof-general
            #:phases
            (modify-phases %standard-phases
              (delete 'configure)
-             (add-after 'unpack 'disable-byte-compile-error-on-warn
-               (lambda _
-                 (substitute* "Makefile"
-                   (("\\(setq byte-compile-error-on-warn t\\)")
-                    "(setq byte-compile-error-on-warn nil)"))))
-             (add-after 'unpack 'modify-readme-name
-               ;; The README file is called "README.md", but the Make variable
-               ;; "DOC_FILES" still refers to "README".
-               (lambda _
-                 (substitute* "Makefile"
-                   (("README") "README.md"))))
              (add-after 'unpack 'patch-hardcoded-paths
                (lambda _
                  (substitute* "Makefile"
                    (("/sbin/install-info") "install-info"))))
-             (add-after 'unpack 'remove-which
+             (add-after 'unpack 'do-not-patch-script-shebangs
                (lambda _
                  (substitute* "Makefile"
-                   (("`which perl`") "perl")
-                   (("`which bash`") "bash"))))
-             (add-after 'unpack 'clean
-               (lambda _
-                 ;; Delete the pre-compiled elc files for Emacs 23.
-                 (invoke "make" "clean")))
+                   (("install-bin: scripts") "install-bin:"))))
              (add-after 'install 'install-doc
                (lambda* (#:key make-flags #:allow-other-keys)
                  ;; XXX FIXME avoid building/installing pdf files,

base-commit: 8e71580aae17f0c84896519d63246ab4bbb77fc4
--
2.46.0





^ permalink raw reply related	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-12-18  4:09 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-12-17 19:32 [bug#74931] [PATCH] gnu: proof-general: Update to 4.5-1.d668946 Morgan Smith
2024-12-17 20:27 ` Suhail Singh
2024-12-18  4:07   ` [bug#74931] [PATCH v2] gnu: proof-general: Update to 4.5-2.d668946 Morgan Smith

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).