GNU bug report logs - #32532
[PATCH] Add emacs-build-farm

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 32532 in the body.
You can then email your comments to 32532 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: Alex Kost <alezost <at> gmail.com>
To: guix-patches <at> gnu.org
Subject: [PATCH] Add emacs-build-farm
Date: Sun, 26 Aug 2018 14:33:16 +0300

[Message part 1 (text/plain, inline)]

Hello,

This emacs package provides an interface for Hydra and Cuirass build
farms.  AFAICT it supports everything that Cuirass and Hydra APIs
currently provide.

It is a replacement for the old 'guix-hydra-…' stuff from 'emacs-guix'
package (which will be removed in next Emacs-Guix release).

I'm going to make the announcement for it on guix-devel list after this
package will be applied to the guix tree, thanks.

[0001-gnu-Add-emacs-build-farm.patch (text/x-patch, attachment)]

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

From: ludo <at> gnu.org (Ludovic Courtès)
To: Alex Kost <alezost <at> gmail.com>
Cc: 32532 <at> debbugs.gnu.org
Subject: Re: [bug#32532] [PATCH] Add emacs-build-farm
Date: Mon, 27 Aug 2018 13:52:24 +0200

Hi Alex,

Alex Kost <alezost <at> gmail.com> skribis:

> This emacs package provides an interface for Hydra and Cuirass build
> farms.  AFAICT it supports everything that Cuirass and Hydra APIs
> currently provide.
>
> It is a replacement for the old 'guix-hydra-…' stuff from 'emacs-guix'
> package (which will be removed in next Emacs-Guix release).

Awesome!

> From dc933a2cb96b562c16580b6fb8d79e55f3535e92 Mon Sep 17 00:00:00 2001
> From: Alex Kost <alezost <at> gmail.com>
> Date: Sun, 26 Aug 2018 14:27:33 +0300
> Subject: [PATCH] gnu: Add emacs-build-farm.
>
> * gnu/packages/emacs.scm (emacs-build-farm): New variable.

LGTM!

Regarding the code, I’m interested in adding new features to (guix ci),
the client module for Cuirass/Hydra.  Just mentioning it in case it’s
something that would make sense to share with emacs-build-farm.

Thanks,
Ludo’.

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

From: Alex Kost <alezost <at> gmail.com>
To: ludo <at> gnu.org (Ludovic Courtès)
Cc: 32532 <at> debbugs.gnu.org
Subject: Re: [bug#32532] [PATCH] Add emacs-build-farm
Date: Mon, 27 Aug 2018 18:59:02 +0300

Ludovic Courtès (2018-08-27 13:52 +0200) wrote:

> Alex Kost <alezost <at> gmail.com> skribis:
[...]
>> From dc933a2cb96b562c16580b6fb8d79e55f3535e92 Mon Sep 17 00:00:00 2001
>> From: Alex Kost <alezost <at> gmail.com>
>> Date: Sun, 26 Aug 2018 14:27:33 +0300
>> Subject: [PATCH] gnu: Add emacs-build-farm.
>>
>> * gnu/packages/emacs.scm (emacs-build-farm): New variable.
>
> LGTM!

Thanks, I will commit it soon.

> Regarding the code, I’m interested in adding new features to (guix ci),
> the client module for Cuirass/Hydra.  Just mentioning it in case it’s
> something that would make sense to share with emacs-build-farm.

Sorry, I don't understand what you mean.  emacs-build-farm is a pure
elisp package, it can't share any guile code with (guix ci), or did you
mean something else?

-- 
Alex

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

From: ludo <at> gnu.org (Ludovic Courtès)
To: Alex Kost <alezost <at> gmail.com>
Cc: 32532 <at> debbugs.gnu.org
Subject: Re: [bug#32532] [PATCH] Add emacs-build-farm
Date: Mon, 27 Aug 2018 23:14:37 +0200

Alex Kost <alezost <at> gmail.com> skribis:

> Ludovic Courtès (2018-08-27 13:52 +0200) wrote:

[...]

>> Regarding the code, I’m interested in adding new features to (guix ci),
>> the client module for Cuirass/Hydra.  Just mentioning it in case it’s
>> something that would make sense to share with emacs-build-farm.
>
> Sorry, I don't understand what you mean.  emacs-build-farm is a pure
> elisp package, it can't share any guile code with (guix ci), or did you
> mean something else?

I know it’s pure elisp, which is why I wrote “in case it’s something
that would make sense.”  That is, if you think we could share that
Scheme code, we can do it (but emacs-build-farm would have to use
Geiser); if you don’t want to, that’s also fine.  I hope it’s clearer
now.

Ludo’.

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

From: Alex Kost <alezost <at> gmail.com>
To: ludo <at> gnu.org (Ludovic Courtès)
Cc: 32532-done <at> debbugs.gnu.org
Subject: Re: [bug#32532] [PATCH] Add emacs-build-farm
Date: Tue, 28 Aug 2018 12:25:08 +0300

Ludovic Courtès (2018-08-27 23:14 +0200) wrote:

> Alex Kost <alezost <at> gmail.com> skribis:
>
>> Ludovic Courtès (2018-08-27 13:52 +0200) wrote:
>
> [...]
>
>>> Regarding the code, I’m interested in adding new features to (guix ci),
>>> the client module for Cuirass/Hydra.  Just mentioning it in case it’s
>>> something that would make sense to share with emacs-build-farm.
>>
>> Sorry, I don't understand what you mean.  emacs-build-farm is a pure
>> elisp package, it can't share any guile code with (guix ci), or did you
>> mean something else?
>
> I know it’s pure elisp, which is why I wrote “in case it’s something
> that would make sense.”  That is, if you think we could share that
> Scheme code, we can do it (but emacs-build-farm would have to use
> Geiser); if you don’t want to, that’s also fine.

Oh, no, no.  emacs-build-farm will not use any scheme code.  There is no
reason for that.  Emacs has in-built "json" library, so parsing json
from Hydra/Cuirass using elisp is easy.  Using additional scheme code
for that would be a big unneeded complication.

> I hope it’s clearer now.

Yes, thanks for explaining.

Applied as 40f7ebac66913a531854cc18f9b417624508d654.

-- 
Alex

Previous Next

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