[PATCH] gnu: Add parole.

[PATCH] gnu: Add parole.

[Kei Yamashita]

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

I've disabled the mpris2 plugin because parole will fail to build,
citing a missing gst/tag/tag.h header file (even though it exists in
gst-plugins-base's include folder). If I someone knows a way that
wouldn't require me to disable the mpris2 plugin, please share.

[-- Attachment #2: 0001-gnu-Add-parole.patch --]
[-- Type: application/octet-stream, Size: 3250 bytes --]

From 20621fbe69d6ed794fe536d4aad940097f772f85 Mon Sep 17 00:00:00 2001
From: Kei Yamashita <kei@openmailbox.org>
Date: Sun, 17 Apr 2016 11:10:19 -0400
Subject: [PATCH] gnu: Add parole.

* gnu/packages/xfce.scm (parole): New variable.
---
 gnu/packages/xfce.scm | 48 ++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 48 insertions(+)

diff --git a/gnu/packages/xfce.scm b/gnu/packages/xfce.scm
index 89071e9..932dec1 100644
--- a/gnu/packages/xfce.scm
+++ b/gnu/packages/xfce.scm
@@ -3,6 +3,7 @@
 ;;; Copyright © 2014, 2015 Mark H Weaver <mhw@netris.org>
 ;;; Copyright © 2016 Andreas Enge <andreas@enge.fr>
 ;;; Copyright © 2016 Florian Paul Schmidt <mista.tapas@gmx.net>
+;;; Copyright © 2016 Kei Yamashita <kei@openmailbox.org>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -39,6 +40,7 @@
   #:use-module (gnu packages pdf)
   #:use-module (gnu packages polkit)
   #:use-module (gnu packages gstreamer)
+  #:use-module (gnu packages mp3)
   #:use-module (gnu packages linux)
   #:use-module (gnu packages photo)
   #:use-module (gnu packages pcre)
@@ -756,3 +758,49 @@ freedesktop-compliant DBus interfaces to inform other applications about current
 power level so that they can adjust their power consumption, and it provides the
 inhibit interface which allows applications to prevent automatic sleep.")
     (license gpl2+)))
+
+(define-public parole
+  (package
+    (name "parole")
+    (version "0.8.1")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "http://archive.xfce.org/src/apps/parole/0.8/"
+                                  name "-" version ".tar.bz2"))
+              (sha256
+               (base32
+                "1x40ppyf30hc2ywi7gs9ab4k825jm3czrczikcm8s3s90196y8ab"))))
+    (build-system gnu-build-system)
+    (arguments
+     `(#:configure-flags '("--disable-mpris2-plugin") ; cannot find gst/tag/tag.h
+       #:phases
+       (modify-phases %standard-phases
+         (add-after
+             'install 'wrap-parole
+           (lambda* (#:key inputs outputs #:allow-other-keys)
+             (let ((out             (assoc-ref outputs "out"))
+                   (gst-plugin-path (getenv "GST_PLUGIN_SYSTEM_PATH")))
+               (wrap-program (string-append out "/bin/parole")
+                 `("GST_PLUGIN_SYSTEM_PATH" ":" prefix (,gst-plugin-path)))))))))
+    (native-inputs
+     `(("pkg-config" ,pkg-config)
+       ("intltool" ,intltool)))
+    (inputs
+     `(("desktop-file-utils" ,desktop-file-utils)
+       ("glib:bin" ,glib "bin")
+       ("gstreamer" ,gstreamer)
+       ("gst-libav" ,gst-libav)
+       ("gst-plugins-bad" ,gst-plugins-bad)
+       ("gst-plugins-base" ,gst-plugins-base)
+       ("gst-plugins-good" ,gst-plugins-good)
+       ("gst-plugins-ugly" ,gst-plugins-ugly)
+       ("libnotify" ,libnotify)
+       ("libxfce4ui" ,libxfce4ui)
+       ("taglib" ,taglib)))
+    (home-page "http://www.xfce.org/")
+    (synopsis "Modern media player based on the GStreamer framework")
+    (description
+     "Parole is a modern simple media player based on the GStreamer framework
+and written to fit well in the Xfce desktop.  It is designed with simplicity,
+speed and resource usage in mind.")
+    (license gpl2+)))
-- 
2.7.3

Re: [PATCH] gnu: Add parole.

[Leo Famulari]

On Sun, Apr 17, 2016 at 11:14:00AM -0400, Kei Yamashita wrote:
> I've disabled the mpris2 plugin because parole will fail to build,
> citing a missing gst/tag/tag.h header file (even though it exists in
> gst-plugins-base's include folder). If I someone knows a way that
> wouldn't require me to disable the mpris2 plugin, please share.

`./configure --help` lists the environment variables you can use to name
"non-standard" locations for headers and libraries.

Could you try using one of those (probably CFLAGS)?

If you grep in 'gnu/packages/ for -I or CFLAGS, you will find some good
examples.

Re: [PATCH] gnu: Add parole.

[Kei Yamashita]

On Fri, 29 Apr 2016 03:05:08 -0400
Leo Famulari <leo@famulari.name> wrote:

> On Sun, Apr 17, 2016 at 11:14:00AM -0400, Kei Yamashita wrote:
> > I've disabled the mpris2 plugin because parole will fail to build,
> > citing a missing gst/tag/tag.h header file (even though it exists in
> > gst-plugins-base's include folder). If I someone knows a way that
> > wouldn't require me to disable the mpris2 plugin, please share.  
> 
> `./configure --help` lists the environment variables you can use to
> name "non-standard" locations for headers and libraries.
> 
> Could you try using one of those (probably CFLAGS)?
> 
> If you grep in 'gnu/packages/ for -I or CFLAGS, you will find some
> good examples.

Looking deeper into the issue, it seems that it's only the mpris2
plugin that can't see the given gstreamer header file. Other pieces of
the build are fine. I am not sure what to do with this.