unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: ludo@gnu.org (Ludovic Courtès)
To: Jan Nieuwenhuizen <janneke@gnu.org>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH] gnu: Resurrect hydra.
Date: Tue, 13 Sep 2016 10:22:10 +0200	[thread overview]
Message-ID: <87eg4o9pl9.fsf@gnu.org> (raw)
In-Reply-To: <87d1kcvkzg.fsf@gnu.org> (Jan Nieuwenhuizen's message of "Fri, 09 Sep 2016 23:09:23 +0200")

Hi Jan,

Jan Nieuwenhuizen <janneke@gnu.org> skribis:

> I wanted to play with Hydra and found it did not build and so did not
> some of its dependencies.
>
> Our nix package got updated which broke the hydra build.  First I
> downgraded nix to 1.8.  That worked, eventually.  We decided on #guix
> that we'd rather upgrade than downgrade.

Neat.

I should mention that I didn’t maintain the Guile part in Hydra (which
is really just the ‘hydra-eval-guile-jobs.in’ script), so the latest
Hydra won’t work with Guix.  :-/

I think it’s “just a matter” of updating this script to produce JSON
instead of XML, but that has to be done (and I thought it’d be more
fruitful to invest in Cuirass).

ISTR that other changes were made that tie Hydra to Nix more closely,
and I’m not sure what the implications are.

> So I upgraded hydra to latest git and did not get past build problems.
> The Hydra git does not mention anything on dependencies or versions.  I
> asked on #guix and later on #nixos.
>
> Finally I found this comment from last February
>
>     https://github.com/NixOS/hydra/issues/270
>
>     "You have to use NixUnstable or go a few Hydra commits back."
>
> So after a couple of iterations in the Hydra git I settled on a commit
> from 20151030 (just after a compile fix that I ran into).
>
> Two perl packages did not build anymore (failing tests), so I disabled
> the tests.  Hydra also has two new dependencies, which I added.

Woow, congrats on getting this far!

> From 9c3cc736629b1c24eae489d586f26a0c19356725 Mon Sep 17 00:00:00 2001
> From: Jan Nieuwenhuizen <janneke@gnu.org>
> Date: Fri, 9 Sep 2016 19:57:26 +0200
> Subject: [PATCH] gnu: Resurrect hydra.
>
> * gnu/packages/gd.scm (perl-gd): Disable tests, 1 fails.  Fixes
> build.
> * gnu/packages/imagemagick.scm (perl-image-magick): Disable tests, 2 fail.
> * gnu/packages/patches/hydra-automake-1.15.patch: Remove.
> * gnu/local.mk (dist_patch_DATA): Remove listing.
> * gnu/packages/ci.scm (perl-net-statsd,libpqxx): Add variable.  (hydra): Bump
> to 20151030.  Fixes building with nix-1.11.x.  Add new libpqxx and
> perl-net-statsd dependencies.

Could you split it into separate packages: one for each new package, one
for each individual package change?

Thank you!

Ludo’.

  reply	other threads:[~2016-09-13  8:22 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-09 21:09 [PATCH] gnu: Resurrect hydra Jan Nieuwenhuizen
2016-09-13  8:22 ` Ludovic Courtès [this message]
2016-09-13 16:59   ` Jan Nieuwenhuizen
2016-09-13 16:59     ` [PATCH 1/5] gnu: perl-gd: disable tests Jan Nieuwenhuizen
2016-09-13 16:59     ` [PATCH 2/5] gnu: perl-image-magick: " Jan Nieuwenhuizen
2016-09-13 16:59     ` [PATCH 3/5] gnu: Add perl-net-statsd Jan Nieuwenhuizen
2016-09-13 16:59     ` [PATCH 4/5] gnu: Add libpqxx Jan Nieuwenhuizen
2016-09-13 16:59     ` [PATCH 5/5] gnu: Resurrect hydra Jan Nieuwenhuizen
2016-09-14 19:44       ` Leo Famulari

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=87eg4o9pl9.fsf@gnu.org \
    --to=ludo@gnu.org \
    --cc=guix-devel@gnu.org \
    --cc=janneke@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).