Subject: bug#29494: closed (Re: bug#29441: [email-masked] failing to build)

From: [email-masked] (GNU bug Tracking System)
Date: Tue, 28 Nov 2017 23:31:06 +0000

Your bug report

#29494: Rename synergy build directory (was: Fix)

which was filed against the guix-patches package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
29494: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=29494
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#29441: [email-masked] failing to build
From: Leo Famulari <[email-masked]>
To: Ryan Watkins <[email-masked]>, Ben Sturmfels <[email-masked]>
Cc: [email-masked], [email-masked], [email-masked]
Date: Tue, 28 Nov 2017 18:29:58 -0500
List-Id: <debbugs-submit.debbugs.gnu.org>

On Sat, Nov 25, 2017 at 06:51:22PM +0100, Ryan Watkins wrote:
> So from a quick discussion on #guix, lfam deduced
> [email-masked] seems to be failing on Hydra.
> 
> Hydra shows that it started failing to build somewhere between commits
> c3bece to 411937: https://hydra.gnu.org/build/2338209
> 
> An obvious candidate might be:
> https://git.savannah.gnu.org/cgit/guix.git/commit/?id=ba2cd6c2d8f40bc4d3fb91dd4c6ea05b8a586dec
> 
> I have a feeling just from where the fail happens that the issue lies
> primarily in the way the test-suite is running.

Fixed with d2bdee8a69d3316c91fab68fe0cfb2a533db9ce6:

https://git.savannah.gnu.org/cgit/guix.git/commit/?id=d2bdee8a69d3316c91fab68fe0cfb2a533db9ce6

Subject: Rename synergy build directory (was: Fix)
From: Ben Sturmfels <[email-masked]>
To: [email-masked]
Date: Wed, 29 Nov 2017 10:05:05 +1100
List-Id: <debbugs-submit.debbugs.gnu.org>

Sorry for the awful subject line! I got distracted.

On Tue, 28 Nov 2017, Ben Sturmfels wrote:

> Hi Folks,
>
> The command `guix build synergy` currently fails with:
>
>     $ guix build synergy
>     ...
>     phase `build' succeeded after 123.5 seconds
>     starting phase `check'
>     phase `check' failed after 0.0 seconds
>     builder for `/gnu/store/xwj5qbqkn4ybiiwskypfgglqyvkl539s-synergy-1.8.8.drv' failed with exit code 1
>     @ build-failed /gnu/store/xwj5qbqkn4ybiiwskypfgglqyvkl539s-synergy-1.8.8.drv - 1 builder for `/gnu/store/xwj5qbqkn4ybiiwskypfgglqyvkl539s-synergy-1.8.8.drv' failed with exit code 1
>     guix build: error: build failed: build of `/gnu/store/xwj5qbqkn4ybiiwskypfgglqyvkl539s-synergy-1.8.8.drv' failed
>
> It appears that the package now builds into the
> "synergy-core-1.8.8-stable" directory instead of "synergy-1.8.8-stable".
> Sadly it sounds like the renaming is to make way for "Synergy" to be
> used as the name of the proprietary release.
>
> Build completes successfully with the attached patch.
>
> Regards,
> Ben
>
> --
> Ben Sturmfels
>
> Sturm Software Engineering
> www.sturm.com.au
> (03) 9024 2467
> From e2754965f87be2e170bbc7011125d43513431559 Mon Sep 17 00:00:00 2001
> From: Ben Sturmfels <[email-masked]>
> Date: Wed, 29 Nov 2017 09:28:43 +1100
> Subject: [PATCH] gnu: synergy: Fix install test/install error.
>
> * gnu/packages/synergy.scm (synergy): Now builds to "synergy-core" directory.
> ---
>  gnu/packages/synergy.scm | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gnu/packages/synergy.scm b/gnu/packages/synergy.scm
> index 8e863149c..ec1872cc0 100644
> --- a/gnu/packages/synergy.scm
> +++ b/gnu/packages/synergy.scm
> @@ -68,7 +68,7 @@
>         ("xinput"  ,xinput)))
>      (arguments
>       `(#:phases
> -       (let ((srcdir (string-append "../synergy-" ,version "-stable")))
> +       (let ((srcdir (string-append "../synergy-core-" ,version "-stable")))
>           (modify-phases %standard-phases
>             (add-before 'configure 'unpack-aux-src
>               ;; TODO: package and use from system