[PATCH 1/2] gnu: Add LLVM variant with RTTI support.

[PATCH 1/2] gnu: Add LLVM variant with RTTI support.

[Ricardo Wurmus]

* gnu/packages/llvm.scm (llvm-with-rtti): New variable.
---
 gnu/packages/llvm.scm | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/gnu/packages/llvm.scm b/gnu/packages/llvm.scm
index 1a1f7ee..96679bf 100644
--- a/gnu/packages/llvm.scm
+++ b/gnu/packages/llvm.scm
@@ -85,6 +85,17 @@ languages is in development.  The compiler infrastructure includes mirror sets
 of programming tools as well as libraries with equivalent functionality.")
     (license license:ncsa)))
 
+(define-public llvm-with-rtti
+  (package (inherit llvm)
+    (name "llvm-with-rtti")
+    (arguments
+     (substitute-keyword-arguments (package-arguments llvm)
+       ((#:configure-flags flags)
+        `(append '("-DCMAKE_SKIP_BUILD_RPATH=FALSE"
+                   "-DCMAKE_BUILD_WITH_INSTALL_RPATH=FALSE"
+                   "-DLLVM_REQUIRES_RTTI=1")
+                 ,flags))))))
+
 (define (clang-runtime-from-llvm llvm hash)
   (package
     (name "clang-runtime")
-- 
2.10.1

[PATCH 2/2] gnu: Add faust-2.

[Ricardo Wurmus]

* gnu/packages/audio.scm (faust-2): New variable.
---
 gnu/packages/audio.scm | 35 +++++++++++++++++++++++++++++++++++
 1 file changed, 35 insertions(+)

diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
index b9034c9..1159e8d 100644
--- a/gnu/packages/audio.scm
+++ b/gnu/packages/audio.scm
@@ -47,6 +47,7 @@
   #:use-module (gnu packages compression)
   #:use-module (gnu packages curl)
   #:use-module (gnu packages databases)
+  #:use-module (gnu packages emacs)
   #:use-module (gnu packages file)
   #:use-module (gnu packages flex)
   #:use-module (gnu packages fltk)
@@ -55,11 +56,13 @@
   #:use-module (gnu packages glib)
   #:use-module (gnu packages gtk)
   #:use-module (gnu packages gnome)
+  #:use-module (gnu packages gnunet) ; libmicrohttpd
   #:use-module (gnu packages gperf)
   #:use-module (gnu packages image)
   #:use-module (gnu packages ncurses)
   #:use-module (gnu packages qt)
   #:use-module (gnu packages linux)
+  #:use-module (gnu packages llvm)
   #:use-module (gnu packages mp3) ;taglib
   #:use-module (gnu packages perl)
   #:use-module (gnu packages pkg-config)
@@ -67,7 +70,9 @@
   #:use-module (gnu packages python)
   #:use-module (gnu packages rdf)
   #:use-module (gnu packages readline)
+  #:use-module (gnu packages tls)
   #:use-module (gnu packages video)
+  #:use-module (gnu packages vim) ;xxd
   #:use-module (gnu packages webkit)
   #:use-module (gnu packages xiph)
   #:use-module (gnu packages xml)
@@ -928,6 +933,36 @@ PS, and DAB+.")
      "Faust is a programming language for realtime audio signal processing.")
     (license license:gpl2+)))
 
+(define-public faust-2
+  (package
+    (inherit faust)
+    (version "2.0.a51")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                    "mirror://sourceforge/faudiostream/faust-" version ".tgz"))
+              (sha256
+               (base32
+                "1yryjqfqmxs7lxy95hjgmrncvl9kig3rcsmg0v49ghzz7vs7haxf"))))
+    (build-system gnu-build-system)
+    (arguments
+     (substitute-keyword-arguments (package-arguments faust)
+       ((#:make-flags flags)
+        `(list (string-append "prefix=" (assoc-ref %outputs "out"))
+               "world"))))
+    (native-inputs
+     `(("llvm" ,llvm-with-rtti)
+       ("which" ,which)
+       ("xxd" ,vim)
+       ("ctags" ,emacs-minimal)  ; for ctags
+       ("pkg-config" ,pkg-config)))
+    (inputs
+     `(("libsndfile" ,libsndfile)
+       ("libmicrohttpd" ,libmicrohttpd)
+       ("ncurses" ,ncurses)
+       ("openssl" ,openssl)
+       ("zlib" ,zlib)))))
+
 (define-public freepats
   (package
     (name "freepats")
-- 
2.10.1

Re: [PATCH 1/2] gnu: Add LLVM variant with RTTI support.

[Leo Famulari]

On Sun, Nov 06, 2016 at 08:34:23PM +0100, Ricardo Wurmus wrote:
> * gnu/packages/llvm.scm (llvm-with-rtti): New variable.

Seems okay, although would it make sense to enable this in the main LLVM
package?

Re: [PATCH 2/2] gnu: Add faust-2.

[Leo Famulari]

On Sun, Nov 06, 2016 at 08:34:24PM +0100, Ricardo Wurmus wrote:
> * gnu/packages/audio.scm (faust-2): New variable.

LGTM

Re: [PATCH 1/2] gnu: Add LLVM variant with RTTI support.

[Eric Bavier]

On Mon, 7 Nov 2016 18:10:17 -0500
Leo Famulari <leo@famulari.name> wrote:

> On Sun, Nov 06, 2016 at 08:34:23PM +0100, Ricardo Wurmus wrote:
> > * gnu/packages/llvm.scm (llvm-with-rtti): New variable.  
> 
> Seems okay, although would it make sense to enable this in the main LLVM
> package?
> 

I think this would make sense.  I was just preparing a patch to do just
that to support a clamav package.

`~Eric

Re: [PATCH 1/2] gnu: Add LLVM variant with RTTI support.

[Ricardo Wurmus]

Leo Famulari <leo@famulari.name> writes:

> On Sun, Nov 06, 2016 at 08:34:23PM +0100, Ricardo Wurmus wrote:
>> * gnu/packages/llvm.scm (llvm-with-rtti): New variable.
>
> Seems okay, although would it make sense to enable this in the main LLVM
> package?

I’m not sure, because it seems to require dependent packages to also
support this or else be built with “-fno-rtti”.  We should build this
out in a separate branch and see how much damage this causes and how
many changes to existing packages this requires.

~~ Ricardo

Re: [PATCH 1/2] gnu: Add LLVM variant with RTTI support.

[Leo Famulari]

On Tue, Nov 08, 2016 at 09:10:46AM +0100, Ricardo Wurmus wrote:
> 
> Leo Famulari <leo@famulari.name> writes:
> 
> > On Sun, Nov 06, 2016 at 08:34:23PM +0100, Ricardo Wurmus wrote:
> >> * gnu/packages/llvm.scm (llvm-with-rtti): New variable.
> >
> > Seems okay, although would it make sense to enable this in the main LLVM
> > package?
> 
> I’m not sure, because it seems to require dependent packages to also
> support this or else be built with “-fno-rtti”.  We should build this
> out in a separate branch and see how much damage this causes and how
> many changes to existing packages this requires.

Perhaps, but let's not let it delay this patch :)

Re: [PATCH 1/2] gnu: Add LLVM variant with RTTI support.

[Ricardo Wurmus]

Leo Famulari <leo@famulari.name> writes:

> On Tue, Nov 08, 2016 at 09:10:46AM +0100, Ricardo Wurmus wrote:
>> 
>> Leo Famulari <leo@famulari.name> writes:
>> 
>> > On Sun, Nov 06, 2016 at 08:34:23PM +0100, Ricardo Wurmus wrote:
>> >> * gnu/packages/llvm.scm (llvm-with-rtti): New variable.
>> >
>> > Seems okay, although would it make sense to enable this in the main LLVM
>> > package?
>> 
>> I’m not sure, because it seems to require dependent packages to also
>> support this or else be built with “-fno-rtti”.  We should build this
>> out in a separate branch and see how much damage this causes and how
>> many changes to existing packages this requires.
>
> Perhaps, but let's not let it delay this patch :)

Okay :)  Pushed as 8ae60404510fedb89502e27c0e73f830ab429641.
Thanks for the review!

~~ Ricardo