Re: failing packages

[Ricardo Wurmus]

Andreas Enge <address@hidden> writes:

> How about non-sequencer?
>    http://hydra.gnu.org/build/725259/nixlog/1/tail-reload
>
> It fails with messages
> g++: error: unrecognized command line option ‘-msse2’
> g++: error: unrecognized command line option ‘-mfpmath=sse’
> g++: error: unrecognized command line option ‘-msse2’
> g++: error: unrecognized command line option ‘-mfpmath=sse’
> g++: error: unrecognized command line option ‘-msse2’
> g++: error: unrecognized command line option ‘-mfpmath=sse’
> g++: error: unrecognized command line option ‘-msse2’
> g++: error: unrecognized command line option ‘-mfpmath=sse’
>
> Shall we disable everything but x86_64?

No.  How about the attached patch?

>From dfde69d64fa16b653d87283930cddb2082e8fa49 Mon Sep 17 00:00:00 2001
From: Ricardo Wurmus <address@hidden>
Date: Sat, 17 Oct 2015 10:10:07 +0200
Subject: [PATCH] gnu: non-sequencer: Disable SSE when not building on x86_64.

* gnu/packages/music.scm (non-sequencer)[arguments]: Add "--disable-sse"
  flag when not building on x86_64.
---
 gnu/packages/music.scm | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm
index fe8e6f1..a72f754 100644
--- a/gnu/packages/music.scm
+++ b/gnu/packages/music.scm
@@ -274,7 +274,13 @@ Guile.")
       (build-system waf-build-system)
       (arguments
        `(#:tests? #f ;no "check" target
-         #:configure-flags '("--project=sequencer")
+         #:configure-flags
+         (list "--project=sequencer"
+               ;; Disable the use of SSE unless on x86_64.
+               ,@(if (not (string-prefix? "x86_64" (or (%current-target-system)
+                                                       (%current-system))))
+                     '("--disable-sse")
+                     '()))
          #:python ,python-2))
       (inputs
        `(("jack" ,jack-1)
-- 
2.5.0