all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH 1/2] gnu: Add LLVM variant with RTTI support.
@ 2016-11-06 19:34 Ricardo Wurmus
  2016-11-06 19:34 ` [PATCH 2/2] gnu: Add faust-2 Ricardo Wurmus
  2016-11-07 23:10 ` [PATCH 1/2] gnu: Add LLVM variant with RTTI support Leo Famulari
  0 siblings, 2 replies; 8+ messages in thread
From: Ricardo Wurmus @ 2016-11-06 19:34 UTC (permalink / raw)
  To: guix-devel

* 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

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [PATCH 2/2] gnu: Add faust-2.
  2016-11-06 19:34 [PATCH 1/2] gnu: Add LLVM variant with RTTI support Ricardo Wurmus
@ 2016-11-06 19:34 ` Ricardo Wurmus
  2016-11-07 23:10   ` Leo Famulari
  2016-11-07 23:10 ` [PATCH 1/2] gnu: Add LLVM variant with RTTI support Leo Famulari
  1 sibling, 1 reply; 8+ messages in thread
From: Ricardo Wurmus @ 2016-11-06 19:34 UTC (permalink / raw)
  To: guix-devel

* 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

^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] gnu: Add LLVM variant with RTTI support.
  2016-11-06 19:34 [PATCH 1/2] gnu: Add LLVM variant with RTTI support Ricardo Wurmus
  2016-11-06 19:34 ` [PATCH 2/2] gnu: Add faust-2 Ricardo Wurmus
@ 2016-11-07 23:10 ` Leo Famulari
  2016-11-08  5:40   ` Eric Bavier
  2016-11-08  8:10   ` Ricardo Wurmus
  1 sibling, 2 replies; 8+ messages in thread
From: Leo Famulari @ 2016-11-07 23:10 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: guix-devel

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?

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 2/2] gnu: Add faust-2.
  2016-11-06 19:34 ` [PATCH 2/2] gnu: Add faust-2 Ricardo Wurmus
@ 2016-11-07 23:10   ` Leo Famulari
  0 siblings, 0 replies; 8+ messages in thread
From: Leo Famulari @ 2016-11-07 23:10 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: guix-devel

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

LGTM

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] gnu: Add LLVM variant with RTTI support.
  2016-11-07 23:10 ` [PATCH 1/2] gnu: Add LLVM variant with RTTI support Leo Famulari
@ 2016-11-08  5:40   ` Eric Bavier
  2016-11-08  8:10   ` Ricardo Wurmus
  1 sibling, 0 replies; 8+ messages in thread
From: Eric Bavier @ 2016-11-08  5:40 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] gnu: Add LLVM variant with RTTI support.
  2016-11-07 23:10 ` [PATCH 1/2] gnu: Add LLVM variant with RTTI support Leo Famulari
  2016-11-08  5:40   ` Eric Bavier
@ 2016-11-08  8:10   ` Ricardo Wurmus
  2016-11-08 16:04     ` Leo Famulari
  1 sibling, 1 reply; 8+ messages in thread
From: Ricardo Wurmus @ 2016-11-08  8:10 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel


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

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] gnu: Add LLVM variant with RTTI support.
  2016-11-08  8:10   ` Ricardo Wurmus
@ 2016-11-08 16:04     ` Leo Famulari
  2016-11-08 20:02       ` Ricardo Wurmus
  0 siblings, 1 reply; 8+ messages in thread
From: Leo Famulari @ 2016-11-08 16:04 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: guix-devel

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 :)

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/2] gnu: Add LLVM variant with RTTI support.
  2016-11-08 16:04     ` Leo Famulari
@ 2016-11-08 20:02       ` Ricardo Wurmus
  0 siblings, 0 replies; 8+ messages in thread
From: Ricardo Wurmus @ 2016-11-08 20:02 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel


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

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2016-11-08 20:02 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-06 19:34 [PATCH 1/2] gnu: Add LLVM variant with RTTI support Ricardo Wurmus
2016-11-06 19:34 ` [PATCH 2/2] gnu: Add faust-2 Ricardo Wurmus
2016-11-07 23:10   ` Leo Famulari
2016-11-07 23:10 ` [PATCH 1/2] gnu: Add LLVM variant with RTTI support Leo Famulari
2016-11-08  5:40   ` Eric Bavier
2016-11-08  8:10   ` Ricardo Wurmus
2016-11-08 16:04     ` Leo Famulari
2016-11-08 20:02       ` Ricardo Wurmus

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.