[bug#61008] [PATCH] gnu: emacs-smartparens: Fix package cl deprecation.

[bug#61008] [PATCH] gnu: emacs-smartparens: Fix package cl deprecation.

[Nicolas Graves via Guix-patches via]

* gnu/packages/emacs-xyz.scm (emacs-smartparens):
[arguments] Fix package cl deprecation.
---
 gnu/packages/emacs-xyz.scm | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index ed5c3bd71d..8bde2825f1 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -10575,6 +10575,17 @@ (define-public emacs-smartparens
        (sha256
         (base32 "0zij2f2rjjym98w68jkp10n1ckpfprlkk217c3fg16hz5nq4vnm6"))))
     (build-system emacs-build-system)
+    (arguments
+     (list
+      #:phases
+      #~(modify-phases %standard-phases
+          (add-after 'unpack 'fix-deprecated-cl-warning
+            (lambda _
+              (substitute* "smartparens.el"
+                ((string-append
+                  "\\(eval-when-compile \\(require 'cl\\)\\) "
+                  " ; for `lexical-let'")
+                 "")))))))
     (propagated-inputs
      (list emacs-dash emacs-markdown-mode))
     (home-page "https://github.com/Fuco1/smartparens")
-- 
2.39.1

[bug#61008] [PATCH] gnu: emacs-smartparens: Fix package cl deprecation.

[Nicolas Goaziou]

Hello,

Nicolas Graves via Guix-patches via <guix-patches@gnu.org> writes:

> * gnu/packages/emacs-xyz.scm (emacs-smartparens):
> [arguments] Fix package cl deprecation.
> ---
>  gnu/packages/emacs-xyz.scm | 11 +++++++++++
>  1 file changed, 11 insertions(+)
>
> diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
> index ed5c3bd71d..8bde2825f1 100644
> --- a/gnu/packages/emacs-xyz.scm
> +++ b/gnu/packages/emacs-xyz.scm
> @@ -10575,6 +10575,17 @@ (define-public emacs-smartparens
>         (sha256
>          (base32 "0zij2f2rjjym98w68jkp10n1ckpfprlkk217c3fg16hz5nq4vnm6"))))
>      (build-system emacs-build-system)
> +    (arguments
> +     (list
> +      #:phases
> +      #~(modify-phases %standard-phases
> +          (add-after 'unpack 'fix-deprecated-cl-warning
> +            (lambda _
> +              (substitute* "smartparens.el"
> +                ((string-append
> +                  "\\(eval-when-compile \\(require 'cl\\)\\) "
> +                  " ; for `lexical-let'")
> +                 "")))))))

AFAIU, we're fixing a mere warning. Why do you think it is worth
a patch?

Regards,
-- 
Nicolas Goaziou

[bug#61008] [PATCH] gnu: emacs-smartparens: Fix package cl deprecation.

[Nicolas Graves via Guix-patches via]

> AFAIU, we're fixing a mere warning.
Indeed, it's the case for both patches.

> Why do you think it is worth a patch?

I was just curious about the warning and looked for packages trigerring
it in my config, which are the two packages I sent.

Smartparens is regularly updated but doesn't make new releases, I
hesitated upadting it based on a commit, or use this patch.

Origami's original repo hasn't changed in 5 years, but it had a pull
request waiting for it.

Both are quite used, but still trigger the warning since their codebase
age. Their guix users have the warning and might enquire "for nothing
serious". But all-in-all, it is neither urgent nor important, you can
close both patches if you feel it's better without ;)

-- 
Best regards,
Nicolas Graves

[bug#61008] [PATCH] gnu: emacs-smartparens: Fix package cl deprecation.

[Nicolas Goaziou]

Nicolas Graves via Guix-patches via <guix-patches@gnu.org> writes:

> Smartparens is regularly updated but doesn't make new releases, I
> hesitated upadting it based on a commit, or use this patch.

Note that emacs-smartparens doesn't build with your patch
<https://qa.guix.gnu.org/issue/61008>.

It may be worth updating it to latest commit in that case.

[bug#61008] [PATCH v2] gnu: emacs-smartparens: Update to 1.11.0-0.7afd647.

[Nicolas Graves via Guix-patches via]

* gnu/packages/emacs-xyz.scm: Update to 1.11.0-0.7afd647.
---
 gnu/packages/emacs-xyz.scm | 42 ++++++++++++++++++++------------------
 1 file changed, 22 insertions(+), 20 deletions(-)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 5ebf8f0e1d..408ecd77f9 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -10562,26 +10562,28 @@ (define-public emacs-marginalia
     (license license:gpl3+)))
 
 (define-public emacs-smartparens
-  (package
-    (name "emacs-smartparens")
-    (version "1.11.0")
-    (source
-     (origin
-       (method git-fetch)
-       (uri (git-reference
-             (url "https://github.com/Fuco1/smartparens")
-             (commit version)))
-       (file-name (git-file-name name version))
-       (sha256
-        (base32 "0zij2f2rjjym98w68jkp10n1ckpfprlkk217c3fg16hz5nq4vnm6"))))
-    (build-system emacs-build-system)
-    (propagated-inputs
-     (list emacs-dash emacs-markdown-mode))
-    (home-page "https://github.com/Fuco1/smartparens")
-    (synopsis "Paredit-like insertion, wrapping and navigation with user
+  (let ((commit "7afd647395018a26633673ed92ce7a9cb3ccb8f2")
+        (revision "0"))
+    (package
+      (name "emacs-smartparens")
+      (version (git-version "1.11.0" revision commit))
+      (source
+       (origin
+         (method git-fetch)
+         (uri (git-reference
+               (url "https://github.com/Fuco1/smartparens")
+               (commit commit)))
+         (file-name (git-file-name name version))
+         (sha256
+          (base32 "0ihvz1w2ggjplk604cdd5alxcq2iy3nm95m7wb6vncrbyb6v0ad2"))))
+      (build-system emacs-build-system)
+      (propagated-inputs
+       (list emacs-dash emacs-markdown-mode))
+      (home-page "https://github.com/Fuco1/smartparens")
+      (synopsis "Paredit-like insertion, wrapping and navigation with user
 defined pairs")
-    (description
-     "Smartparens is a minor mode for Emacs that deals with parens pairs
+      (description
+       "Smartparens is a minor mode for Emacs that deals with parens pairs
 and tries to be smart about it.  It started as a unification effort to
 combine functionality of several existing packages in a single,
 compatible and extensible way to deal with parentheses, delimiters, tags
@@ -10589,7 +10591,7 @@ (define-public emacs-smartparens
 wrap-region, electric-pair-mode, paredit and others.  With the basic
 features found in other packages it also brings many improvements as
 well as completely new features.")
-    (license license:gpl3+)))
+      (license license:gpl3+))))
 
 (define-public emacs-dumbparens
   ;; There are no releases.
-- 
2.39.1

bug#61008: [PATCH v2] gnu: emacs-smartparens: Update to 1.11.0-0.7afd647.

[Christopher Baines]

[-- Attachment #1: Type: text/plain, Size: 375 bytes --]


Nicolas Graves via Guix-patches via <guix-patches@gnu.org> writes:

> * gnu/packages/emacs-xyz.scm: Update to 1.11.0-0.7afd647.
> ---
>  gnu/packages/emacs-xyz.scm | 42 ++++++++++++++++++++------------------
>  1 file changed, 22 insertions(+), 20 deletions(-)

This looks to have been merged as
f6286812ec932e282ef540bab0f614b13218d54b, so marking as done.

Thanks,

Chris

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 987 bytes --]