Ludovic Courtès writes:

Hey Ludo',

>> I opted for build-aux/cuiras/gnu-cuirass.scm.
>
> Sounds good!

Ok, grand.

>> +(define (symbol-alist-entry->keyword-alist-entry entry)
>> +  (cons (symbol->keyword (car entry)) (cdr entry)))
>
> I *think* that’s correct, though we’ll need to double check.

I tested today and there were minor problems.  Cuirass actually doesn't
take an alist; instead takes a list that includes (#:job-name . "name").

Also, Cuirass performs an sexp-read and thus #<<license> ...> needs to
get sexp'ified.

Anyway, what I proposed was close and attached is a tested, working
version (that may need some work, see below).

>> --- a/build-aux/hydra/gnu-system.scm
>> +++ b/build-aux/hydra/gnu-system.scm
>> @@ -270,6 +270,8 @@ valid."
>>    (define subset
>>      (match (assoc-ref arguments 'subset)
>>        ("core" 'core)                              ; only build core packages
>> +      ("hello" 'hello)                            ; only build hello
>> +      (((? string?) (? string?) ...) 'list)       ; only build selected list of packages
>>        (_ 'all)))                                  ; build everything
>
> This part could be added separately.

Yes...it could.  Do you mean a separate patch, or ...

> (It’s not usuable via Hydra since its UIs does not support passing
> list-of-strings arguments.)

...I don't quite understand what you propose here.  I appreciate that
I'm adding functionality for Cuirass to the hydra file where in itself
that does not make much sense...

Otoh, I don't see how to move this functionality to
cuirass/gnu-system.scm only without duplicating much of `hydra-jobs'; so
that's probably not what you mean... / somewhat confused here. ;-)

I'd be happy to update or split this patch if once I understand what you
want or feel free to do it for me if that's easier for you.

Thanks!
janneke