[PATCH 16/24] gnu: gstreamer: Add qt-gstreamer.

[PATCH 16/24] gnu: gstreamer: Add qt-gstreamer.

[David Craven]

* gnu/packages/gstreamer.scm (qt-gstreamer): New variable.
---
 gnu/packages/gstreamer.scm | 42 ++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 42 insertions(+)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index 54919cd..ee3c9e4 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -24,11 +24,13 @@
   #:use-module ((guix licenses) #:prefix license:)
   #:use-module (guix packages)
   #:use-module (guix download)
+  #:use-module (guix build-system cmake)
   #:use-module (guix build-system gnu)
   #:use-module (guix utils)
   #:use-module (gnu packages)
   #:use-module (gnu packages audio)
   #:use-module (gnu packages bison)
+  #:use-module (gnu packages boost)
   #:use-module (gnu packages cdrom)
   #:use-module (gnu packages curl)
   #:use-module (gnu packages compression)
@@ -467,3 +469,43 @@ be used by Python applications using GStreamer.")
     (propagated-inputs
      `(("gst-plugins-base" ,gst-plugins-base)
        ("python-pygobject" ,python2-pygobject)))))
+
+(define-public qt-gstreamer
+  (package
+    (name "qt-gstreamer")
+    (version "1.2.0")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                "https://gstreamer.freedesktop.org/src/qt-gstreamer/"
+                "qt-gstreamer-" version ".tar.xz"))
+              (sha256
+               (base32
+                "1m4g5vcs8r4b8dzndr9a5w0rrawdyinvjkacis8vxnfafhmljfwz"))))
+    (build-system cmake-build-system)
+    (native-inputs
+     `(("glib:bin" ,glib "bin")
+       ("pkg-config" ,pkg-config)))
+    (propagated-inputs
+     `(("boost" ,boost)))
+    (inputs
+     `(("glib" ,glib)
+       ("gstreamer" ,gstreamer)
+       ("gst-libav" ,gst-libav)
+       ("gst-plugins-base" ,gst-plugins-base)
+       ("qtbase" ,qtbase)
+       ("qtdeclarative" ,qtdeclarative)))
+    (arguments
+     `(#:configure-flags
+        '("-DQT_VERSION=5"
+          "-DUSE_GST_PLUGIN_DIR=OFF"
+          "-DUSE_QT_PLUGIN_DIR=OFF")
+       #:validate-runpath? #f))
+    (home-page "https://gstreamer.freedesktop.org/modules/qt-gstreamer.html")
+    (synopsis "C++ bindings for GStreamer with a Qt-style API")
+    (description "QtGStreamer is a set of libraries providing C++ bindings for
+GStreamer with a Qt-style API, plus some helper classes and elements for
+integrating GStreamer better in Qt applications.  The goal of this module is to
+allow easy use of GStreamer for applications targetting MeeGo Mobile or the KDE
+desktop.")
+    (license license:lgpl2.1+)))
-- 
2.9.0

Re: [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer.

[Leo Famulari]

On Tue, Aug 16, 2016 at 08:39:30PM +0200, David Craven wrote:
> * gnu/packages/gstreamer.scm (qt-gstreamer): New variable.

> +       #:validate-runpath? #f))

My understanding is that if the runpath validator fails, then something is wrong with the
package. Why is it disabled here?

Re: [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer.

[Eric Bavier]

Hello David,

You can remove the "gstreamer: " bit from the commit subject.

On Tue, 16 Aug 2016 20:39:30 +0200
David Craven <david@craven.ch> wrote:

> * gnu/packages/gstreamer.scm (qt-gstreamer): New variable.
> ---
>  gnu/packages/gstreamer.scm | 42 ++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 42 insertions(+)
> 
> diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
> index 54919cd..ee3c9e4 100644
> --- a/gnu/packages/gstreamer.scm
> +++ b/gnu/packages/gstreamer.scm
> @@ -24,11 +24,13 @@
>    #:use-module ((guix licenses) #:prefix license:)
>    #:use-module (guix packages)
>    #:use-module (guix download)
> +  #:use-module (guix build-system cmake)
>    #:use-module (guix build-system gnu)
>    #:use-module (guix utils)
>    #:use-module (gnu packages)
>    #:use-module (gnu packages audio)
>    #:use-module (gnu packages bison)
> +  #:use-module (gnu packages boost)
>    #:use-module (gnu packages cdrom)
>    #:use-module (gnu packages curl)
>    #:use-module (gnu packages compression)
> @@ -467,3 +469,43 @@ be used by Python applications using GStreamer.")
>      (propagated-inputs
>       `(("gst-plugins-base" ,gst-plugins-base)
>         ("python-pygobject" ,python2-pygobject)))))
> +
> +(define-public qt-gstreamer
> +  (package
> +    (name "qt-gstreamer")
> +    (version "1.2.0")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append
> +                "https://gstreamer.freedesktop.org/src/qt-gstreamer/"
> +                "qt-gstreamer-" version ".tar.xz"))
> +              (sha256
> +               (base32
> +                "1m4g5vcs8r4b8dzndr9a5w0rrawdyinvjkacis8vxnfafhmljfwz"))))
> +    (build-system cmake-build-system)
> +    (native-inputs
> +     `(("glib:bin" ,glib "bin")
> +       ("pkg-config" ,pkg-config)))
> +    (propagated-inputs
> +     `(("boost" ,boost)))

