[PATCH] gnu: emacs-erc-status-sidebar: Update to 0.1-2.b589b742.

Michael Rohleder wrote on 29 Aug 2020 03:11

Recipients:(address . guix-patches@gnu.org)(name . Michael Rohleder)(address . mike@rohleder.de)

Message-ID:20200829011157.3382-1-mike@rohleder.de

* gnu/packages/emacs-xyz.scm (emacs-erc-status-sidebar): Update to 0.1-2.b589b742.[propagated-inputs]: removed emacs-seq.[license]: changed to gpl3+.---emacs-seq seems to make lots of trouble in emacs27 (max-lisp-eval-depth).I think, we should remove that dep everywhere (it's provided by emacs) andmark it as deprecated or also remove it.
 gnu/packages/emacs-xyz.scm | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-)

Toggle diff (37 lines)diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scmindex 0755926ffe..7a6eb2e878 100644--- a/gnu/packages/emacs-xyz.scm+++ b/gnu/packages/emacs-xyz.scm@@ -3486,8 +3486,8 @@ IRC bouncer with ERC.")     (license license:expat)))  (define-public emacs-erc-status-sidebar-  (let ((commit "ea4189a1dbfe60117359c36e681ad7c389e2968c")-        (revision "1"))+  (let ((commit "b589b74245591257c9e666bd5cd25812e04b7a34")+        (revision "2"))     (package       (name "emacs-erc-status-sidebar")       (version (git-version "0.1" revision commit))@@ -3499,9 +3499,8 @@ IRC bouncer with ERC.")                (commit commit)))          (file-name (git-file-name name version))          (sha256-          (base32 "1hwlhzgx03z8891sblz56zdp8zj0izh72kxykgcnz5rrkyc3vfi3"))))+          (base32 "08fvr5ba6gb3y52sypbdj236nxr5rl95zgz1b120scz6xal7mysy"))))       (build-system emacs-build-system)-      (propagated-inputs `(("emacs-seq" ,emacs-seq)))       (home-page "https://github.com/drewbarbs/erc-status-sidebar")       (synopsis "Hexchat-like activity overview for ERC channels")       (description@@ -3509,7 +3508,7 @@ IRC bouncer with ERC.") ERC, an Emacs client for IRC (Internet Relay Chat).  It relies on the @code{erc-track} module, and displays all the same information @code{erc-track} does in the mode line, but in an alternative format.")-      (license license:expat))))+      (license license:gpl3+))))  (define-public emacs-shut-up   (package-- 2.28.0

Brett Gilio wrote on 29 Aug 2020 03:45

Recipients:(name . Michael Rohleder)(address . mike@rohleder.de)(address . 43096@debbugs.gnu.org)

Message-ID:87h7sm6x7v.fsf@gnu.org

Michael Rohleder <mike@rohleder.de> writes:

Toggle quote (47 lines)> * gnu/packages/emacs-xyz.scm (emacs-erc-status-sidebar): Update to 0.1-2.b589b742.> [propagated-inputs]: removed emacs-seq.> [license]: changed to gpl3+.> ---> emacs-seq seems to make lots of trouble in emacs27 (max-lisp-eval-depth).> I think, we should remove that dep everywhere (it's provided by emacs) and> mark it as deprecated or also remove it.>>  gnu/packages/emacs-xyz.scm | 9 ++++----->  1 file changed, 4 insertions(+), 5 deletions(-)>> diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm> index 0755926ffe..7a6eb2e878 100644> --- a/gnu/packages/emacs-xyz.scm> +++ b/gnu/packages/emacs-xyz.scm> @@ -3486,8 +3486,8 @@ IRC bouncer with ERC.")>      (license license:expat)))>  >  (define-public emacs-erc-status-sidebar> -  (let ((commit "ea4189a1dbfe60117359c36e681ad7c389e2968c")> -        (revision "1"))> +  (let ((commit "b589b74245591257c9e666bd5cd25812e04b7a34")> +        (revision "2"))>      (package>        (name "emacs-erc-status-sidebar")>        (version (git-version "0.1" revision commit))> @@ -3499,9 +3499,8 @@ IRC bouncer with ERC.")>                 (commit commit)))>           (file-name (git-file-name name version))>           (sha256> -          (base32 "1hwlhzgx03z8891sblz56zdp8zj0izh72kxykgcnz5rrkyc3vfi3"))))> +          (base32 "08fvr5ba6gb3y52sypbdj236nxr5rl95zgz1b120scz6xal7mysy"))))>        (build-system emacs-build-system)> -      (propagated-inputs `(("emacs-seq" ,emacs-seq)))>        (home-page "https://github.com/drewbarbs/erc-status-sidebar")>        (synopsis "Hexchat-like activity overview for ERC channels")>        (description> @@ -3509,7 +3508,7 @@ IRC bouncer with ERC.")>  ERC, an Emacs client for IRC (Internet Relay Chat).  It relies on the>  @code{erc-track} module, and displays all the same information>  @code{erc-track} does in the mode line, but in an alternative format.")> -      (license license:expat))))> +      (license license:gpl3+))))>  >  (define-public emacs-shut-up>    (package

I might be mistaken, but I think the license change belongs in aseparate patch.
Brett Gilio

Tobias Geerinckx-Rice wrote on 29 Aug 2020 03:51

Recipients:(name . Brett Gilio)(address . brettg@gnu.org)

Message-ID:87ft86kynv.fsf@nckx

Brett Gilio 写道：

Toggle quote (9 lines)

>> - (license license:expat))))>> + (license license:gpl3+))))>> >> (define-public emacs-shut-up>> (package>> I might be mistaken, but I think the license change belongs in a> separate patch.

Not if the licence actually changed between versions.
If it's correcting a previously incorrect licence in Guix, then indeed it does.
Kind regards,
T G-R

Brett Gilio wrote on 29 Aug 2020 03:58

Recipients:(name . Tobias Geerinckx-Rice)(address . me@tobias.gr)

Message-ID:87y2ly5i33.fsf@gnu.org

Tobias Geerinckx-Rice <me@tobias.gr> writes:

Toggle quote (18 lines)> Brett Gilio 写道：>>> -      (license license:expat))))>>> +      (license license:gpl3+))))>>>   (define-public emacs-shut-up>>>    (package>>>> I might be mistaken, but I think the license change belongs in a>> separate patch.>> Not if the licence actually changed between versions.>> If it's correcting a previously incorrect licence in Guix, then indeed> it does.>> Kind regards,>> T G-R

It's kind of a weird case where it is both? The LICENSE file has alwaysbeen GPL3, but the code header erroneously stated it as expat-style.
https://github.com/drewbarbs/erc-status-sidebar/commit/b589b74245591257c9e666bd5cd25812e04b7a34
I kind of lean on the side of a separate patch since the license neverreally changed, but was an error both upstream and on our end as well.
Brett Gilio

Brett Gilio wrote on 29 Aug 2020 03:59

Recipients:(name . Tobias Geerinckx-Rice)(address . me@tobias.gr)

Message-ID:87sgc65i01.fsf@gnu.org

Brett Gilio <brettg@gnu.org> writes:

Toggle quote (30 lines)> Tobias Geerinckx-Rice <me@tobias.gr> writes:>>> Brett Gilio 写道：>>>> -      (license license:expat))))>>>> +      (license license:gpl3+))))>>>>   (define-public emacs-shut-up>>>>    (package>>>>>> I might be mistaken, but I think the license change belongs in a>>> separate patch.>>>> Not if the licence actually changed between versions.>>>> If it's correcting a previously incorrect licence in Guix, then indeed>> it does.>>>> Kind regards,>>>> T G-R>> It's kind of a weird case where it is both? The LICENSE file has always> been GPL3, but the code header erroneously stated it as expat-style.>> https://github.com/drewbarbs/erc-status-sidebar/commit/b589b74245591257c9e666bd5cd25812e04b7a34>> I kind of lean on the side of a separate patch since the license never> really changed, but was an error both upstream and on our end as well.>> Brett Gilio

Actually, the change of commits may not be necessary at all. There is nocode changes between the commits, and the license file is intact.

Michael Rohleder wrote on 30 Aug 2020 03:29

Recipients:(name . Brett Gilio)(address . brettg@gnu.org)

Message-ID:87y2lw3oqb.fsf@rohleder.de

Hey Brett, hey Tobias,
Brett Gilio <brettg@gnu.org> writes:

Toggle quote (14 lines)>> It's kind of a weird case where it is both? The LICENSE file has always>> been GPL3, but the code header erroneously stated it as expat-style.>>>> https://github.com/drewbarbs/erc-status-sidebar/commit/b589b74245591257c9e666bd5cd25812e04b7a34>>>> I kind of lean on the side of a separate patch since the license never>> really changed, but was an error both upstream and on our end as well.>>>> Brett Gilio>> Actually, the change of commits may not be necessary at all. There is no> code changes between the commits, and the license file is intact.>

Ok, here are 2 patches, one for the license and one for removingemacs-seq:

From 340699d5ceb2db40824a403399556536f2e07e23 Mon Sep 17 00:00:00 2001
* gnu/packages/emacs-xyz.scm (emacs-erc-status-sidebar)[license]: Set to GPL3+.--- gnu/packages/emacs-xyz.scm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

Toggle diff (15 lines)diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scmindex ff789ffedf..9975fc8366 100644--- a/gnu/packages/emacs-xyz.scm+++ b/gnu/packages/emacs-xyz.scm@@ -3509,7 +3509,7 @@ IRC bouncer with ERC.") ERC, an Emacs client for IRC (Internet Relay Chat).  It relies on the @code{erc-track} module, and displays all the same information @code{erc-track} does in the mode line, but in an alternative format.")-      (license license:expat))))+      (license license:gpl3+))))  (define-public emacs-shut-up   (package-- 2.28.0

From 028021a0654c3a82d951ec991d63d392c421324d Mon Sep 17 00:00:00 2001
* gnu/packages/emacs-xyz.scm (emacs-erc-status-sidebar)[propagated-inputs]: remove emacs-seq.
This package works well with the seq library that is provided with emacs.Current emacs-seq (0.20) is broken for emacs27. A fix seems on it'sway: https://lists.gnu.org/archive/html/emacs-devel/2020-08/msg00996.html--- gnu/packages/emacs-xyz.scm | 1 - 1 file changed, 1 deletion(-)

Toggle diff (14 lines)diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scmindex 9975fc8366..a2427112aa 100644--- a/gnu/packages/emacs-xyz.scm+++ b/gnu/packages/emacs-xyz.scm@@ -3501,7 +3501,6 @@ IRC bouncer with ERC.")          (sha256           (base32 "1hwlhzgx03z8891sblz56zdp8zj0izh72kxykgcnz5rrkyc3vfi3"))))       (build-system emacs-build-system)-      (propagated-inputs `(("emacs-seq" ,emacs-seq)))       (home-page "https://github.com/drewbarbs/erc-status-sidebar")       (synopsis "Hexchat-like activity overview for ERC channels")       (description-- 2.28.0

-- "How should I know if it works? That's what beta testers are for. I onlycoded it."(Attributed to Linus Torvalds, somewhere in a posting)

Nicolas Goaziou wrote on 30 Aug 2020 11:22

Recipients:(name . Michael Rohleder)(address . mike@rohleder.de)(address . 43096@debbugs.gnu.org)

Message-ID:87d038lc7l.fsf@nicolasgoaziou.fr

Hello,
Michael Rohleder <mike@rohleder.de> writes:

Toggle quote (3 lines)

> I think, we should remove that dep everywhere (it's provided by emacs) and> mark it as deprecated or also remove it.

I agree that the dependency should be removed from Emacs packages. Theemacs-seq package can be deprecated, indeed.
Regards,-- Nicolas Goaziou

Michael Rohleder wrote on 1 Sep 2020 05:48

Recipients:(name . Nicolas Goaziou)(address . mail@nicolasgoaziou.fr)(address . 43096@debbugs.gnu.org)

Message-ID:87imcycg2e.fsf@rohleder.de

Nicolas Goaziou <mail@nicolasgoaziou.fr> writes:

Toggle quote (6 lines)

>> I think, we should remove that dep everywhere (it's provided by emacs) and>> mark it as deprecated or also remove it.>> I agree that the dependency should be removed from Emacs packages. The> emacs-seq package can be deprecated, indeed.

Pierre made a patch for bug #43138:https://lists.gnu.org/archive/html/bug-guix/2020-08/msg00311.html
-- There are two reasons why we dont trust people.First - we dont know them.Second - we know them.

Ludovic Courtès wrote on 18 Dec 2020 14:18

Recipients:(name . Michael Rohleder)(address . mike@rohleder.de)

Message-ID:87wnxfckwn.fsf@gnu.org

Hi!
Michael Rohleder <mike@rohleder.de> skribis:

Toggle quote (7 lines)

> From 340699d5ceb2db40824a403399556536f2e07e23 Mon Sep 17 00:00:00 2001> From: Michael Rohleder <mike@rohleder.de>> Date: Sun, 30 Aug 2020 02:48:32 +0200> Subject: [PATCH] gnu: emacs-erc-status-sidebar: Correct license.>> * gnu/packages/emacs-xyz.scm (emacs-erc-status-sidebar)[license]: Set to GPL3+.

Applied, thanks!
The emacs-seq bit is already done, as discussed on IRC.
Ludo’.

Closed

Your comment

Commenting via the web interface is currently disabled.

To comment on this conversation send email to 43096@debbugs.gnu.org

is:open	open issues
is:done	closed issues
submitter:<who>	search issue submitter
author:<who>	search by message author
date:yesterday..now	search by issue date
mdate:3m..2d	search by message date