GNU bug report logs - #46928
[PATCH] gnu: Add emacs-stig-paren.

Previous Next

To reply to this bug, email your comments to 46928 AT debbugs.gnu.org.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Stefan Reichör <stefan <at> xsteve.at>
To: guix-patches <at> gnu.org
Cc: Stefan Reichör <stefan <at> xsteve.at>
Subject: [PATCH] gnu: Add emacs-stig-paren.
Date: Thu,  4 Mar 2021 20:40:25 +0100

* gnu/packages/emacs-xyz.scm (emacs-stig-paren): New variable.
---
 gnu/packages/emacs-xyz.scm | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 783f7790e9..85a612296c 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -93,6 +93,7 @@
 ;;; Copyright © 2021 Alexandr Vityazev <avityazew <at> gmail.com>
 ;;; Copyright © 2021 Yurii Kholodkov <urist.mckorobochka <at> gmail.com>
 ;;; Copyright © 2021 Alexey Abramov <levenson <at> mmer.org>
+;;; Copyright © 2021 Stefan Reichör <stefan <at> xsteve.at>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -385,6 +386,25 @@ for those who may want transient periods of unbalanced parentheses, such as
 when typing parentheses directly or commenting out code line by line.")
     (license license:gpl3+)))
 
+(define-public emacs-stig-paren
+  (package
+    (name "emacs-stig-paren")
+    (version "1.22")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "http://nicolas.francois.free.fr/"
+                                  "fichiers/emacs/stig-paren.el"))
+              (sha256
+               (base32
+                "1k4rd4k53c671akssza91p0d7psxagizc74nka3hi1pjkybmw6mn"))))
+    (build-system emacs-build-system)
+    (home-page "http://nicolas.francois.free.fr/fichiers/emacs/stig-paren.el")
+    (synopsis "Highlight matching parens for Emacs")
+    (description
+     "This package highlights matching parens (or whole sexps) for easier
+editing of source code, particularly lisp source code.")
+    (license license:gpl2+)))
+
 (define-public emacs-project
   (package
     (name "emacs-project")
-- 
2.25.1

Message #8 received at 46928 <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
To: Stefan Reichör <stefan <at> xsteve.at>
Cc: 46928 <at> debbugs.gnu.org
Subject: Re: [bug#46928] [PATCH] gnu: Add emacs-stig-paren.
Date: Fri, 05 Mar 2021 17:12:22 +0100

Hello,

Stefan Reichör <stefan <at> xsteve.at> writes:

> * gnu/packages/emacs-xyz.scm (emacs-stig-paren): New variable.

Thank you.

I think there is an issue with this library: old revisions are not kept
around and file name is not versioned. It is unlikely it is updated
anytime soon, but if that ever happens, package definition will break.

Considering the features it provides has been part of Emacs for a long
time now (IIUC), do you think it is worth including?

Regards,
-- 
Nicolas Goaziou

Message #11 received at 46928 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Reichör <stefan <at> xsteve.at>
To: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
Cc: 46928 <at> debbugs.gnu.org
Subject: Re: [bug#46928] [PATCH] gnu: Add emacs-stig-paren.
Date: Fri, 05 Mar 2021 20:59:57 +0100

Hi Nicolas!

> Hello,
>
> Stefan Reichör <stefan <at> xsteve.at> writes:
>
>> * gnu/packages/emacs-xyz.scm (emacs-stig-paren): New variable.
>
> Thank you.
>
> I think there is an issue with this library: old revisions are not kept
> around and file name is not versioned. It is unlikely it is updated
> anytime soon, but if that ever happens, package definition will break.
>
> Considering the features it provides has been part of Emacs for a long
> time now (IIUC), do you think it is worth including?

Thanks for your hint. I use stig-paren.el since more than 20 years. I
guess it worked better than the built-in show-paren-mode these days.

I thought it would be great to have all my emacs packages as part of guix.

Now I tried show-paren-mode again.
Here is what I came up with as stig-paren.el replacement:
,----
| (setq show-paren-style 'expression)
| (face-spec-set 'show-paren-match-expression '((t (:inherit show-paren-match :background "pale green"))))
| (show-paren-mode 1)
`----

That way my .emacs has grown a bit, but I no longer need stig-paren.el ;-)

You are correct, there is no need to add stig-paren.el to guix.

By the way: thanks for applying my other patches.

Thanks
  Stefan.

Message #14 received at 46928 <at> debbugs.gnu.org (full text, mbox):

From: zimoun <zimon.toutoune <at> gmail.com>
To: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
Cc: 46928 <at> debbugs.gnu.org, Stefan Reichör <stefan <at> xsteve.at>
Subject: Re: [bug#46928] [PATCH] gnu: Add emacs-stig-paren.
Date: Fri, 5 Mar 2021 21:34:43 +0100

Hi Nicolas,

On Fri, 5 Mar 2021 at 17:13, Nicolas Goaziou <mail <at> nicolasgoaziou.fr> wrote:

> I think there is an issue with this library: old revisions are not kept
> around and file name is not versioned. It is unlikely it is updated
> anytime soon, but if that ever happens, package definition will break.

I think it is the same story as
<http://issues.guix.gnu.org/issue/46849>.  A mechanism to fallback to
an archiver (say SWH) seems missing.

BTW, I have no opinion about the inclusion of the package at hand. :-)

Cheers,
simon

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.