Is boost propagated due to header references?

> +    (inputs
> +     `(("glib" ,glib)
> +       ("gstreamer" ,gstreamer)
> +       ("gst-libav" ,gst-libav)
> +       ("gst-plugins-base" ,gst-plugins-base)
> +       ("qtbase" ,qtbase)
> +       ("qtdeclarative" ,qtdeclarative)))
> +    (arguments
> +     `(#:configure-flags
> +        '("-DQT_VERSION=5"
> +          "-DUSE_GST_PLUGIN_DIR=OFF"
> +          "-DUSE_QT_PLUGIN_DIR=OFF")
> +       #:validate-runpath? #f))

As Leo mentioned, what is the reason for disabling runpath validation?

> +    (home-page "https://gstreamer.freedesktop.org/modules/qt-gstreamer.html")
> +    (synopsis "C++ bindings for GStreamer with a Qt-style API")
> +    (description "QtGStreamer is a set of libraries providing C++ bindings for
> +GStreamer with a Qt-style API, plus some helper classes and elements for
> +integrating GStreamer better in Qt applications.  The goal of this module is to
> +allow easy use of GStreamer for applications targetting MeeGo Mobile or the KDE
> +desktop.")
> +    (license license:lgpl2.1+)))

Re: [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer.

[David Craven]

> Is boost propagated due to header references?

The include files reference these headers:

#include <boost/type_traits.hpp>
#include <boost/utility/enable_if.hpp>
#include <boost/config.hpp>
#include <boost/mpl/if.hpp>

> what is the reason for disabling runpath validation?

I'm getting a bunch of these errors. The problem seems to be that
/gnu/store/2g5xcc8hp16nfnf37449f3p492m8i03h-qt-gstreamer-1.2.0-1.fc159e5/lib64
isn't in the RUNPATH. But isn't that folder implicitly in the RUNPATH
for libraries in the same directory? How can I get it in the RUNPATH,
is there a better way than using patchelf? what is the reason for
disabling runpath validation?

validating RUNPATH of 7 binaries in
"/gnu/store/2g5xcc8hp16nfnf37449f3p492m8i03h-qt-gstreamer-1.2.0-1.fc159e5/lib64"...
/gnu/store/2g5xcc8hp16nfnf37449f3p492m8i03h-qt-gstreamer-1.2.0-1.fc159e5/lib64/libQt5GStreamer-1.0.so:
error: depends on 'libQt5GLib-2.0.so.0', which cannot be found in
RUNPATH ("/gnu/store/2g5xcc8hp16nfnf37449f3p492m8i03h-qt-gstreamer-1.2.0-1.fc159e5/lib"
"/gnu/store/m9vxvhdj691bq1f85lpflvnhcvrdilih-glibc-2.23/lib"
"/gnu/store/9nifwk709wajpyfwa0jzaa3p6mf10vxs-gcc-4.9.3-lib/lib"
"/gnu/store/l1s4cw9g58hmcpd2qgbckfl228143qzx-glib-2.48.0/lib"
"/gnu/store/gspsx2v3zpfbl2s88xn7kb2ka9gv87bp-gstreamer-1.8.2/lib"
"/gnu/store/cc0192xs2lwcnkpjsv8msqv83d4s9zzv-gst-plugins-base-1.8.2/lib"
"/gnu/store/jz6wxyy2i279w92srv9p3qlabd7y0rjy-qtbase-5.7.0/lib"
"/gnu/store/9nifwk709wajpyfwa0jzaa3p6mf10vxs-gcc-4.9.3-lib/lib/gcc/x86_64-unknown-linux-gnu/4.9.3/../../..")

Re: [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer.

[Eric Bavier]

On 2016-08-17 06:14, David Craven wrote:
>> Is boost propagated due to header references?
> 
> The include files reference these headers:
> 
> #include <boost/type_traits.hpp>
> #include <boost/utility/enable_if.hpp>
> #include <boost/config.hpp>
> #include <boost/mpl/if.hpp>
> 
>> what is the reason for disabling runpath validation?
> 
> I'm getting a bunch of these errors. The problem seems to be that
> /gnu/store/2g5xcc8hp16nfnf37449f3p492m8i03h-qt-gstreamer-1.2.0-1.fc159e5/lib64

Can you get qt-gstreamer to install its libraries into 
"/gnu/store/...-qt-gstreamer-1.2.0-=.fc159e5/lib"?  Things might Just 
Work™ then.

-- 
`~Eric

Re: [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer.

[David Craven]

I'll withhold this patch for now pending further investigation...

Thank you!