GNU bug report logs - #61513
emacs-next@29.0.60 self-reports as version 30.0.50

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 61513 in the body.
You can then email your comments to 61513 AT debbugs.gnu.org in the normal way.

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

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

From: Jelle Licht <jlicht <at> fsfe.org>
To: bug-guix <at> gnu.org
Subject: emacs-next <at> 29.0.60 self-reports as version 30.0.50
Date: Tue, 14 Feb 2023 20:34:45 +0000

Hi guix,

Since commit 6f0c905324 on master, the version of our emacs-next package
was bumped to 29.0.60(-0.ac7ec87).

Yet:
--8<---------------cut here---------------start------------->8---
jlicht <at> revint ~$ guix shell --pure emacs-next -- emacs --version
GNU Emacs 30.0.50
--8<---------------cut here---------------end--------------->8---

Is this some peculiarity of the Emacs's development cycle, or did we end
up having a "emacs-next-next" package?

Kind regards,
- Jelle

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

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: 61513 <at> debbugs.gnu.org
Cc: Jelle Licht <jlicht <at> fsfe.org>
Subject: [PATCH 2/2] gnu: emacs-next: Update to 29.0.60-1.f1f571e.
Date: Thu, 16 Feb 2023 21:45:24 +0100

* gnu/packages/emacs.scm (emacs-next): Update to 29.0.60-1.f1f571e.
---
Note: Haven't tried to build the commit yet, but the hash checks out.

 gnu/packages/emacs.scm | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index e230fdc84d..51ce4e481b 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -391,8 +391,8 @@ (define* (emacs-byte-compile-directory dir)
     (license license:gpl3+)))
 
 (define-public emacs-next
-  (let ((commit "ac7ec87a7a0db887e4ae7fe9005aea517958b778")
-        (revision "0"))
+  (let ((commit "f1f571e72ae10285762d3a941e56f7c4048272af")
+        (revision "1"))
     (package
       (inherit emacs)
       (name "emacs-next")
@@ -411,7 +411,7 @@ (define-public emacs-next
                                   "emacs-native-comp-driver-options.patch"))
          (sha256
           (base32
-           "1akq6dbllwwqwx21wnwnv6aax1nsi2ypbd7j3i79sw62s3gf399z"))))
+           "1rildbxq53yvc2rllg2qccgxzbbnr6qbija0lyqacsy8dlzaysch"))))
       (inputs
        (modify-inputs (package-inputs emacs)
          (prepend sqlite)))
-- 
2.39.1

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

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: 61513 <at> debbugs.gnu.org
Cc: Jelle Licht <jlicht <at> fsfe.org>
Subject: [PATCH 1/2] gnu: Add emacs-next-tree-sitter.
Date: Thu, 16 Feb 2023 21:36:52 +0100

