all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Ricardo Wurmus <ricardo.wurmus@mdc-berlin.de>
To: Mathieu Othacehe <m.othacehe@gmail.com>
Cc: guix-devel@gnu.org, Alex Kost <alezost@gmail.com>
Subject: Re: Update of Emacs to 26.1.
Date: Tue, 29 May 2018 14:03:40 +0200	[thread overview]
Message-ID: <87po1er7mr.fsf@elephly.net> (raw)
In-Reply-To: <87bmcyhegg.fsf@gmail.com>


Hi Mathieu,

> Here is a serie to update emacs and the broken dependencies I found,
> what do you think?

Heh, I *just* finished the update myself and came up with very similar
patches to yours.

> Subject: [PATCH 1/4] gnu: emacs: Update to 26.1.
>
> * gnu/packages/emacs.scm (emacs): Update to 26.1,
> [arguments]: disable tests as they do not exist in release.
> * gnu/packages/patches/emacs-fix-scheme-indent-function.patch: Update patch
> indentation and line numbers.

Ok.  I’ll take this instead of my revert of your revert :)

> Subject: [PATCH 2/4] gnu: emacs-minimal: Fix build.
>
> * gnu/packages/emacs.scm: (emacs-minimal): Pass "--with-gnutls=no" is required
> to build without gnutls since emacs version 26.1.

The wording is odd here, but the patch is the same as mine.

> From d0b615809a33ab59d077576e68c79b415852b4a2 Mon Sep 17 00:00:00 2001
> From: Mathieu Othacehe <m.othacehe@gmail.com>
> Date: Tue, 29 May 2018 11:45:07 +0200
> Subject: [PATCH 3/4] gnu: emacs-s: Disable tests.
>
> * gnu/packages/emacs.scm (emacs-s): Since emacs version 26, the tests defined
> in this package fail at first evaluation and succeed after, probably due to
> macro expansion.

I found that using the “#:emacs ,emacs” makes the tests pass, so I’d
prefer that over disabling the tests.

> From 7133b98a9a65cb85c9fb56db9569fd95388c9549 Mon Sep 17 00:00:00 2001
> From: Mathieu Othacehe <m.othacehe@gmail.com>
> Date: Tue, 29 May 2018 11:42:09 +0200
> Subject: [PATCH 4/4] gnu: emacs-dash: Update to 2.14.1.
>
> * gnu/packages/emacs.scm (emacs-dash): Update to 2.14.1.

I did the same but also took the sources from git because the generated
tarballs are not reliable.

I’ll push a mix of our patches to master in the next few minutes.

Thanks!

--
Ricardo

  reply	other threads:[~2018-05-29 12:04 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-05-28 19:25 Update of Emacs to 26.1 Ricardo Wurmus
2018-05-28 19:51 ` Mathieu Othacehe
2018-05-28 20:20   ` Ricardo Wurmus
2018-05-29  9:34 ` Alex Kost
2018-05-29 11:32   ` Ricardo Wurmus
2018-05-29 11:46     ` Mathieu Othacehe
2018-05-29 12:03       ` Ricardo Wurmus [this message]
2018-05-29 12:15         ` Mathieu Othacehe
2018-05-29 16:35     ` Alex Kost

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

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

  git send-email \
    --in-reply-to=87po1er7mr.fsf@elephly.net \
    --to=ricardo.wurmus@mdc-berlin.de \
    --cc=alezost@gmail.com \
    --cc=guix-devel@gnu.org \
    --cc=m.othacehe@gmail.com \
    /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 external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.