Re: [PATCH] Add libtheora package

[David Thompson <dthompson2@worcester.edu>]

[-- Attachment #1: Type: text/plain, Size: 624 bytes --]

On 10/14/2013 05:23 AM, Andreas Enge wrote:
> On Sun, Oct 13, 2013 at 07:20:59PM -0400, David Thompson wrote:
>> How does it look?
>
> It compiles well and looks good, congratulations! One point, though:
> If you do
>     cd /nix/store/f06apny3g77kjk0wxzhmgl4b3ya77f0b-libtheora-1.1.1/lib/pkgconfig/
>     grep Requires *.pc
> you only see ogg, not vorbis.
> So I think you can move libvorbis from propagated-inputs into regular inputs.
>
> At a later point, you might wish to add libsdl as an input (I think you were
> working on it already.)
>
> Andreas
>

One more try, this time with what I hope is a proper commit log.

[-- Attachment #2: 0001-gnu-Add-libtheora.patch --]
[-- Type: text/x-patch, Size: 1897 bytes --]

From 0865a4a55bd7b0cccf2e45b0f5bb655344d98017 Mon Sep 17 00:00:00 2001
From: David Thompson <dthompson2@worcester.edu>
Date: Mon, 14 Oct 2013 16:40:02 -0400
Subject: [PATCH] gnu: Add libtheora.

* gnu/packages/oggvorbis.scm (libtheora): New variable.
---
 gnu/packages/oggvorbis.scm | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/gnu/packages/oggvorbis.scm b/gnu/packages/oggvorbis.scm
index 0652ec3..952ef9e 100644
--- a/gnu/packages/oggvorbis.scm
+++ b/gnu/packages/oggvorbis.scm
@@ -34,6 +34,7 @@
   #:use-module (guix build-system gnu)
   #:export (libogg
             libvorbis
+            libtheora
             speex
             ao
             flac
@@ -88,6 +89,29 @@ polyphonic) audio and music at fixed and variable bitrates from 16 to
                                "See COPYING in the distribution."))
    (home-page "http://xiph.org/vorbis/")))
 
+(define libtheora
+  (package
+    (name "libtheora")
+    (version "1.1.1")
+    (source (origin
+             (method url-fetch)
+             (uri (string-append "http://downloads.xiph.org/releases/theora/libtheora-"
+                                 version ".tar.xz"))
+             (sha256
+              (base32
+               "0q8wark9ribij57dciym5vdikg2464p8q2mgqvfb78ksjh4s8vgk"))))
+    (build-system gnu-build-system)
+    (inputs `(("libvorbis" ,libvorbis)))
+    (propagated-inputs `(("libogg" ,libogg)))
+    (synopsis "A library implementing the theora video format")
+    (description
+     "The libtheora library implements the ogg theora video format,
+a fully open, non-proprietary, patent-and-royalty-free, general-purpose
+compressed video format.")
+    (license (license:bsd-style "file://COPYING"
+                                "See COPYING in the distribution."))
+    (home-page "http://xiph.org/theora/")))
+
 (define speex
   (package
     (name "speex")
-- 
1.8.4.rc3