* gnu/packages/emacs.scm (emacs-next-tree-sitter): New variable.
(emacs-next): Remove tree sitter support.
---
 gnu/packages/emacs.scm | 29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 953aad961c..e230fdc84d 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -397,6 +397,35 @@ (define-public emacs-next
       (inherit emacs)
       (name "emacs-next")
       (version (git-version "29.0.60" revision commit))
+      (source
+       (origin
+         (inherit (package-source emacs))
+         (method git-fetch)
+         (uri (git-reference
+               (url "https://git.savannah.gnu.org/git/emacs.git/")
+               (commit commit)))
+         (file-name (git-file-name name version))
+         ;; emacs-source-date-epoch.patch is no longer necessary
+         (patches (search-patches "emacs-exec-path.patch"
+                                  "emacs-fix-scheme-indent-function.patch"
+                                  "emacs-native-comp-driver-options.patch"))
+         (sha256
+          (base32
+           "1akq6dbllwwqwx21wnwnv6aax1nsi2ypbd7j3i79sw62s3gf399z"))))
+      (inputs
+       (modify-inputs (package-inputs emacs)
+         (prepend sqlite)))
+      (native-inputs
+       (modify-inputs (package-native-inputs emacs)
+         (prepend autoconf))))))
+
+(define-public emacs-next-tree-sitter
+  (let ((commit "ac7ec87a7a0db887e4ae7fe9005aea517958b778")
+        (revision "0"))
+    (package
+      (inherit emacs)
+      (name "emacs-next-tree-sitter")
+      (version (git-version "30.0.50" revision commit))
       (source
        (origin
          (inherit (package-source emacs))

base-commit: 312f1f41d3f3f3e5d2c36ff46920c6dce1c21a17
-- 
2.39.1

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

From: Simon Tournier <zimon.toutoune <at> gmail.com>
To: Liliana Marie Prikler <liliana.prikler <at> gmail.com>, 61513 <at> debbugs.gnu.org
Cc: Jelle Licht <jlicht <at> fsfe.org>
Subject: Re: bug#61513: [PATCH 1/2] gnu: Add emacs-next-tree-sitter.
Date: Mon, 20 Feb 2023 11:33:23 +0100

Hi,

On jeu., 16 févr. 2023 at 21:36, Liliana Marie Prikler <liliana.prikler <at> gmail.com> wrote:
> * gnu/packages/emacs.scm (emacs-next-tree-sitter): New variable.
> (emacs-next): Remove tree sitter support.

Naive question: why not squash the 2 patches?  Adding the new variable
and fix the version of emacs-next.

Cheers,
simon

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

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: Simon Tournier <zimon.toutoune <at> gmail.com>, 61513 <at> debbugs.gnu.org
Cc: Jelle Licht <jlicht <at> fsfe.org>
Subject: Re: bug#61513: [PATCH 1/2] gnu: Add emacs-next-tree-sitter.
Date: Mon, 20 Feb 2023 20:21:17 +0100

Am Montag, dem 20.02.2023 um 11:33 +0100 schrieb Simon Tournier:
> Hi,
> 
> On jeu., 16 févr. 2023 at 21:36, Liliana Marie Prikler
> <liliana.prikler <at> gmail.com> wrote:
> > * gnu/packages/emacs.scm (emacs-next-tree-sitter): New variable.
> > (emacs-next): Remove tree sitter support.
> 
> Naive question: why not squash the 2 patches?  Adding the new
> variable and fix the version of emacs-next.
Because the next patch "upgrades" emacs-next to the then HEAD of the
emacs-29 branch, which wasn't hitherto packaged in Guix. 
Alternatively, one could reset it to an earlier version in the first
commit, but we have a(n informal) policy that discourages downgrades.

Cheers

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

From: Simon Tournier <zimon.toutoune <at> gmail.com>
To: Liliana Marie Prikler <liliana.prikler <at> gmail.com>, 61513 <at> debbugs.gnu.org
Cc: Jelle Licht <jlicht <at> fsfe.org>
Subject: Re: bug#61513: [PATCH 1/2] gnu: Add emacs-next-tree-sitter.
Date: Tue, 21 Feb 2023 13:38:01 +0100

Hi,

On Mon, 20 Feb 2023 at 20:21, Liliana Marie Prikler <liliana.prikler <at> gmail.com> wrote:

>> Naive question: why not squash the 2 patches?  Adding the new
>> variable and fix the version of emacs-next.
>
> Because the next patch "upgrades" emacs-next to the then HEAD of the
> emacs-29 branch, which wasn't hitherto packaged in Guix.
> Alternatively, one could reset it to an earlier version in the first
> commit, but we have a(n informal) policy that discourages downgrades.

Ah indeed, thanks for the explanations.

IIUC, the error seems Guix commit
6f0c9053244d6b4cfc4130c963a15dbf83cbcdac in the first place, introducing
Emacs commit ac7ec87a7a0db887e4ae7fe9005aea517958b778 which is not from
the Emacs 29 series but from the Emacs 30 series.

Well, I do not think if we can speak about downgrade here. :-)  Anyway.

It LGTM.


Cheers,
simon

Message #25 received at 61513-done <at> debbugs.gnu.org (full text, mbox):

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: Simon Tournier <zimon.toutoune <at> gmail.com>, 61513-done <at> debbugs.gnu.org
Cc: Jelle Licht <jlicht <at> fsfe.org>
Subject: Re: bug#61513: [PATCH 1/2] gnu: Add emacs-next-tree-sitter.
Date: Sun, 26 Feb 2023 09:25:30 +0100

Am Dienstag, dem 21.02.2023 um 13:38 +0100 schrieb Simon Tournier:
> It LGTM.
And it pushed now :)

Previous Next

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