[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Commit 958dd3ce68 will break emacs interface after "guix pull"

From: Alex Kost
Subject: Re: Commit 958dd3ce68 will break emacs interface after "guix pull"
Date: Fri, 06 May 2016 21:25:33 +0300
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)

Ludovic Courtès (2016-05-06 13:36 +0300) wrote:

> Alex Kost <address@hidden> skribis:
>> Hello Ludo, this is not related to system tests, but in this patchset
>> you also moved 'memoize' to (guix combinators).  In commit 86670ca4b7 I
>> adjusted emacs interface to use it.
> Ooops, sorry for the breakage.  I guess I don’t master ‘grep’ yet.  ;-)

No problem, I keep an eye emacs UI :-)

>> However, it is not enough (see [1]): currently, if a user does "guix
>> pull", (s)he gets the recent guile code with the new (guix combinators)
>> module, but the emacs stuff is still old as it comes with the
>> development snapshot.
>> So in this case emacs interface will not work (complaining about unbound
>> 'memoize' variable).
> Indeed.
>> So I think it's the time to update guix-devel package.  Is it OK to do
>> it now?
> Yes, we could do that, but the problem remains on foreign distros
> though.

Not really: on foreign distros the recommended way (it's probably the
only way except using git checkout) to use emacs interface is to install
guix into a user profile, so once we update guix-devel snapshot, it can
be updated by a user, and this breakage will be fixed.

> What about simply #:re-export (memoize) in (guix utils) for now?

Oh, indeed, this is a good workaround, thanks!  OK for the attached
patch or should I add some comment there?

>> In general, it is not perfect that "guix pull" updates only guile
>> modules, and not emacs interface.  I recall there were similar (but much
>> smaller) breakages when emacs stuff and guile code (from
>> "~/.config/guix/latest") were out of sync.
> Yes, that’s definitely one of the shortcomings we should address:

Yes, I'm aware of this, thanks (and sorry that I'm not the taker).

Attachment: 0001-guix-utils-Re-export-memoize.patch
Description: Text Data

reply via email to

[Prev in Thread] Current Thread [Next in Thread]