unofficial mirror of bug-guix@gnu.org 
 help / color / mirror / code / Atom feed
From: divoplade <d@divoplade.fr>
To: 43093@debbugs.gnu.org
Subject: bug#43093: emacs-ess is broken by emacs 27.1
Date: Mon, 07 Sep 2020 23:43:59 +0200	[thread overview]
Message-ID: <10bf3da9dac586a986c4a098777aefeba3b5f5b5.camel@divoplade.fr> (raw)
In-Reply-To: <a08c9907f6354c9ed13b6676e558ad96738b2789.camel@divoplade.fr>

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

Le lundi 07 septembre 2020 à 23:19 +0200, divoplade a écrit :
> Sorry, I don't know how to make a patch.

Trying: does it work like that?

[-- Attachment #2: 0001-Update-emacs-ess-to-a-more-recent-snapshot.patch --]
[-- Type: text/x-patch, Size: 3937 bytes --]

From 06cb64632af8982054eb85ab4bbe5c43e86ab653 Mon Sep 17 00:00:00 2001
From: divoplade <d@divoplade.fr>
Date: Mon, 7 Sep 2020 23:33:32 +0200
Subject: [PATCH] Update emacs-ess to a more recent snapshot

Hello guix!

There are a few things to note in that new version:
1. The license has changed, it is now GPLv3+;
2. The previous fix for roxygen is not needed anymore, as far as I can tell;
3. The makefiles seem to have changed, so removing julia is done a little
   differently;
4. The failing test =ess-test-r-help-mode= looks suspiciously like the next
   test (this one passes), =ess-test-r-index-mode=, which has the following line:

  (skip-unless (not noninteractive))

So I figured out that the previous one needed it too, it just got forgotten.

Best regards,

divoplade
---
 gnu/packages/statistics.scm | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/gnu/packages/statistics.scm b/gnu/packages/statistics.scm
index 70a85e8b60..6585ce54b0 100644
--- a/gnu/packages/statistics.scm
+++ b/gnu/packages/statistics.scm
@@ -5805,24 +5805,29 @@ Java package that provides routines for various statistical distributions.")
 (define-public emacs-ess
   (package
     (name "emacs-ess")
-    (version "18.10.2")
+    (version "20200905.1022")
     (source (origin
               (method git-fetch)
               (uri (git-reference
                     (url "https://github.com/emacs-ess/ESS")
-                    (commit (string-append "v" version))))
+                    (commit "82cd308ae54a6b918bbceb235e6bf02f53e48e19")))
               (sha256
                (base32
-                "1yq41l2bicwjrc0b731iic20cpcnz6ppigri1jn621qv2qv22vy3"))
+                "0zw6j8jzrdmy41g6313js7c0xlmc2wmiazx4d4wm6hdvykn8q39k"))
               (file-name (git-file-name name version))
               (modules '((guix build utils)))
               (snippet
                '(begin
                   ;; Stop ESS from trying to bundle an external julia-mode.el.
                   (substitute* "lisp/Makefile"
-                    (("^ess-julia.elc: julia-mode.elc") "")
-                    (("^all: julia-mode.el")
-                     "all:"))
+                    (("JULIAS := julia-mode.el julia-mode-latexsubs.el") "JULIAS ="))
+                  (substitute* "Makefile"
+                    (("$(MAKE) julia-mode.el") ""))
+                  (substitute* "test/Makefile"
+			       (("lisp: julia") "lisp:"))
+		  ;; It looks like this test misses a paste
+                  (substitute* "test/ess-test-r.el"
+			       (("\\(ert-deftest ess-test-r-help-mode \\(\\)") "(ert-deftest ess-test-r-help-mode () (skip-unless (not noninteractive))"))
                   ;; Include *.el files in install target.
                   (substitute* "lisp/Makefile"
                     (("\t\\$\\(INSTALL) \\$\\(ELC\\) \\$\\(LISPDIR\\)" elc)
@@ -5837,11 +5842,6 @@ Java package that provides routines for various statistical distributions.")
                   ;; Stop install-info from trying to update the info directory.
                   (substitute* "doc/Makefile"
                     ((".*\\$\\(INFODIR\\)/dir.*") ""))
-                  ;; Fix roxygen preview test.
-                  (substitute* "test/ess-r-tests.el"
-                               (("Add together two numbers.\n")
-                                "Add together two numbers. ")
-                               (("##' add\\(10, 1\\)") "add(10, 1)"))
                   #t))))
     (build-system gnu-build-system)
     (arguments
@@ -5877,4 +5877,4 @@ Java package that provides routines for various statistical distributions.")
     (description "Emacs Speaks Statistics (ESS) is an add-on package for GNU
 Emacs.  It is designed to support editing of scripts and interaction with
 various statistical analysis programs such as R, Julia, and JAGS.")
-    (license license:gpl2+)))
+    (license license:gpl3+)))
-- 
2.28.0


  reply	other threads:[~2020-09-07 21:45 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-28 20:07 bug#43093: emacs-ess is broken by emacs 27.1 divoplade
2020-09-04 14:44 ` bug#43093: Is Emacs 27.1 that breaks emacs-ess? zimoun
2020-09-04 15:26   ` divoplade
2020-09-04 17:00     ` Mark H Weaver
2020-09-04 18:09     ` zimoun
2020-09-04 18:25 ` bug#43093: emacs-ess is broken by emacs 27.1 Tim Howes via Bug reports for GNU Guix
2020-09-07 21:19   ` divoplade
2020-09-07 21:43     ` divoplade [this message]
2020-09-07 21:52     ` divoplade
2020-09-08 19:24       ` Tim Howes via Bug reports for GNU Guix
2020-09-04 23:25 ` bug#43093: [PATCH] gnu: emacs-ess: Update to 20200903.1516 Tim Howes via Bug reports for GNU Guix
2020-09-28  3:28   ` Maxim Cournoyer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=10bf3da9dac586a986c4a098777aefeba3b5f5b5.camel@divoplade.fr \
    --to=d@divoplade.fr \
    --cc=43093@debbugs.gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).