[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Add Elixir
From: |
Ludovic Courtès |
Subject: |
Re: [PATCH] Add Elixir |
Date: |
Tue, 02 Aug 2016 19:26:12 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Hello!
Ricardo Wurmus <address@hidden> skribis:
> I documented the most common test errors, but there don’t seem to be any
> quick fixes. Attached is a new patch based on yours. If this looks
> good to you I’ll push it with you as the commit author and with a
> “Co-authored-by: Ricardo Wurmus” line in the commit summary.
Woow, congratulation, and a big thanks for taking the time to
investigate the issues!
> +++ b/gnu/packages/patches/elixir-disable-failing-tests.patch
> @@ -0,0 +1,261 @@
> +Most of these tests fail for unknown reasons when run in the chroot
> +environment of a Guix build process.
> +
> +Common failures are:
> +
> + * Mix.Shell.cmd() fails with error 130
Could it be that this method relies on /bin/sh? We have
patches/substitutions for similar functions in libc, gawk, and Guile,
for instance.
> + * The git_repo fixture cannot be found
> + * Communication with spawned processes fails with EPIPE
It might be that processes are spawned with /bin/sh too, and thus fail
instantly.
Cheers,
Ludo’.