all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH 01/24] gnu: qt: Add qtquickcontrols.
@ 2016-08-16 18:39 David Craven
  2016-08-16 18:39 ` [PATCH 02/24] gnu: qt: Add qtquickcontrols2 David Craven
                   ` (22 more replies)
  0 siblings, 23 replies; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

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

diff --git a/gnu/packages/qt.scm b/gnu/packages/qt.scm
index a12c526..ca777dd 100644
--- a/gnu/packages/qt.scm
+++ b/gnu/packages/qt.scm
@@ -802,6 +802,23 @@ developers using C++ or QML, a CSS & JavaScript like language.")
     (inputs
      `(("qtbase" ,qtbase)))))
 
+(define-public qtquickcontrols
+  (package (inherit qtsvg)
+    (name "qtquickcontrols")
+    (version "5.6.1-1")
+    (source (origin
+             (method url-fetch)
+             (uri (string-append "https://download.qt.io/official_releases/qt/"
+                                 (version-major+minor version) "/" version
+                                 "/submodules/" name "-opensource-src-"
+                                 version ".tar.xz"))
+             (sha256
+              (base32
+               "0cjzf844r7wi32ssc9vbw1a2m9hnr8c0i1p7yyljy962ifplf401"))))
+    (inputs
+     `(("qtbase" ,qtbase)
+       ("qtdeclarative" ,qtdeclarative)))))
+
 (define-public python-sip
   (package
     (name "python-sip")
-- 
2.9.0

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

* [PATCH 02/24] gnu: qt: Add qtquickcontrols2.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 18:39 ` [PATCH 03/24] gnu: qt: Add qtgraphicaleffects David Craven
                   ` (21 subsequent siblings)
  22 siblings, 0 replies; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

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

diff --git a/gnu/packages/qt.scm b/gnu/packages/qt.scm
index ca777dd..65b6d82 100644
--- a/gnu/packages/qt.scm
+++ b/gnu/packages/qt.scm
@@ -819,6 +819,23 @@ developers using C++ or QML, a CSS & JavaScript like language.")
      `(("qtbase" ,qtbase)
        ("qtdeclarative" ,qtdeclarative)))))
 
+(define-public qtquickcontrols2
+  (package (inherit qtsvg)
+    (name "qtquickcontrols2")
+    (version "5.6.1-1")
+    (source (origin
+             (method url-fetch)
+             (uri (string-append "https://download.qt.io/official_releases/qt/"
+                                 (version-major+minor version) "/" version
+                                 "/submodules/" name "-opensource-src-"
+                                 version ".tar.xz"))
+             (sha256
+              (base32
+               "0wfa2xcqsvx3zihd5nb9f9qhq0xn14c03sw1qdymzfsryqwmk4ac"))))
+    (inputs
+     `(("qtbase" ,qtbase)
+       ("qtdeclarative" ,qtdeclarative)))))
+
 (define-public python-sip
   (package
     (name "python-sip")
-- 
2.9.0

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

* [PATCH 03/24] gnu: qt: Add qtgraphicaleffects.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
  2016-08-16 18:39 ` [PATCH 02/24] gnu: qt: Add qtquickcontrols2 David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 19:29   ` Danny Milosavljevic
  2016-08-16 18:39 ` [PATCH 04/24] gnu: databases: Add lmdb David Craven
                   ` (20 subsequent siblings)
  22 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

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

diff --git a/gnu/packages/qt.scm b/gnu/packages/qt.scm
index 65b6d82..19488ac 100644
--- a/gnu/packages/qt.scm
+++ b/gnu/packages/qt.scm
@@ -836,6 +836,23 @@ developers using C++ or QML, a CSS & JavaScript like language.")
      `(("qtbase" ,qtbase)
        ("qtdeclarative" ,qtdeclarative)))))
 
+(define-public qtgraphicaleffects
+  (package (inherit qtsvg)
+    (name "qtgraphicaleffects")
+    (version "5.6.1-1")
+    (source (origin
+             (method url-fetch)
+             (uri (string-append "https://download.qt.io/official_releases/qt/"
+                                 (version-major+minor version) "/" version
+                                 "/submodules/" name "-opensource-src-"
+                                 version ".tar.xz"))
+             (sha256
+              (base32
+               "0560800fa9sd6dw1vk0ia9vq8ywdrwch2cpsi1vmh4iyxgwfr71b"))))
+    (inputs
+     `(("qtbase" ,qtbase)
+       ("qtdeclarative" ,qtdeclarative)))))
+
 (define-public python-sip
   (package
     (name "python-sip")
-- 
2.9.0

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

* [PATCH 04/24] gnu: databases: Add lmdb.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
  2016-08-16 18:39 ` [PATCH 02/24] gnu: qt: Add qtquickcontrols2 David Craven
  2016-08-16 18:39 ` [PATCH 03/24] gnu: qt: Add qtgraphicaleffects David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 18:39 ` [PATCH 05/24] gnu: xml: Add yaml-cpp David Craven
                   ` (19 subsequent siblings)
  22 siblings, 0 replies; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/databases.scm (lmdb): New variable.
---
 gnu/packages/databases.scm | 29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)

diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
index d3a44f7..0a6b267 100644
--- a/gnu/packages/databases.scm
+++ b/gnu/packages/databases.scm
@@ -57,6 +57,7 @@
   #:use-module ((guix licenses)
                 #:select (gpl2 gpl3 gpl3+ lgpl2.1+ lgpl3+ x11-style non-copyleft
                           bsd-2 bsd-3 public-domain))
+  #:use-module ((guix licenses) #:prefix license:)
   #:use-module (guix packages)
   #:use-module (guix download)
   #:use-module (guix build-system gnu)
@@ -1064,3 +1065,31 @@ trees (LSM), for sustained throughput under random insert workloads.")
   (description
     "The DB::File module provides Perl bindings to the Berkeley DB version 1.x.")
   (license (package-license perl))))
+
+(define-public lmdb
+  (package
+    (name "lmdb")
+    (version "0.9.18")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "https://github.com/LMDB/lmdb/archive/"
+                                  "LMDB_" version ".tar.gz"))
+              (sha256
+               (base32
+                "12crvzxky8in99ibh22k4ppmkgqs28yy3v7yy944za7fsrqv8dfx"))))
+    (build-system gnu-build-system)
+    (arguments
+     `(#:test-target "test"
+       #:phases
+       (modify-phases %standard-phases
+         (replace 'configure
+           (lambda* (#:key outputs #:allow-other-keys)
+             (chdir (string-append
+               (getenv "PWD") "/lmdb-LMDB_" ,version "/libraries/liblmdb"))
+               (substitute* "Makefile"
+                 (("/usr/local") (assoc-ref outputs "out")))
+            #t)))))
+    (home-page "https://symas.com/products/lightning-memory-mapped-database")
+    (synopsis "Lightning memory-mapped database library")
+    (description "Lightning memory-mapped database library.")
+    (license license:openldap2.8)))
-- 
2.9.0

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

* [PATCH 05/24] gnu: xml: Add yaml-cpp.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (2 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 04/24] gnu: databases: Add lmdb David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 19:58   ` Alex Kost
  2016-08-16 18:39 ` [PATCH 06/24] gnu: xml: Add jsoncpp David Craven
                   ` (18 subsequent siblings)
  22 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

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

diff --git a/gnu/packages/xml.scm b/gnu/packages/xml.scm
index cfdc6c0..660a0b9 100644
--- a/gnu/packages/xml.scm
+++ b/gnu/packages/xml.scm
@@ -12,6 +12,7 @@
 ;;; Copyright © 2016 Leo Famulari <leo@famulari.name>
 ;;; Copyright © 2016 Ben Woodcroft <donttrustben@gmail.com>
 ;;; Copyright © 2016 Jan Nieuwenhuizen <janneke@gnu.org>
+;;; Copyright © 2016 David Craven <david@craven.ch>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -31,6 +32,7 @@
 (define-module (gnu packages xml)
   #:use-module (gnu packages)
   #:use-module (gnu packages autotools)
+  #:use-module (gnu packages boost)
   #:use-module (gnu packages compression)
   #:use-module (gnu packages gnupg)
   #:use-module (gnu packages perl)
@@ -808,3 +810,25 @@ XSLT and EXSLT.")
 XLSM) format spreadsheets into plaintext @dfn{comma separated values} (CSV)
 files.  It is designed to be fast and to handle large input files.")
     (license license:gpl2+)))
+
+(define-public yaml-cpp
+  (package
+    (name "yaml-cpp")
+    (version "0.5.3")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                "https://github.com/jbeder/yaml-cpp/archive/"
+                "yaml-cpp-" version ".tar.gz"))
+              (sha256
+               (base32
+                "1vk6pjh0f5k6jwk2sszb9z5169whmiha9ainbdpa1arxlkq7v3b6"))))
+    (build-system cmake-build-system)
+    (inputs
+     `(("boost" ,boost)))
+    (native-inputs
+     `(("python" ,python)))
+    (home-page "https://github.com/jbeder/yaml-cpp")
+    (synopsis "YAML parser and emitter in C++ matching the YAML 1.2 spec")
+    (description "YAML parser and emitter in C++ matching the YAML 1.2 spec.")
+    (license license:bsd-3)))
-- 
2.9.0

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

* [PATCH 06/24] gnu: xml: Add jsoncpp.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (3 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 05/24] gnu: xml: Add yaml-cpp David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 19:28   ` Danny Milosavljevic
  2016-08-16 20:00   ` Alex Kost
  2016-08-16 18:39 ` [PATCH 07/24] gnu: compression: Add snappy David Craven
                   ` (17 subsequent siblings)
  22 siblings, 2 replies; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/xml.scm (jsoncpp): New variable.
---
 gnu/packages/xml.scm | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/gnu/packages/xml.scm b/gnu/packages/xml.scm
index 660a0b9..72403c2 100644
--- a/gnu/packages/xml.scm
+++ b/gnu/packages/xml.scm
@@ -832,3 +832,22 @@ files.  It is designed to be fast and to handle large input files.")
     (synopsis "YAML parser and emitter in C++ matching the YAML 1.2 spec")
     (description "YAML parser and emitter in C++ matching the YAML 1.2 spec.")
     (license license:bsd-3)))
+
+(define-public jsoncpp
+  (package
+    (name "jsoncpp")
+    (version "1.7.4")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                "https://github.com/open-source-parsers/jsoncpp/archive/"
+                version ".tar.gz"))
+              (file-name (string-append name "-" version ".tar.gz"))
+              (sha256
+               (base32
+                "0sgp6nc4c6pfn92f369v08zdwpqswn9j2ihy59bpwwl0grkx1p0h"))))
+    (build-system cmake-build-system)
+    (home-page "https://github.com/open-source-parsers/jsoncpp")
+    (synopsis "C++ library for interacting with JSON")
+    (description "C++ library for interacting with JSON.")
+    (license license:expat)))
-- 
2.9.0

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

* [PATCH 07/24] gnu: compression: Add snappy.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (4 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 06/24] gnu: xml: Add jsoncpp David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 19:45   ` Alex Kost
  2016-08-16 18:39 ` [PATCH 08/24] gnu: xorg: Add lndir David Craven
                   ` (16 subsequent siblings)
  22 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/compression.scm (snappy): New variable.
---
 gnu/packages/compression.scm | 36 ++++++++++++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)

diff --git a/gnu/packages/compression.scm b/gnu/packages/compression.scm
index e63c1af..cd1866b 100644
--- a/gnu/packages/compression.scm
+++ b/gnu/packages/compression.scm
@@ -11,6 +11,7 @@
 ;;; Copyright © 2016 Ben Woodcroft <donttrustben@gmail.com>
 ;;; Copyright © 2016 Danny Milosavljevic <dannym@scratchpost.org>
 ;;; Copyright © 2016 Tobias Geerinckx-Rice <me@tobias.gr>
+;;; Copyright © 2016 David Craven <david@craven.ch>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -871,3 +872,38 @@ even LZMA can provide, or a higher speed than gzip while compressing as
 well as bzip2.")
     (license (list license:gpl3+
                    license:public-domain)))) ; most files in lzma/
+
+(define-public snappy
+  (package
+    (name "snappy")
+    (version "1.1.3")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                "https://github.com/google/snappy/archive/"
+                version ".tar.gz"))
+              (file-name (string-append name "-" version ".tar.gz"))
+              (sha256
+               (base32
+                "04lc9n1fr4kak8yd68v61w561zcbykfyj9f745smva8hcn0hyk7r"))))
+    (build-system gnu-build-system)
+    (native-inputs
+     `(("autoconf" ,autoconf)
+       ("automake" ,automake)
+       ("libtool" ,libtool)
+       ("pkg-config" ,pkg-config)))
+    (arguments
+     `(#:phases
+       (modify-phases %standard-phases
+         (add-after 'unpack 'autoconf
+           (lambda _
+             (zero? (system* "autoreconf" "-vfi")))))))
+    (home-page "https://github.com/open-source-parsers/jsoncpp")
+    (synopsis "Fast compressor/decompressor")
+    (description "Snappy is a compression/decompression library. It does not
+aim for maximum compression, or compatibility with any other compression library;
+instead, it aims for very high speeds and reasonable compression. For instance,
+compared to the fastest mode of zlib, Snappy is an order of magnitude faster
+for most inputs, but the resulting compressed files are anywhere from 20% to
+100% bigger.")
+    (license license:asl2.0)))
-- 
2.9.0

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

* [PATCH 08/24] gnu: xorg: Add lndir.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (5 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 07/24] gnu: compression: Add snappy David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 19:53   ` Alex Kost
  2016-08-16 18:39 ` [PATCH 09/24] gnu: freedesktop: Add wayland-protocols David Craven
                   ` (15 subsequent siblings)
  22 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/xorg.scm (lndir): New variable.
---
 gnu/packages/xorg.scm | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/gnu/packages/xorg.scm b/gnu/packages/xorg.scm
index b24e437..1e7da68 100644
--- a/gnu/packages/xorg.scm
+++ b/gnu/packages/xorg.scm
@@ -133,6 +133,31 @@ projects.  Software developers are encouraged to migrate software to the GNU
 autotools system.")
     (license license:x11)))
 
+(define-public lndir
+  (package
+    (name "lndir")
+    (version "1.0.3")
+    (source
+      (origin
+        (method url-fetch)
+        (uri (string-append
+                "mirror://xorg/individual/util/lndir-"
+                version
+                ".tar.bz2"))
+        (sha256
+         (base32
+          "0pdngiy8zdhsiqx2am75yfcl36l7kd7d7nl0rss8shcdvsqgmx29"))))
+    (build-system gnu-build-system)
+    (native-inputs
+     `(("pkg-config" ,pkg-config)))
+    (inputs
+     `(("xproto" ,xproto)))
+    (home-page "http://www.x.org")
+    (synopsis "Symlink directory into tree")
+    (description "Create a shadow directory of symbolic links to another
+directory tree.")
+    (license license:x11)))
+
 (define-public bdftopcf
   (package
     (name "bdftopcf")
-- 
2.9.0

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

* [PATCH 09/24] gnu: freedesktop: Add wayland-protocols.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (6 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 08/24] gnu: xorg: Add lndir David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 19:51   ` Danny Milosavljevic
  2016-08-16 20:02   ` Alex Kost
  2016-08-16 18:39 ` [PATCH 10/24] gnu: networkmanager-qt: Propagate network-manager David Craven
                   ` (14 subsequent siblings)
  22 siblings, 2 replies; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/freedesktop.scm (wayland-protocols): New variable.
---
 gnu/packages/freedesktop.scm | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm
index 0262d0a..a24d8fa 100644
--- a/gnu/packages/freedesktop.scm
+++ b/gnu/packages/freedesktop.scm
@@ -295,6 +295,28 @@ application, or a wayland client itself.  The clients can be traditional
 applications, X servers (rootless or fullscreen) or other display servers.")
     (license license:x11)))
 
+(define-public wayland-protocols
+  (package
+    (name "wayland-protocols")
+    (version "1.4")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                "https://wayland.freedesktop.org/releases/"
+                "wayland-protocols-" version ".tar.xz"))
+              (sha256
+               (base32
+                "0wpm7mz7ww6nn3vrgz7a9iyk7mk6za73wnq0n54lzl8yq8irljh1"))))
+    (build-system gnu-build-system)
+    (inputs
+     `(("wayland" ,wayland)))
+    (native-inputs
+     `(("pkg-config" ,pkg-config)))
+    (synopsis "Wayland protocols")
+    (description "Contains Xml definitions of the wayland protocols.")
+    (home-page "https://wayland.freedesktop.org")
+    (license license:expat)))
+
 (define-public exempi
   (package
     (name "exempi")
-- 
2.9.0

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

* [PATCH 10/24] gnu: networkmanager-qt: Propagate network-manager.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (7 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 09/24] gnu: freedesktop: Add wayland-protocols David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 20:07   ` Alex Kost
  2016-08-16 18:39 ` [PATCH 11/24] gnu: modemmanager-qt: Propagate modem-manager David Craven
                   ` (13 subsequent siblings)
  22 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/kde-frameworks.scm (networkmanager-qt)
  [propagated-inputs]: Add NETWORK-MANAGER.
  [inputs]: Remove NETWORK-MANAGER.
---
 gnu/packages/kde-frameworks.scm | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm
index dbec5a3..61ed4aa 100644
--- a/gnu/packages/kde-frameworks.scm
+++ b/gnu/packages/kde-frameworks.scm
@@ -934,9 +934,10 @@ messages.")
       `(("extra-cmake-modules" ,extra-cmake-modules)
         ("dbus" ,dbus)
         ("pkg-config" ,pkg-config)))
+     (propagated-inputs
+      `(("network-manager", network-manager)))
      (inputs
-      `(("network-manager", network-manager)
-        ("qtbase" ,qtbase)))
+      `(("qtbase" ,qtbase)))
      (arguments
       `(#:phases
          (modify-phases %standard-phases
-- 
2.9.0

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

* [PATCH 11/24] gnu: modemmanager-qt: Propagate modem-manager.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (8 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 10/24] gnu: networkmanager-qt: Propagate network-manager David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 18:39 ` [PATCH 12/24] gnu: gnome: Add mobile-broadband-provider-info David Craven
                   ` (12 subsequent siblings)
  22 siblings, 0 replies; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/kde-frameworks.scm (modemmanager-qt)
  [propagated-inputs]: Add MODEM-MANAGER.
  [inputs]: Remove MODEM-MANAGER.
---
 gnu/packages/kde-frameworks.scm | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm
index 61ed4aa..e08b8cf 100644
--- a/gnu/packages/kde-frameworks.scm
+++ b/gnu/packages/kde-frameworks.scm
@@ -898,9 +898,10 @@ lower level classes for interaction with the X Windowing System.")
      `(("extra-cmake-modules" ,extra-cmake-modules)
        ("dbus" ,dbus)
        ("pkg-config" ,pkg-config)))
+    (propagated-inputs
+     `(("modem-manager", modem-manager)))
     (inputs
-     `(("modem-manager", modem-manager)
-       ("qtbase" ,qtbase)))
+     `(("qtbase" ,qtbase)))
     (arguments
      `(#:phases
         (modify-phases %standard-phases
-- 
2.9.0

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

* [PATCH 12/24] gnu: gnome: Add mobile-broadband-provider-info.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (9 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 11/24] gnu: modemmanager-qt: Propagate modem-manager David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 19:53   ` Danny Milosavljevic
  2016-08-17  9:33   ` Mark H Weaver
  2016-08-16 18:39 ` [PATCH 13/24] gnu: polkit-qt: Use modular qt David Craven
                   ` (11 subsequent siblings)
  22 siblings, 2 replies; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/gnome.scm (mobile-broadband-provider-info): New variable.
---
 gnu/packages/gnome.scm | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
index 4a0be01..7bc0445 100644
--- a/gnu/packages/gnome.scm
+++ b/gnu/packages/gnome.scm
@@ -21,6 +21,7 @@
 ;;; Copyright © 2016 Leo Famulari <leo@famulari.name>
 ;;; Copyright © 2016 Alex Griffin <a@ajgrf.com>
 ;;; Copyright © 2016 ng0 <ng0@we.make.ritual.n0.is>
+;;; Copyright © 2016 David Craven <david@craven.ch>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -4460,6 +4461,27 @@ devices, and provides VPN integration with a variety of different VPN
 services.")
     (license license:gpl2+)))
 
+(define-public mobile-broadband-provider-info
+  (package
+    (name "mobile-broadband-provider-info")
+    (version "20151214")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                "mirror://gnome/sources/"
+                "mobile-broadband-provider-info/" version "/"
+                "mobile-broadband-provider-info-" version ".tar.xz"))
+              (sha256
+               (base32
+                "1905nab1h8p4hx0m1w0rn4mkg9209x680dcr4l77bngy21pmvr4a"))))
+    (build-system gnu-build-system)
+    (arguments
+     `(#:tests? #f)) ; No tests
+    (home-page "https://wiki.gnome.org/Projects/NetworkManager")
+    (synopsis "Datbase of broadband connection configuration")
+    (description "Datbase of broadband connection configuration.")
+    (license license:public-domain)))
+
 (define-public network-manager-applet
   (package
     (name "network-manager-applet")
-- 
2.9.0

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

* [PATCH 13/24] gnu: polkit-qt: Use modular qt.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (10 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 12/24] gnu: gnome: Add mobile-broadband-provider-info David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 19:17   ` Danny Milosavljevic
  2016-08-16 18:39 ` [PATCH 14/24] gnu: gstreamer: Use license: prefix David Craven
                   ` (10 subsequent siblings)
  22 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/polkit.scm (polkit-qt)[propagated-inputs]: Use QTBASE instead
  of QT.
---
 gnu/packages/polkit.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/polkit.scm b/gnu/packages/polkit.scm
index 08b753a..e224ca2 100644
--- a/gnu/packages/polkit.scm
+++ b/gnu/packages/polkit.scm
@@ -142,7 +142,7 @@ for unprivileged applications.")
     (inputs
      `(("polkit" ,polkit)))
     (propagated-inputs
-     `(("qt" ,qt))) ; qt-4 according to the pkg-config files
+     `(("qtbase" ,qtbase)))
     (native-inputs
      `(("pkg-config" ,pkg-config)))
     (arguments
-- 
2.9.0

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

* [PATCH 14/24] gnu: gstreamer: Use license: prefix.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (11 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 13/24] gnu: polkit-qt: Use modular qt David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 19:18   ` Danny Milosavljevic
  2016-08-16 20:15   ` Alex Kost
  2016-08-16 18:39 ` [PATCH 15/24] gnu: gstreamer: Symlink gstconfig.h into include directory David Craven
                   ` (9 subsequent siblings)
  22 siblings, 2 replies; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/gstreamer.scm (define-module): Import guix licenses with #:prefix
  license:.
  (orc): Likewise.
  (gstreamer): Likewise.
  (gst-plugins-base): Likewise.
  (gst-plugins-good): Likewise.
  (gst-plugins-bad): Likewise.
  (gst-plugins-ugly): Likewise.
  (gst-libav): Likewise.
  (python-gst): Likewise.
---
 gnu/packages/gstreamer.scm | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index a39a3b4..bd99880 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -21,7 +21,7 @@
 ;;; along with GNU Guix.  If not, see <http://www.gnu.org/licenses/>.
 
 (define-module (gnu packages gstreamer)
-  #:use-module ((guix licenses) #:select (lgpl2.0+ lgpl2.1+ bsd-2 bsd-3 gpl2+))
+  #:use-module ((guix licenses) #:prefix license:)
   #:use-module (guix packages)
   #:use-module (guix download)
   #:use-module (guix build-system gnu)
@@ -92,7 +92,7 @@ associated tools for compiling and executing simple programs that operate on
 arrays of data.")
     ;; The source code implementing the Marsenne Twister algorithm is licensed
     ;; under the 3-clause BSD license, the rest is under 2-clause BSD license.
-    (license (list bsd-2 bsd-3))))
+    (license (list license:bsd-2 license:bsd-3))))
 
 (define-public gstreamer
   (package
@@ -140,7 +140,7 @@ transparently.  Developers can add new codecs and filters by writing a
 simple plugin with a clean, generic interface.
 
 This package provides the core library and elements.")
-    (license lgpl2.0+)))
+    (license license:lgpl2.0+)))
 
 (define-public gst-plugins-base
   (package
@@ -192,7 +192,7 @@ This package provides the core library and elements.")
      "Plugins for the GStreamer multimedia library")
     (description "This package provides an essential exemplary set of plug-ins
 for the GStreamer multimedia library.")
-    (license lgpl2.0+)))
+    (license license:lgpl2.0+)))
 
 
 (define-public gst-plugins-good
@@ -258,7 +258,7 @@ for the GStreamer multimedia library.")
     (description "GStreamer Good Plug-ins is a set of plug-ins for the
 GStreamer multimedia library.  This set contains those plug-ins which the
 developers consider to have good quality code and correct functionality.")
-    (license lgpl2.0+)))
+    (license license:lgpl2.0+)))
 
 (define-public gst-plugins-bad
   (package
@@ -331,7 +331,7 @@ developers consider to have good quality code and correct functionality.")
     (description
      "GStreamer Bad Plug-ins is a set of plug-ins whose quality aren't up to
 par compared to the rest.")
-    (license lgpl2.0+)))
+    (license license:lgpl2.0+)))
 
 (define-public gst-plugins-ugly
   (package
@@ -369,7 +369,7 @@ par compared to the rest.")
     (description "GStreamer Ugly Plug-ins.  This set contains those plug-ins
 which the developers consider to have good quality code but that might pose
 distribution problems in some jurisdictions, e.g. due to patent threats.")
-    (license lgpl2.0+)))
+    (license license:lgpl2.0+)))
 
 (define-public gst-libav
   (package
@@ -406,7 +406,7 @@ distribution problems in some jurisdictions, e.g. due to patent threats.")
     (description
      "This GStreamer plugin supports a large number of audio and video
 compression formats through the use of the libav library.")
-    (license gpl2+)))
+    (license license:gpl2+)))
 
 (define-public python-gst
   (package
@@ -446,7 +446,7 @@ compression formats through the use of the libav library.")
     (description
      "This package contains GObject Introspection overrides for Python that can
 be used by Python applications using GStreamer.")
-    (license lgpl2.1+)
+    (license license:lgpl2.1+)
     (properties `((python2-variant . ,(delay python2-gst))))))
 
 (define-public python2-gst
-- 
2.9.0

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

* [PATCH 15/24] gnu: gstreamer: Symlink gstconfig.h into include directory.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (12 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 14/24] gnu: gstreamer: Use license: prefix David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-17  9:28   ` Mark H Weaver
  2016-08-16 18:39 ` [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer David Craven
                   ` (8 subsequent siblings)
  22 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/gstreamer.scm (gstreamer)[arguments]: Add symlink-gstconfig.h
  phase.
---
 gnu/packages/gstreamer.scm | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index bd99880..54919cd 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -113,7 +113,16 @@ arrays of data.")
      `(#:configure-flags
        (list (string-append "--with-html-dir="
                             (assoc-ref %outputs "doc")
-                            "/share/gtk-doc/html"))))
+                            "/share/gtk-doc/html"))
+       #:phases
+        (modify-phases %standard-phases
+          (add-after 'install 'symlink-gstconfig.h
+            (lambda* (#:key outputs #:allow-other-keys)
+              (symlink
+                (string-append (assoc-ref outputs "out")
+                 "/lib/gstreamer-1.0/include/gst/gstconfig.h")
+                (string-append (assoc-ref outputs "out")
+                 "/include/gstreamer-1.0/gst/gstconfig.h")))))))
     (propagated-inputs `(("glib" ,glib))) ; required by gstreamer-1.0.pc.
     (native-inputs
      `(("bison" ,bison)
-- 
2.9.0

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

* [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (13 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 15/24] gnu: gstreamer: Symlink gstconfig.h into include directory David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 21:08   ` Leo Famulari
  2016-08-16 22:28   ` Eric Bavier
  2016-08-16 18:39 ` [PATCH 17/24] gnu: kwidgetsaddons: Fix test failure David Craven
                   ` (7 subsequent siblings)
  22 siblings, 2 replies; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: 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

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

* [PATCH 17/24] gnu: kwidgetsaddons: Fix test failure.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (14 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 19:22   ` Danny Milosavljevic
  2016-08-16 18:39 ` [PATCH 18/24] gnu: qsynth: Use modular qt David Craven
                   ` (6 subsequent siblings)
  22 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/kde-frameworks.scm (kwidgetsaddons)[arguments]:
  Enable tests. Set Xvfb pixel depth to 24 bits.
---
 gnu/packages/kde-frameworks.scm | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm
index e08b8cf..113257e 100644
--- a/gnu/packages/kde-frameworks.scm
+++ b/gnu/packages/kde-frameworks.scm
@@ -817,19 +817,19 @@ represented by a QPoint or a QSize.")
     (inputs
      `(("qtbase" ,qtbase)))
     (arguments
-     `(#:tests? #f ; FIXME: libGL error: failed to load driver: swrast.
-       #:phases
+     `(#:phases
         (modify-phases %standard-phases
           (add-before 'check 'check-setup
             (lambda* _
-              (setenv "CTEST_OUTPUT_ON_FAILURE" "1") ; enable debug output
-              (setenv "LIBGL_DEBUG" "verbose") ; enable debug output
               (setenv "DBUS_FATAL_WARNINGS" "0")))
           (add-before 'check 'start-xorg-server
             (lambda* (#:key inputs #:allow-other-keys)
               ;; The test suite requires a running X server.
+              ;; Xvfb doesn't have proper glx support and needs a pixeldepth
+              ;; of 24 bit to avoid "libGL error: failed to load driver: swrast"
+              ;;                    "Could not initialize GLX"
               (system (string-append (assoc-ref inputs "xorg-server")
-                                     "/bin/Xvfb :1 &"))
+                                     "/bin/Xvfb :1 -screen 0 640x480x24 &"))
               (setenv "DISPLAY" ":1")
              #t)))))
     (home-page "https://community.kde.org/Frameworks")
-- 
2.9.0

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

* [PATCH 18/24] gnu: qsynth: Use modular qt.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (15 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 17/24] gnu: kwidgetsaddons: Fix test failure David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 19:16   ` Danny Milosavljevic
  2016-08-16 18:39 ` [PATCH 19/24] gnu: fritzing: " David Craven
                   ` (5 subsequent siblings)
  22 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/audio.scm (qsynth)[inputs]: Add QTBASE, QTX11EXTRAS. Remove QT.
  [native-inputs]: Add QTTOOLS.
---
 gnu/packages/audio.scm | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
index 5c42aaa..a21cd5b 100644
--- a/gnu/packages/audio.scm
+++ b/gnu/packages/audio.scm
@@ -2112,9 +2112,12 @@ interface.")
     (build-system gnu-build-system)
     (arguments
      `(#:tests? #f)) ; no "check" phase
+    (native-inputs
+     `(("qttools" ,qttools)))
     (inputs
-     `(("qt" ,qt)
-       ("fluidsynth" ,fluidsynth)))
+     `(("fluidsynth" ,fluidsynth)
+       ("qtbase" ,qtbase)
+       ("qtx11extras" ,qtx11extras)))
     (home-page "http://qsynth.sourceforge.net")
     (synopsis "Graphical user interface for FluidSynth")
     (description
-- 
2.9.0

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

* [PATCH 19/24] gnu: fritzing: Use modular qt.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (16 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 18/24] gnu: qsynth: Use modular qt David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 19:17   ` Danny Milosavljevic
  2016-08-16 18:39 ` [PATCH 20/24] gnu: bitcoin-core: " David Craven
                   ` (4 subsequent siblings)
  22 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/engineering.scm (fritzing)[inputs]: Add QTBASE, QTSERIALPORT,
  QTSVG. Remove QT.
---
 gnu/packages/engineering.scm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/engineering.scm b/gnu/packages/engineering.scm
index f6c3d5f..dad38e0 100644
--- a/gnu/packages/engineering.scm
+++ b/gnu/packages/engineering.scm
@@ -388,7 +388,9 @@ multipole-accelerated algorithm.")
                                                  (assoc-ref outputs "out"))
                                   "phoenix.pro"))))))))
     (inputs
-     `(("qt" ,qt)
+     `(("qtbase" ,qtbase)
+       ("qtserialport" ,qtserialport)
+       ("qtsvg" ,qtsvg)
        ("boost" ,boost)
        ("zlib" ,zlib)
        ("fritzing-parts-db"
-- 
2.9.0

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

* [PATCH 20/24] gnu: bitcoin-core: Use modular qt.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (17 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 19/24] gnu: fritzing: " David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 19:23   ` Danny Milosavljevic
  2016-08-16 18:39 ` [PATCH 21/24] gnu: gst-plugins-bad: " David Craven
                   ` (3 subsequent siblings)
  22 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/finance.scm (bitcoin-core)[inputs]: Add QTBASE. Remove QT.
---
 gnu/packages/finance.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/finance.scm b/gnu/packages/finance.scm
index 179e325..21a2276 100644
--- a/gnu/packages/finance.scm
+++ b/gnu/packages/finance.scm
@@ -67,7 +67,7 @@
        ("miniupnpc" ,miniupnpc)
        ("openssl" ,openssl)
        ("protobuf" ,protobuf)
-       ("qt" ,qt)))
+       ("qtbase" ,qtbase)))
     (arguments
      `(#:configure-flags
         (list
-- 
2.9.0

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

* [PATCH 21/24] gnu: gst-plugins-bad: Use modular qt.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (18 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 20/24] gnu: bitcoin-core: " David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 19:23   ` Danny Milosavljevic
  2016-08-16 18:39 ` [PATCH 22/24] gnu: poppler-qt5: " David Craven
                   ` (2 subsequent siblings)
  22 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/gstreamer.scm (gst-plugins-bad)[inputs]: Add QTBASE. Remove QT.
---
 gnu/packages/gstreamer.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
index ee3c9e4..0d8a74d 100644
--- a/gnu/packages/gstreamer.scm
+++ b/gnu/packages/gstreamer.scm
@@ -334,7 +334,7 @@ developers consider to have good quality code and correct functionality.")
        ("openssl" ,openssl)
        ("opus" ,opus)
        ("orc" ,orc)
-       ("qt" ,qt)
+       ("qtbase" ,qtbase)
        ("soundtouch" ,soundtouch)
        ("wayland" ,wayland)))
     (home-page "http://gstreamer.freedesktop.org/")
-- 
2.9.0

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

* [PATCH 22/24] gnu: poppler-qt5: Use modular qt.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (19 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 21/24] gnu: gst-plugins-bad: " David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 19:24   ` Danny Milosavljevic
  2016-08-16 18:39 ` [PATCH 23/24] gnu: qtkeychain: " David Craven
  2016-08-16 18:39 ` [PATCH 24/24] gnu: qt: Update qt to 5.7.0 David Craven
  22 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/pdf.scm (poppler-qt)[inputs]: Add QTBASE. Remove QT.
---
 gnu/packages/pdf.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/pdf.scm b/gnu/packages/pdf.scm
index 439adb9..4b08048 100644
--- a/gnu/packages/pdf.scm
+++ b/gnu/packages/pdf.scm
@@ -118,7 +118,7 @@
 (define-public poppler-qt5
   (package (inherit poppler)
    (name "poppler-qt5")
-   (inputs `(("qt" ,qt)
+   (inputs `(("qtbase" ,qtbase)
              ,@(package-inputs poppler)))
    (synopsis "Qt5 frontend for the Poppler PDF rendering library")))
 
-- 
2.9.0

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

* [PATCH 23/24] gnu: qtkeychain: Use modular qt.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (20 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 22/24] gnu: poppler-qt5: " David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 19:25   ` Danny Milosavljevic
  2016-08-16 18:39 ` [PATCH 24/24] gnu: qt: Update qt to 5.7.0 David Craven
  22 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/qt.scm (qtkeychain)[inputs]: Add QTBASE. Remove QT.
  [native-inputs]: Add QTTOOLS.
---
 gnu/packages/qt.scm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/qt.scm b/gnu/packages/qt.scm
index 19488ac..021bb70 100644
--- a/gnu/packages/qt.scm
+++ b/gnu/packages/qt.scm
@@ -1070,8 +1070,10 @@ contain over 620 classes.")
         (sha256
          (base32 "0fka5q5cdzlf79igcjgbnb2smvwbwfasqawkzkbr34whispgm6lz"))))
     (build-system cmake-build-system)
+    (native-inputs
+     `(("qttools" ,qttools)))
     (inputs
-     `(("qt" ,qt)))
+     `(("qtbase" ,qtbase)))
     (arguments
      `(#:tests? #f ; No tests included
        #:phases
-- 
2.9.0

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

* [PATCH 24/24] gnu: qt: Update qt to 5.7.0.
  2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
                   ` (21 preceding siblings ...)
  2016-08-16 18:39 ` [PATCH 23/24] gnu: qtkeychain: " David Craven
@ 2016-08-16 18:39 ` David Craven
  2016-08-16 19:41   ` Danny Milosavljevic
  2016-08-19  0:28   ` Mark H Weaver
  22 siblings, 2 replies; 108+ messages in thread
From: David Craven @ 2016-08-16 18:39 UTC (permalink / raw)
  To: guix-devel; +Cc: David Craven

* gnu/packages/qt.scm
  (qtbase): Update it.
  (qtsvg): Update it.
  (qtimageformats): Update it.
  (qtx11extras): Update it.
  (qtxmlpatterns): Update it.
  (qtdeclarative): Update it.
  (qtconnectivity): Update it.
  (qtwebsockets): Update it.
  (qtsensors): Update it.
  (qtmultimedia): Update it.
  (qtwayland): Update it.
  (qtserialport): Update it.
  (qtwebchannel): Update it.
  (qtlocation): Update it.
  (qttools): Update it.
  (qtscript): Update it.
  (qtquickcontrols): Update it.
  (qtquickcontrols2): Update it.
  (qtgraphicaleffects): Update it.
---
 gnu/packages/qt.scm | 78 ++++++++++++++++++++++++++---------------------------
 1 file changed, 39 insertions(+), 39 deletions(-)

diff --git a/gnu/packages/qt.scm b/gnu/packages/qt.scm
index 021bb70..a7eee7c 100644
--- a/gnu/packages/qt.scm
+++ b/gnu/packages/qt.scm
@@ -313,7 +313,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
 (define-public qtbase
   (package
     (name "qtbase")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -322,7 +322,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "0fbwprlhqmdyhh2wb9122fcpq7pbil530iak482b9sy5gqs7i5ij"))
+               "0ip6xnizsn269r4s1nq9lkx8cdxkjqr1fidwrj3sa8xb7h96syry"))
              (modules '((guix build utils)))
              (snippet
               '(begin
@@ -484,7 +484,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
 (define-public qtsvg
   (package (inherit qtbase)
     (name "qtsvg")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -493,7 +493,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "1w0jvhgaiddafcms2nv8wl1klg07lncmjwm1zhdw3l6rxi9071sw"))))
+               "10fqrlqkiq83xhx79g8d2sjy7hjdnp28067z8f4byj7db81rzy51"))))
     (propagated-inputs `())
     (native-inputs `(("perl" ,perl)))
     (inputs
@@ -518,7 +518,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
 (define-public qtimageformats
   (package (inherit qtsvg)
     (name "qtimageformats")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -527,7 +527,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "1p98acvsm3azka2by1ph4gdb31qbnndrr5k5wns4xk2d760y8ifc"))))
+               "1rb27x7i2pmvsck6wax2cg31gqpzaakciy45wm5l3lcl86j48czg"))))
     (native-inputs `())
     (inputs
      `(("libmng" ,libmng)
@@ -540,7 +540,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
 (define-public qtx11extras
   (package (inherit qtsvg)
     (name "qtx11extras")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -549,7 +549,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "0yj5yg2dqkrwbgbicmk2rpqsagmi8dsffkrprpsj0fmkx4awhv5y"))))
+               "1yrkn8pqdbvbqykas3wx1vdfimhjkgx3s5jgdxib9dgmgyx6vjzw"))))
     (native-inputs `(("perl" ,perl)))
     (inputs
      `(("mesa" ,mesa)
@@ -558,7 +558,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
 (define-public qtxmlpatterns
   (package (inherit qtsvg)
     (name "qtxmlpatterns")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -567,14 +567,14 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "1966rrk7f6c55k57j33rffdjs77kk4mawrnnl8yv1ckcirxc3np1"))))
+               "02z2qxamslg6sphnaykjcjfpypq4b69pb586s43vw4fplm72m21q"))))
     (native-inputs `(("perl" ,perl)))
     (inputs `(("qtbase" ,qtbase)))))
 
 (define-public qtdeclarative
   (package (inherit qtsvg)
     (name "qtdeclarative")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -583,7 +583,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "094gx5mzqzcga97y7ihf052b6i5iv512lh7m0702m5q94nsn1pqw"))))
+               "1x7rij423g5chlfd2kix54f393vxwjvdfsn1c7sybqmfycwn5pl6"))))
     (native-inputs
      `(("perl" ,perl)
        ("pkg-config" ,pkg-config)
@@ -597,7 +597,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
 (define-public qtconnectivity
   (package (inherit qtsvg)
     (name "qtconnectivity")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -606,7 +606,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "0sr6sxp0q45pacs25knr28139xdrphcjgrwlksdhdpsryfw19mzi"))))
+               "00r7lc1w3snfp2qfqmviqzv0cw16zd8m1sfpvxvpl65yqmzcli4q"))))
     (native-inputs
      `(("perl" ,perl)
        ("pkg-config" ,pkg-config)
@@ -618,7 +618,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
 (define-public qtwebsockets
   (package (inherit qtsvg)
     (name "qtwebsockets")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -627,7 +627,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "1fz0x8570zxc00a22skd848svma3p2g3xyxj14jq10559jihqqil"))))
+               "0hwb2l7iwf4wf7l95dli8j3b7h0nffp56skfg1x810kzj0df26vl"))))
     (native-inputs
      `(("perl" ,perl)
        ("qtdeclarative" ,qtdeclarative)))
@@ -636,7 +636,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
 (define-public qtsensors
   (package (inherit qtsvg)
     (name "qtsensors")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -645,7 +645,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "0kcrvf6vzn6g2v2m70f9r3raalzmfp48rwjlqhss3w84jfz3y04r"))))
+               "1gii6wg2xd3bkb86y5hgpmwcpl04xav030zscpl6fhscl9kcqg98"))))
     (native-inputs
      `(("perl" ,perl)
        ("qtdeclarative" ,qtdeclarative)))
@@ -654,7 +654,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
 (define-public qtmultimedia
   (package (inherit qtsvg)
     (name "qtmultimedia")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -663,7 +663,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "0paffx0614ivjbf87lr9klpbqik6r1pzbc14l41np6d9jv3dqa2f"))))
+               "0ndmhiflmyr144nq8drd5njsdi282ixsm4730q5n0ji2v9dp1bh5"))))
     (native-inputs
      `(("perl" ,perl)
        ("pkg-config" ,pkg-config)
@@ -678,7 +678,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
 (define-public qtwayland
   (package (inherit qtsvg)
     (name "qtwayland")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -687,7 +687,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "1fnvgpi49ilds3ah9iizxj9qhhb5rnwqd9h03bhkwf0ydywv52c4"))))
+               "04dynjcr6gxi3hcqdf688a4hkabi2l17slpcx9k0f3dxygwcgf96"))))
     (native-inputs
      `(("glib" ,glib)
        ("perl" ,perl)
@@ -709,7 +709,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
 (define-public qtserialport
   (package (inherit qtsvg)
     (name "qtserialport")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -718,7 +718,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "135cbgghxk0c6dblmyyrw6znfb9m8sac9hhyc2dm6vq7vzy8id52"))))
+               "0rc2l14s59qskp16wqlkizfai32s41qlm7a86r3qahx28gc51qaw"))))
     (native-inputs `(("perl" ,perl)))
     (inputs
      `(("qtbase" ,qtbase)
@@ -727,7 +727,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
 (define-public qtwebchannel
   (package (inherit qtsvg)
     (name "qtwebchannel")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -736,7 +736,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "10kys3ppjkj60fs1s335fdcpdsbxsjn6ibvm6zph9gqbncabd2l7"))))
+               "05lqfidlh1ahdd1j9y20p2037qbcq51zkdzj2m8fwhn7ghbwvd1s"))))
     (native-inputs
      `(("perl" ,perl)
        ("qtdeclarative" ,qtdeclarative)
@@ -746,7 +746,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
 (define-public qtlocation
   (package (inherit qtsvg)
     (name "qtlocation")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -755,18 +755,18 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "0my4pbcxa58yzvdh65l5qx99ln03chjr5c3ml5v37wfk7nx23k69"))))
+               "0rd898gndn41jrp78203lxd94ybfv693l0qg0myag4r46ikk69vh"))))
     (native-inputs
      `(("perl" ,perl)
        ("qtdeclarative" ,qtdeclarative)
-       ;("qtquickcontrols" ,qtquickcontrols)
+       ("qtquickcontrols" ,qtquickcontrols)
        ("qtserialport" ,qtserialport)))
     (inputs `(("qtbase" ,qtbase)))))
 
 (define-public qttools
   (package (inherit qtsvg)
     (name "qttools")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -775,7 +775,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "0haic027a2d7p7k8xz83fbvci4a4dln34360rlwgy7hlyy5m4nip"))))
+               "004m9l7bgh7qnncbyl3d5fkggdrqx58ib21xv4hflvvarxrssibg"))))
     (native-inputs
      `(("perl" ,perl)
        ("qtdeclarative" ,qtdeclarative)))
@@ -786,7 +786,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
 (define-public qtscript
   (package (inherit qtsvg)
     (name "qtscript")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -795,7 +795,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "1gini9483flqa9q4a4bl81bh7g5s408bycqykqhgbklmfd29y5lx"))))
+               "0040890p5ilyrmcpndz1hhp08x2ms5gw4lp4n5iax2a957yy2i4w"))))
     (native-inputs
      `(("perl" ,perl)
        ("qttools" ,qttools)))
@@ -805,7 +805,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
 (define-public qtquickcontrols
   (package (inherit qtsvg)
     (name "qtquickcontrols")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -814,7 +814,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "0cjzf844r7wi32ssc9vbw1a2m9hnr8c0i1p7yyljy962ifplf401"))))
+               "0cpcrmz9n5b4bgmshmk093lirl9xwqb23inchnai1zqg21vrmqfq"))))
     (inputs
      `(("qtbase" ,qtbase)
        ("qtdeclarative" ,qtdeclarative)))))
@@ -822,7 +822,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
 (define-public qtquickcontrols2
   (package (inherit qtsvg)
     (name "qtquickcontrols2")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -831,7 +831,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "0wfa2xcqsvx3zihd5nb9f9qhq0xn14c03sw1qdymzfsryqwmk4ac"))))
+               "0i8h933vhvx1bmniqdx0idg6vk82w9byd3dq0bb2phwjg5vv1xb3"))))
     (inputs
      `(("qtbase" ,qtbase)
        ("qtdeclarative" ,qtdeclarative)))))
@@ -839,7 +839,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
 (define-public qtgraphicaleffects
   (package (inherit qtsvg)
     (name "qtgraphicaleffects")
-    (version "5.6.1-1")
+    (version "5.7.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "https://download.qt.io/official_releases/qt/"
@@ -848,7 +848,7 @@ developers using C++ or QML, a CSS & JavaScript like language.")
                                  version ".tar.xz"))
              (sha256
               (base32
-               "0560800fa9sd6dw1vk0ia9vq8ywdrwch2cpsi1vmh4iyxgwfr71b"))))
+               "1rwdjg5mk6xpadmxfq64xfp573zp5lrj9illb9105ra5wff565n8"))))
     (inputs
      `(("qtbase" ,qtbase)
        ("qtdeclarative" ,qtdeclarative)))))
-- 
2.9.0

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

* Re: [PATCH 18/24] gnu: qsynth: Use modular qt.
  2016-08-16 18:39 ` [PATCH 18/24] gnu: qsynth: Use modular qt David Craven
@ 2016-08-16 19:16   ` Danny Milosavljevic
  0 siblings, 0 replies; 108+ messages in thread
From: Danny Milosavljevic @ 2016-08-16 19:16 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

LGTM

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

* Re: [PATCH 19/24] gnu: fritzing: Use modular qt.
  2016-08-16 18:39 ` [PATCH 19/24] gnu: fritzing: " David Craven
@ 2016-08-16 19:17   ` Danny Milosavljevic
  0 siblings, 0 replies; 108+ messages in thread
From: Danny Milosavljevic @ 2016-08-16 19:17 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

LGTM.

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

* Re: [PATCH 13/24] gnu: polkit-qt: Use modular qt.
  2016-08-16 18:39 ` [PATCH 13/24] gnu: polkit-qt: Use modular qt David Craven
@ 2016-08-16 19:17   ` Danny Milosavljevic
  2016-08-16 21:00     ` Leo Famulari
  0 siblings, 1 reply; 108+ messages in thread
From: Danny Milosavljevic @ 2016-08-16 19:17 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

> -     `(("qt" ,qt))) ; qt-4 according to the pkg-config files
> +     `(("qtbase" ,qtbase)))

That won't be qt-4 anymore. Is that OK?

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

* Re: [PATCH 14/24] gnu: gstreamer: Use license: prefix.
  2016-08-16 18:39 ` [PATCH 14/24] gnu: gstreamer: Use license: prefix David Craven
@ 2016-08-16 19:18   ` Danny Milosavljevic
  2016-08-16 20:15   ` Alex Kost
  1 sibling, 0 replies; 108+ messages in thread
From: Danny Milosavljevic @ 2016-08-16 19:18 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

LGTM.

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

* Re: [PATCH 17/24] gnu: kwidgetsaddons: Fix test failure.
  2016-08-16 18:39 ` [PATCH 17/24] gnu: kwidgetsaddons: Fix test failure David Craven
@ 2016-08-16 19:22   ` Danny Milosavljevic
  0 siblings, 0 replies; 108+ messages in thread
From: Danny Milosavljevic @ 2016-08-16 19:22 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

>                (system (string-append (assoc-ref inputs "xorg-server")
> -                                     "/bin/Xvfb :1 &"))
> +                                     "/bin/Xvfb :1 -screen 0 640x480x24 &"))

Might

             (setenv "QT_QPA_PLATFORM" "offscreen")

also be enough? Starting an Xvfb server and then not stopping it again is kinda ... extreme. Also, who says that :1 is free?

I can see that it was there before, but just to be sure...

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

* Re: [PATCH 20/24] gnu: bitcoin-core: Use modular qt.
  2016-08-16 18:39 ` [PATCH 20/24] gnu: bitcoin-core: " David Craven
@ 2016-08-16 19:23   ` Danny Milosavljevic
  0 siblings, 0 replies; 108+ messages in thread
From: Danny Milosavljevic @ 2016-08-16 19:23 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

LGTM!

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

* Re: [PATCH 21/24] gnu: gst-plugins-bad: Use modular qt.
  2016-08-16 18:39 ` [PATCH 21/24] gnu: gst-plugins-bad: " David Craven
@ 2016-08-16 19:23   ` Danny Milosavljevic
  0 siblings, 0 replies; 108+ messages in thread
From: Danny Milosavljevic @ 2016-08-16 19:23 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

LGTM!

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

* Re: [PATCH 22/24] gnu: poppler-qt5: Use modular qt.
  2016-08-16 18:39 ` [PATCH 22/24] gnu: poppler-qt5: " David Craven
@ 2016-08-16 19:24   ` Danny Milosavljevic
  0 siblings, 0 replies; 108+ messages in thread
From: Danny Milosavljevic @ 2016-08-16 19:24 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

LGTM!

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

* Re: [PATCH 23/24] gnu: qtkeychain: Use modular qt.
  2016-08-16 18:39 ` [PATCH 23/24] gnu: qtkeychain: " David Craven
@ 2016-08-16 19:25   ` Danny Milosavljevic
  0 siblings, 0 replies; 108+ messages in thread
From: Danny Milosavljevic @ 2016-08-16 19:25 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

LGTM!

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

* Re: [PATCH 06/24] gnu: xml: Add jsoncpp.
  2016-08-16 18:39 ` [PATCH 06/24] gnu: xml: Add jsoncpp David Craven
@ 2016-08-16 19:28   ` Danny Milosavljevic
  2016-08-16 19:59     ` Danny Milosavljevic
  2016-08-16 20:00   ` Alex Kost
  1 sibling, 1 reply; 108+ messages in thread
From: Danny Milosavljevic @ 2016-08-16 19:28 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

Might it make sense to also mention that it is optionally public domain?

Otherwise LGTM.

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

* Re: [PATCH 03/24] gnu: qt: Add qtgraphicaleffects.
  2016-08-16 18:39 ` [PATCH 03/24] gnu: qt: Add qtgraphicaleffects David Craven
@ 2016-08-16 19:29   ` Danny Milosavljevic
  0 siblings, 0 replies; 108+ messages in thread
From: Danny Milosavljevic @ 2016-08-16 19:29 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

LGTM!

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

* Re: [PATCH 24/24] gnu: qt: Update qt to 5.7.0.
  2016-08-16 18:39 ` [PATCH 24/24] gnu: qt: Update qt to 5.7.0 David Craven
@ 2016-08-16 19:41   ` Danny Milosavljevic
  2016-08-19  0:28   ` Mark H Weaver
  1 sibling, 0 replies; 108+ messages in thread
From: Danny Milosavljevic @ 2016-08-16 19:41 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

LGTM!

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

* Re: [PATCH 07/24] gnu: compression: Add snappy.
  2016-08-16 18:39 ` [PATCH 07/24] gnu: compression: Add snappy David Craven
@ 2016-08-16 19:45   ` Alex Kost
  0 siblings, 0 replies; 108+ messages in thread
From: Alex Kost @ 2016-08-16 19:45 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven (2016-08-16 21:39 +0300) wrote:

> * gnu/packages/compression.scm (snappy): New variable.
> ---
>  gnu/packages/compression.scm | 36 ++++++++++++++++++++++++++++++++++++
>  1 file changed, 36 insertions(+)
>
>
> diff --git a/gnu/packages/compression.scm b/gnu/packages/compression.scm
> index e63c1af..cd1866b 100644
> --- a/gnu/packages/compression.scm
> +++ b/gnu/packages/compression.scm
> @@ -11,6 +11,7 @@
>  ;;; Copyright © 2016 Ben Woodcroft <donttrustben@gmail.com>
>  ;;; Copyright © 2016 Danny Milosavljevic <dannym@scratchpost.org>
>  ;;; Copyright © 2016 Tobias Geerinckx-Rice <me@tobias.gr>
> +;;; Copyright © 2016 David Craven <david@craven.ch>
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -871,3 +872,38 @@ even LZMA can provide, or a higher speed than gzip while compressing as
>  well as bzip2.")
>      (license (list license:gpl3+
>                     license:public-domain)))) ; most files in lzma/
> +
> +(define-public snappy
> +  (package
> +    (name "snappy")
> +    (version "1.1.3")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append
> +                "https://github.com/google/snappy/archive/"
> +                version ".tar.gz"))
> +              (file-name (string-append name "-" version ".tar.gz"))
> +              (sha256
> +               (base32
> +                "04lc9n1fr4kak8yd68v61w561zcbykfyj9f745smva8hcn0hyk7r"))))
> +    (build-system gnu-build-system)
> +    (native-inputs
> +     `(("autoconf" ,autoconf)
> +       ("automake" ,automake)
> +       ("libtool" ,libtool)
> +       ("pkg-config" ,pkg-config)))
> +    (arguments
> +     `(#:phases
> +       (modify-phases %standard-phases
> +         (add-after 'unpack 'autoconf
> +           (lambda _
> +             (zero? (system* "autoreconf" "-vfi")))))))
> +    (home-page "https://github.com/open-source-parsers/jsoncpp")

I think you meant "https://github.com/google/snappy"

Also there is a real release that shouldn't require autoreconf phase:
<https://github.com/google/snappy/releases/download/1.1.3/snappy-1.1.3.tar.gz>
Did you try using it?

> +    (synopsis "Fast compressor/decompressor")
> +    (description "Snappy is a compression/decompression library. It does not
> +aim for maximum compression, or compatibility with any other compression library;
> +instead, it aims for very high speeds and reasonable compression. For instance,
> +compared to the fastest mode of zlib, Snappy is an order of magnitude faster
> +for most inputs, but the resulting compressed files are anywhere from 20% to
> +100% bigger.")
> +    (license license:asl2.0)))

-- 
Alex

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

* Re: [PATCH 09/24] gnu: freedesktop: Add wayland-protocols.
  2016-08-16 18:39 ` [PATCH 09/24] gnu: freedesktop: Add wayland-protocols David Craven
@ 2016-08-16 19:51   ` Danny Milosavljevic
  2016-08-16 20:02   ` Alex Kost
  1 sibling, 0 replies; 108+ messages in thread
From: Danny Milosavljevic @ 2016-08-16 19:51 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

> +    (inputs
> +     `(("wayland" ,wayland)))
> +    (native-inputs
> +     `(("pkg-config" ,pkg-config)))
> +    (synopsis "Wayland protocols")
> +    (description "Contains Xml definitions of the wayland protocols.")
                              ^XML                   ^Wayland

If it only contains XML definitions of the Wayland protocols why does it need "wayland" and "pkg-config". It probably does something else, too. I can see that it uses wayland-scanner for something. I suggest to update the description.

Otherwise LGTM!

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

* Re: [PATCH 12/24] gnu: gnome: Add mobile-broadband-provider-info.
  2016-08-16 18:39 ` [PATCH 12/24] gnu: gnome: Add mobile-broadband-provider-info David Craven
@ 2016-08-16 19:53   ` Danny Milosavljevic
  2016-08-17  9:33   ` Mark H Weaver
  1 sibling, 0 replies; 108+ messages in thread
From: Danny Milosavljevic @ 2016-08-16 19:53 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

> +    (synopsis "Datbase of broadband connection configuration")
> +    (description "Datbase of broadband connection configuration.")
                       ^^ a

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

* Re: [PATCH 08/24] gnu: xorg: Add lndir.
  2016-08-16 18:39 ` [PATCH 08/24] gnu: xorg: Add lndir David Craven
@ 2016-08-16 19:53   ` Alex Kost
  0 siblings, 0 replies; 108+ messages in thread
From: Alex Kost @ 2016-08-16 19:53 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven (2016-08-16 21:39 +0300) wrote:

> * gnu/packages/xorg.scm (lndir): New variable.

Nice program, I didn't know about it, thanks!

> ---
>  gnu/packages/xorg.scm | 25 +++++++++++++++++++++++++
>  1 file changed, 25 insertions(+)
>
>
> diff --git a/gnu/packages/xorg.scm b/gnu/packages/xorg.scm
> index b24e437..1e7da68 100644
> --- a/gnu/packages/xorg.scm
> +++ b/gnu/packages/xorg.scm
> @@ -133,6 +133,31 @@ projects.  Software developers are encouraged to migrate software to the GNU
>  autotools system.")
>      (license license:x11)))
>  
> +(define-public lndir
> +  (package
> +    (name "lndir")
> +    (version "1.0.3")
> +    (source
> +      (origin
> +        (method url-fetch)
> +        (uri (string-append
> +                "mirror://xorg/individual/util/lndir-"
> +                version
> +                ".tar.bz2"))

The patch looks good to me, except the indentation of this
(string-append ...).  I usually write 'source' fields like this:

    (source (origin
              (method url-fetch)
              (uri (string-append "mirror://xorg/individual/util/lndir-"
                                  version ".tar.bz2"))
              (sha256
               (base32
                "0pdngiy8zdhsiqx2am75yfcl36l7kd7d7nl0rss8shcdvsqgmx29"))))

But this is not a big deal, of course :-)

> +        (sha256
> +         (base32
> +          "0pdngiy8zdhsiqx2am75yfcl36l7kd7d7nl0rss8shcdvsqgmx29"))))
> +    (build-system gnu-build-system)
> +    (native-inputs
> +     `(("pkg-config" ,pkg-config)))
> +    (inputs
> +     `(("xproto" ,xproto)))
> +    (home-page "http://www.x.org")
> +    (synopsis "Symlink directory into tree")
> +    (description "Create a shadow directory of symbolic links to another
> +directory tree.")
> +    (license license:x11)))
> +
>  (define-public bdftopcf
>    (package
>      (name "bdftopcf")

-- 
Alex

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

* Re: [PATCH 05/24] gnu: xml: Add yaml-cpp.
  2016-08-16 18:39 ` [PATCH 05/24] gnu: xml: Add yaml-cpp David Craven
@ 2016-08-16 19:58   ` Alex Kost
  0 siblings, 0 replies; 108+ messages in thread
From: Alex Kost @ 2016-08-16 19:58 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven (2016-08-16 21:39 +0300) wrote:

> * gnu/packages/xml.scm (yaml-cpp): New variable.
> ---
>  gnu/packages/xml.scm | 24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
>
>
> diff --git a/gnu/packages/xml.scm b/gnu/packages/xml.scm
> index cfdc6c0..660a0b9 100644
> --- a/gnu/packages/xml.scm
> +++ b/gnu/packages/xml.scm
> @@ -12,6 +12,7 @@
>  ;;; Copyright © 2016 Leo Famulari <leo@famulari.name>
>  ;;; Copyright © 2016 Ben Woodcroft <donttrustben@gmail.com>
>  ;;; Copyright © 2016 Jan Nieuwenhuizen <janneke@gnu.org>
> +;;; Copyright © 2016 David Craven <david@craven.ch>
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -31,6 +32,7 @@
>  (define-module (gnu packages xml)
>    #:use-module (gnu packages)
>    #:use-module (gnu packages autotools)
> +  #:use-module (gnu packages boost)
>    #:use-module (gnu packages compression)
>    #:use-module (gnu packages gnupg)
>    #:use-module (gnu packages perl)
> @@ -808,3 +810,25 @@ XSLT and EXSLT.")
>  XLSM) format spreadsheets into plaintext @dfn{comma separated values} (CSV)
>  files.  It is designed to be fast and to handle large input files.")
>      (license license:gpl2+)))
> +
> +(define-public yaml-cpp
> +  (package
> +    (name "yaml-cpp")
> +    (version "0.5.3")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append
> +                "https://github.com/jbeder/yaml-cpp/archive/"
> +                "yaml-cpp-" version ".tar.gz"))

Please indent string-append like this:

                 (uri (string-append
                       "https://github.com/jbeder/yaml-cpp/archive/"
                       "yaml-cpp-" version ".tar.gz"))

> +              (sha256
> +               (base32
> +                "1vk6pjh0f5k6jwk2sszb9z5169whmiha9ainbdpa1arxlkq7v3b6"))))
> +    (build-system cmake-build-system)
> +    (inputs
> +     `(("boost" ,boost)))
> +    (native-inputs
> +     `(("python" ,python)))
> +    (home-page "https://github.com/jbeder/yaml-cpp")
> +    (synopsis "YAML parser and emitter in C++ matching the YAML 1.2 spec")

I would simplify the synopsis a bit: "YAML parser and emitter in C++"
(just because I don't like when synopsis and description are the same :-))

> +    (description "YAML parser and emitter in C++ matching the YAML 1.2 spec.")
> +    (license license:bsd-3)))

-- 
Alex

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

* Re: [PATCH 06/24] gnu: xml: Add jsoncpp.
  2016-08-16 19:28   ` Danny Milosavljevic
@ 2016-08-16 19:59     ` Danny Milosavljevic
  0 siblings, 0 replies; 108+ messages in thread
From: Danny Milosavljevic @ 2016-08-16 19:59 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

Also, why the XML package? I suggest "serialization.scm" or something.

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

* Re: [PATCH 06/24] gnu: xml: Add jsoncpp.
  2016-08-16 18:39 ` [PATCH 06/24] gnu: xml: Add jsoncpp David Craven
  2016-08-16 19:28   ` Danny Milosavljevic
@ 2016-08-16 20:00   ` Alex Kost
  1 sibling, 0 replies; 108+ messages in thread
From: Alex Kost @ 2016-08-16 20:00 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven (2016-08-16 21:39 +0300) wrote:

> * gnu/packages/xml.scm (jsoncpp): New variable.
> ---
>  gnu/packages/xml.scm | 19 +++++++++++++++++++
>  1 file changed, 19 insertions(+)
>
>
> diff --git a/gnu/packages/xml.scm b/gnu/packages/xml.scm
> index 660a0b9..72403c2 100644
> --- a/gnu/packages/xml.scm
> +++ b/gnu/packages/xml.scm
> @@ -832,3 +832,22 @@ files.  It is designed to be fast and to handle large input files.")
>      (synopsis "YAML parser and emitter in C++ matching the YAML 1.2 spec")
>      (description "YAML parser and emitter in C++ matching the YAML 1.2 spec.")
>      (license license:bsd-3)))
> +
> +(define-public jsoncpp
> +  (package
> +    (name "jsoncpp")
> +    (version "1.7.4")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append
> +                "https://github.com/open-source-parsers/jsoncpp/archive/"
> +                version ".tar.gz"))

Indentation of 'string-append', please :-)

> +              (file-name (string-append name "-" version ".tar.gz"))
> +              (sha256
> +               (base32
> +                "0sgp6nc4c6pfn92f369v08zdwpqswn9j2ihy59bpwwl0grkx1p0h"))))
> +    (build-system cmake-build-system)
> +    (home-page "https://github.com/open-source-parsers/jsoncpp")
> +    (synopsis "C++ library for interacting with JSON")
> +    (description "C++ library for interacting with JSON.")

Maybe expand a bit (taken from the project README):

"JsonCpp is a C++ library that allows manipulating JSON values, including
serialization and deserialization to and from strings.  It can also
preserve existing comment in unserialization/serialization steps, making
it a convenient format to store user input files."

> +    (license license:expat)))

-- 
Alex

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

* Re: [PATCH 09/24] gnu: freedesktop: Add wayland-protocols.
  2016-08-16 18:39 ` [PATCH 09/24] gnu: freedesktop: Add wayland-protocols David Craven
  2016-08-16 19:51   ` Danny Milosavljevic
@ 2016-08-16 20:02   ` Alex Kost
  1 sibling, 0 replies; 108+ messages in thread
From: Alex Kost @ 2016-08-16 20:02 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven (2016-08-16 21:39 +0300) wrote:

> * gnu/packages/freedesktop.scm (wayland-protocols): New variable.
> ---
>  gnu/packages/freedesktop.scm | 22 ++++++++++++++++++++++
>  1 file changed, 22 insertions(+)
>
>
> diff --git a/gnu/packages/freedesktop.scm b/gnu/packages/freedesktop.scm
> index 0262d0a..a24d8fa 100644
> --- a/gnu/packages/freedesktop.scm
> +++ b/gnu/packages/freedesktop.scm
> @@ -295,6 +295,28 @@ application, or a wayland client itself.  The clients can be traditional
>  applications, X servers (rootless or fullscreen) or other display servers.")
>      (license license:x11)))
>  
> +(define-public wayland-protocols
> +  (package
> +    (name "wayland-protocols")
> +    (version "1.4")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append
> +                "https://wayland.freedesktop.org/releases/"
> +                "wayland-protocols-" version ".tar.xz"))

Please fix indentation here (and in other packages)

> +              (sha256
> +               (base32
> +                "0wpm7mz7ww6nn3vrgz7a9iyk7mk6za73wnq0n54lzl8yq8irljh1"))))
> +    (build-system gnu-build-system)
> +    (inputs
> +     `(("wayland" ,wayland)))
> +    (native-inputs
> +     `(("pkg-config" ,pkg-config)))
> +    (synopsis "Wayland protocols")
> +    (description "Contains Xml definitions of the wayland protocols.")

Descriptions should contain full sentences, like:

"This package contains XML definitions of the Wayland protocols."

> +    (home-page "https://wayland.freedesktop.org")
> +    (license license:expat)))
> +
>  (define-public exempi
>    (package
>      (name "exempi")

-- 
Alex

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

* Re: [PATCH 10/24] gnu: networkmanager-qt: Propagate network-manager.
  2016-08-16 18:39 ` [PATCH 10/24] gnu: networkmanager-qt: Propagate network-manager David Craven
@ 2016-08-16 20:07   ` Alex Kost
  0 siblings, 0 replies; 108+ messages in thread
From: Alex Kost @ 2016-08-16 20:07 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven (2016-08-16 21:39 +0300) wrote:

> * gnu/packages/kde-frameworks.scm (networkmanager-qt)
>   [propagated-inputs]: Add NETWORK-MANAGER.
>   [inputs]: Remove NETWORK-MANAGER.

I would write 'network-manager' instead of NETWORK-MANAGER.

Also there should be a reason for propagating (probably written as a
comment).  Ideally it should be avoided.

> ---
>  gnu/packages/kde-frameworks.scm | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>
>
> diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm
> index dbec5a3..61ed4aa 100644
> --- a/gnu/packages/kde-frameworks.scm
> +++ b/gnu/packages/kde-frameworks.scm
> @@ -934,9 +934,10 @@ messages.")
>        `(("extra-cmake-modules" ,extra-cmake-modules)
>          ("dbus" ,dbus)
>          ("pkg-config" ,pkg-config)))
> +     (propagated-inputs
> +      `(("network-manager", network-manager)))
                             ↑↑
BTW please fix the comma placement here:
                             ↓↓
         `(("network-manager" ,network-manager)))

>       (inputs
> -      `(("network-manager", network-manager)
> -        ("qtbase" ,qtbase)))
> +      `(("qtbase" ,qtbase)))
>       (arguments
>        `(#:phases
>           (modify-phases %standard-phases

-- 
Alex

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

* Re: [PATCH 14/24] gnu: gstreamer: Use license: prefix.
  2016-08-16 18:39 ` [PATCH 14/24] gnu: gstreamer: Use license: prefix David Craven
  2016-08-16 19:18   ` Danny Milosavljevic
@ 2016-08-16 20:15   ` Alex Kost
  2016-08-16 20:29     ` David Craven
  2016-08-16 22:38     ` Eric Bavier
  1 sibling, 2 replies; 108+ messages in thread
From: Alex Kost @ 2016-08-16 20:15 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven (2016-08-16 21:39 +0300) wrote:

> * gnu/packages/gstreamer.scm (define-module): Import guix licenses with #:prefix
>   license:.
>   (orc): Likewise.
>   (gstreamer): Likewise.
>   (gst-plugins-base): Likewise.
>   (gst-plugins-good): Likewise.
>   (gst-plugins-bad): Likewise.
>   (gst-plugins-ugly): Likewise.
>   (gst-libav): Likewise.
>   (python-gst): Likewise.
  ^^
Please remove the leading spaces before "(...): Likewise."

I like this patch.  However in general (a question for Guix people): is
there a reason to use #:select for licenses?  Or would it be better to
use #:prefix everywhere?  If the latter (my preference), I think it
would be good to make this "#:select"→"#:prefix" change in a single
patch instead of making separate patches for different modules.

-- 
Alex

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

* Re: [PATCH 14/24] gnu: gstreamer: Use license: prefix.
  2016-08-16 20:15   ` Alex Kost
@ 2016-08-16 20:29     ` David Craven
  2016-08-16 22:38     ` Eric Bavier
  1 sibling, 0 replies; 108+ messages in thread
From: David Craven @ 2016-08-16 20:29 UTC (permalink / raw)
  To: Alex Kost; +Cc: guix-devel

> I like this patch.  However in general (a question for Guix people): is
> there a reason to use #:select for licenses?  Or would it be better to
> use #:prefix everywhere?  If the latter (my preference), I think it
> would be good to make this "#:select"→"#:prefix" change in a single
> patch instead of making separate patches for different modules.

I think that #:select has it's purpose. It helps increase readability when
one or two things are selected for example in bootstrap.scm.

#:use-module ((guix store) #:select (add-to-store add-text-to-store))
#:use-module ((guix derivations) #:select (derivation))
#:use-module ((guix utils) #:select (gnu-triplet->nix-system))

However when it comes to licenses it's verbose and unnecessary. I'll
amend the commit tomorrow to cover all cases where #:select is used
for importing licenses.

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

* Re: [PATCH 13/24] gnu: polkit-qt: Use modular qt.
  2016-08-16 19:17   ` Danny Milosavljevic
@ 2016-08-16 21:00     ` Leo Famulari
  0 siblings, 0 replies; 108+ messages in thread
From: Leo Famulari @ 2016-08-16 21:00 UTC (permalink / raw)
  To: Danny Milosavljevic; +Cc: guix-devel, David Craven

On Tue, Aug 16, 2016 at 09:17:56PM +0200, Danny Milosavljevic wrote:
> > -     `(("qt" ,qt))) ; qt-4 according to the pkg-config files
> > +     `(("qtbase" ,qtbase)))
> 
> That won't be qt-4 anymore. Is that OK?

It won't have been using qt-4 anyways, since the qt variable has always
been qt@5. qt-4 was added after qt@5.

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

* Re: [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer.
  2016-08-16 18:39 ` [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer David Craven
@ 2016-08-16 21:08   ` Leo Famulari
  2016-08-16 22:28   ` Eric Bavier
  1 sibling, 0 replies; 108+ messages in thread
From: Leo Famulari @ 2016-08-16 21:08 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

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?

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

* Re: [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer.
  2016-08-16 18:39 ` [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer David Craven
  2016-08-16 21:08   ` Leo Famulari
@ 2016-08-16 22:28   ` Eric Bavier
  1 sibling, 0 replies; 108+ messages in thread
From: Eric Bavier @ 2016-08-16 22:28 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

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

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

* Re: [PATCH 14/24] gnu: gstreamer: Use license: prefix.
  2016-08-16 20:15   ` Alex Kost
  2016-08-16 20:29     ` David Craven
@ 2016-08-16 22:38     ` Eric Bavier
  2016-08-17  8:02       ` David Craven
  1 sibling, 1 reply; 108+ messages in thread
From: Eric Bavier @ 2016-08-16 22:38 UTC (permalink / raw)
  To: Alex Kost; +Cc: guix-devel, David Craven

On Tue, 16 Aug 2016 23:15:07 +0300
Alex Kost <alezost@gmail.com> wrote:

> David Craven (2016-08-16 21:39 +0300) wrote:
> 
> > * gnu/packages/gstreamer.scm (define-module): Import guix licenses with #:prefix
> >   license:.
> >   (orc): Likewise.
> >   (gstreamer): Likewise.
> >   (gst-plugins-base): Likewise.
> >   (gst-plugins-good): Likewise.
> >   (gst-plugins-bad): Likewise.
> >   (gst-plugins-ugly): Likewise.
> >   (gst-libav): Likewise.
> >   (python-gst): Likewise.  
>   ^^
> Please remove the leading spaces before "(...): Likewise."

In fact, I think this is a case where one could simple say something
like "Adjust license references".

> I like this patch.  However in general (a question for Guix people): is
> there a reason to use #:select for licenses?  Or would it be better to
> use #:prefix everywhere?  If the latter (my preference), I think it
> would be good to make this "#:select"→"#:prefix" change in a single
> patch instead of making separate patches for different modules.

I don't think it's something that needs to be mandated.  If we were to
use '#:prefix license:' everywhere, we might as well save the trouble
and just rename everything in (guix license) with the
prefix.  But I don't think this is necessary.  There are instances of
importing with '#:hide' even, which I think has its place.

`~Eric

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

* Re: [PATCH 14/24] gnu: gstreamer: Use license: prefix.
  2016-08-16 22:38     ` Eric Bavier
@ 2016-08-17  8:02       ` David Craven
  2016-08-17  8:47         ` Danny Milosavljevic
  0 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-17  8:02 UTC (permalink / raw)
  To: Eric Bavier; +Cc: guix-devel, Alex Kost

I wonder who started the #:prefix/#:select thing in the first place. I
don't think that anything from guix licenses is likely to cause a name
collision. Maybe it's best to remove the #:prefix entirely?

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

* Re: [PATCH 14/24] gnu: gstreamer: Use license: prefix.
  2016-08-17  8:02       ` David Craven
@ 2016-08-17  8:47         ` Danny Milosavljevic
  2016-08-17  8:53           ` David Craven
  0 siblings, 1 reply; 108+ messages in thread
From: Danny Milosavljevic @ 2016-08-17  8:47 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel, Alex Kost

On Wed, 17 Aug 2016 10:02:26 +0200
David Craven <david@craven.ch> wrote:

> I wonder who started the #:prefix/#:select thing in the first place. I
> don't think that anything from guix licenses is likely to cause a name
> collision. Maybe it's best to remove the #:prefix entirely?

Expat causes a name collision. 

Also nmap, boost, cecill-c, freetype, imlib2, openldap2.0, openssl, ruby, tcl/tk, vim, x11, zlib.

Should there be new licenses it's a good guess it will be named like the package it originated in.

I would not advise removing the prefix. If anything, use it everywhere.

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

* Re: [PATCH 14/24] gnu: gstreamer: Use license: prefix.
  2016-08-17  8:47         ` Danny Milosavljevic
@ 2016-08-17  8:53           ` David Craven
  2016-08-17  9:17             ` Danny Milosavljevic
  0 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-17  8:53 UTC (permalink / raw)
  To: Danny Milosavljevic; +Cc: guix-devel, Alex Kost

> Expat causes a name collision.
>
> Also nmap, boost, cecill-c, freetype, imlib2, openldap2.0, openssl, ruby, tcl/tk, vim, x11, zlib.
>
> Should there be new licenses it's a good guess it will be named like the package it originated in.
>
> I would not advise removing the prefix. If anything, use it everywhere.

Ah, I was thinking more about lgpl2.1+ and the sorts... So is it a
good idea to prefix all licenses in guix licenses itself (as Eric
mentioned), and be done with it?

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

* Re: [PATCH 14/24] gnu: gstreamer: Use license: prefix.
  2016-08-17  8:53           ` David Craven
@ 2016-08-17  9:17             ` Danny Milosavljevic
  2016-08-17  9:20               ` David Craven
  0 siblings, 1 reply; 108+ messages in thread
From: Danny Milosavljevic @ 2016-08-17  9:17 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel, Alex Kost

On Wed, 17 Aug 2016 10:53:23 +0200
David Craven <david@craven.ch> wrote:

> Ah, I was thinking more about lgpl2.1+ and the sorts... So is it a
> good idea to prefix all licenses in guix licenses itself (as Eric
> mentioned), and be done with it?

That would be a massive refactoring - but can be done eventually.

I would just #:prefix license: whenever one touches something that caused these problems. No need to modify everything in one go.

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

* Re: [PATCH 14/24] gnu: gstreamer: Use license: prefix.
  2016-08-17  9:17             ` Danny Milosavljevic
@ 2016-08-17  9:20               ` David Craven
  0 siblings, 0 replies; 108+ messages in thread
From: David Craven @ 2016-08-17  9:20 UTC (permalink / raw)
  To: Danny Milosavljevic; +Cc: guix-devel, Alex Kost

> That would be a massive refactoring - but can be done eventually.

I don't think it would be too hard, most work can be done through
applying a regex to the entire project and running the testsuite to
find the cases where it didn't work. I did this on the python.scm
file...

So then I'll leave the patch as it is?

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

* Re: [PATCH 15/24] gnu: gstreamer: Symlink gstconfig.h into include directory.
  2016-08-16 18:39 ` [PATCH 15/24] gnu: gstreamer: Symlink gstconfig.h into include directory David Craven
@ 2016-08-17  9:28   ` Mark H Weaver
  2016-08-17  9:35     ` David Craven
  0 siblings, 1 reply; 108+ messages in thread
From: Mark H Weaver @ 2016-08-17  9:28 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:

> * gnu/packages/gstreamer.scm (gstreamer)[arguments]: Add symlink-gstconfig.h
>   phase.

What is the reason for this change?  Would it be appropriate to submit a
bug report upstream to add this to their "make install"?

      Mark
      

> ---
>  gnu/packages/gstreamer.scm | 11 ++++++++++-
>  1 file changed, 10 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/gstreamer.scm b/gnu/packages/gstreamer.scm
> index bd99880..54919cd 100644
> --- a/gnu/packages/gstreamer.scm
> +++ b/gnu/packages/gstreamer.scm
> @@ -113,7 +113,16 @@ arrays of data.")
>       `(#:configure-flags
>         (list (string-append "--with-html-dir="
>                              (assoc-ref %outputs "doc")
> -                            "/share/gtk-doc/html"))))
> +                            "/share/gtk-doc/html"))
> +       #:phases
> +        (modify-phases %standard-phases
> +          (add-after 'install 'symlink-gstconfig.h
> +            (lambda* (#:key outputs #:allow-other-keys)
> +              (symlink
> +                (string-append (assoc-ref outputs "out")
> +                 "/lib/gstreamer-1.0/include/gst/gstconfig.h")
> +                (string-append (assoc-ref outputs "out")
> +                 "/include/gstreamer-1.0/gst/gstconfig.h")))))))
>      (propagated-inputs `(("glib" ,glib))) ; required by gstreamer-1.0.pc.
>      (native-inputs
>       `(("bison" ,bison)

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

* Re: [PATCH 12/24] gnu: gnome: Add mobile-broadband-provider-info.
  2016-08-16 18:39 ` [PATCH 12/24] gnu: gnome: Add mobile-broadband-provider-info David Craven
  2016-08-16 19:53   ` Danny Milosavljevic
@ 2016-08-17  9:33   ` Mark H Weaver
  2016-08-17  9:37     ` David Craven
  1 sibling, 1 reply; 108+ messages in thread
From: Mark H Weaver @ 2016-08-17  9:33 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:

> * gnu/packages/gnome.scm (mobile-broadband-provider-info): New variable.

The summary line to should be "gnu: Add mobile-broadband-provider-info."
without the "gnome: ".  By our conventions, starting a summary line with
"gnu: gnome: " indicates a change to the 'gnome' package, which is not
the case here, and leads to confusion in practice.  I read this summary
line as meaning that 'mobile-broadband-provider-info' was added to
gnome's propagated-inputs.

This same issue applies to the other patches in this patch set as well.
When adding new packages, our convention is simply to write "gnu: Add
<package-name>."

     Thanks,
       Mark

> ---
>  gnu/packages/gnome.scm | 22 ++++++++++++++++++++++
>  1 file changed, 22 insertions(+)
>
> diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm
> index 4a0be01..7bc0445 100644
> --- a/gnu/packages/gnome.scm
> +++ b/gnu/packages/gnome.scm
> @@ -21,6 +21,7 @@
>  ;;; Copyright © 2016 Leo Famulari <leo@famulari.name>
>  ;;; Copyright © 2016 Alex Griffin <a@ajgrf.com>
>  ;;; Copyright © 2016 ng0 <ng0@we.make.ritual.n0.is>
> +;;; Copyright © 2016 David Craven <david@craven.ch>
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -4460,6 +4461,27 @@ devices, and provides VPN integration with a variety of different VPN
>  services.")
>      (license license:gpl2+)))
>  
> +(define-public mobile-broadband-provider-info
> +  (package
> +    (name "mobile-broadband-provider-info")
> +    (version "20151214")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append
> +                "mirror://gnome/sources/"
> +                "mobile-broadband-provider-info/" version "/"
> +                "mobile-broadband-provider-info-" version ".tar.xz"))
> +              (sha256
> +               (base32
> +                "1905nab1h8p4hx0m1w0rn4mkg9209x680dcr4l77bngy21pmvr4a"))))
> +    (build-system gnu-build-system)
> +    (arguments
> +     `(#:tests? #f)) ; No tests
> +    (home-page "https://wiki.gnome.org/Projects/NetworkManager")
> +    (synopsis "Datbase of broadband connection configuration")
> +    (description "Datbase of broadband connection configuration.")
> +    (license license:public-domain)))
> +
>  (define-public network-manager-applet
>    (package
>      (name "network-manager-applet")

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

* Re: [PATCH 15/24] gnu: gstreamer: Symlink gstconfig.h into include directory.
  2016-08-17  9:28   ` Mark H Weaver
@ 2016-08-17  9:35     ` David Craven
  2016-08-17  9:57       ` Mark H Weaver
  0 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-17  9:35 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel

> What is the reason for this change?  Would it be appropriate to submit a
> bug report upstream to add this to their "make install"?

The reason is because applications may contain #include <gst/gstconfig.h>,
but gstconfig.h was moved to the lib directory because it contains "platform
specific information".

What do you suggest?

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

* Re: [PATCH 12/24] gnu: gnome: Add mobile-broadband-provider-info.
  2016-08-17  9:33   ` Mark H Weaver
@ 2016-08-17  9:37     ` David Craven
  0 siblings, 0 replies; 108+ messages in thread
From: David Craven @ 2016-08-17  9:37 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel

> This same issue applies to the other patches in this patch set as well.
> When adding new packages, our convention is simply to write "gnu: Add
> <package-name>."

:thumbs up:

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

* Re: [PATCH 15/24] gnu: gstreamer: Symlink gstconfig.h into include directory.
  2016-08-17  9:35     ` David Craven
@ 2016-08-17  9:57       ` Mark H Weaver
  2016-08-17 10:10         ` David Craven
  0 siblings, 1 reply; 108+ messages in thread
From: Mark H Weaver @ 2016-08-17  9:57 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:

>> What is the reason for this change?  Would it be appropriate to submit a
>> bug report upstream to add this to their "make install"?
>
> The reason is because applications may contain #include <gst/gstconfig.h>,
> but gstconfig.h was moved to the lib directory because it contains "platform
> specific information".
>
> What do you suggest?

The pkgconfig file for gstreamer, $out/lib/pkgconfig/pkgconfig-1.0.pc,
includes this:

  Cflags: -I${includedir} -I${libdir}/gstreamer-1.0/include

Where ${libdir} is ${prefix}/lib.  So, the include path needed to find
<gst/gstconfig.h> should automatically be included by non-broken build
systems.

What application(s) are you aware of that are unable to find
<gst/gstconfig.h>?

In general, we should stick to upstream packaging unless it is clearly
broken or there is a compelling reason to deviate.  In this case, if an
application can't find <gst/gstconfig.h>, then it would seem to have a
buggy build system, and we should fix the problem there.

What do you think?

    Thanks,
      Mark

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

* Re: [PATCH 15/24] gnu: gstreamer: Symlink gstconfig.h into include directory.
  2016-08-17  9:57       ` Mark H Weaver
@ 2016-08-17 10:10         ` David Craven
  0 siblings, 0 replies; 108+ messages in thread
From: David Craven @ 2016-08-17 10:10 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel

> In general, we should stick to upstream packaging unless it is clearly
> broken or there is a compelling reason to deviate.  In this case, if an
> application can't find <gst/gstconfig.h>, then it would seem to have a
> buggy build system, and we should fix the problem there.

The qt-gstreamer package doesn't build without this, but I agree that the
problem is with the qt-gstreamer package and not gstreamer and should
therefore be fixed there.

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

* Re: [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer.
@ 2016-08-17 11:14 David Craven
  2016-08-17 15:09 ` Eric Bavier
  0 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-17 11:14 UTC (permalink / raw)
  To: guix-devel, Eric Bavier, Leo Famulari

> 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/../../..")

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

* Re: [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer.
  2016-08-17 11:14 [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer David Craven
@ 2016-08-17 15:09 ` Eric Bavier
  2016-08-18  0:18   ` David Craven
  0 siblings, 1 reply; 108+ messages in thread
From: Eric Bavier @ 2016-08-17 15:09 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

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

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

* Re: [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer.
  2016-08-17 15:09 ` Eric Bavier
@ 2016-08-18  0:18   ` David Craven
  0 siblings, 0 replies; 108+ messages in thread
From: David Craven @ 2016-08-18  0:18 UTC (permalink / raw)
  To: Eric Bavier; +Cc: guix-devel

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

Thank you!

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

* Re: [PATCH 24/24] gnu: qt: Update qt to 5.7.0.
  2016-08-16 18:39 ` [PATCH 24/24] gnu: qt: Update qt to 5.7.0 David Craven
  2016-08-16 19:41   ` Danny Milosavljevic
@ 2016-08-19  0:28   ` Mark H Weaver
  2016-08-19  8:56     ` David Craven
  1 sibling, 1 reply; 108+ messages in thread
From: Mark H Weaver @ 2016-08-19  0:28 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

Hi David,

David Craven <david@craven.ch> writes:

> * gnu/packages/qt.scm
>   (qtbase): Update it.
>   (qtsvg): Update it.
>   (qtimageformats): Update it.
>   (qtx11extras): Update it.
>   (qtxmlpatterns): Update it.
>   (qtdeclarative): Update it.
>   (qtconnectivity): Update it.
>   (qtwebsockets): Update it.
>   (qtsensors): Update it.
>   (qtmultimedia): Update it.
>   (qtwayland): Update it.
>   (qtserialport): Update it.
>   (qtwebchannel): Update it.
>   (qtlocation): Update it.
>   (qttools): Update it.
>   (qtscript): Update it.
>   (qtquickcontrols): Update it.
>   (qtquickcontrols2): Update it.
>   (qtgraphicaleffects): Update it.

Thank you for all of this!  Hydra indicates that there are some
regressions.  Can you take a look?

  https://hydra.gnu.org/eval/109084#tabs-now-fail
  https://hydra.gnu.org/eval/109084#tabs-new

      Mark

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

* Re: [PATCH 24/24] gnu: qt: Update qt to 5.7.0.
  2016-08-19  0:28   ` Mark H Weaver
@ 2016-08-19  8:56     ` David Craven
  2016-08-19 18:41       ` Mark H Weaver
  0 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-19  8:56 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel

> Thank you for all of this!  Hydra indicates that there are some
> regressions.  Can you take a look?
>
>   https://hydra.gnu.org/eval/109084#tabs-now-fail
>   https://hydra.gnu.org/eval/109084#tabs-new

Ah, now I understand where the fun in distributing starts =P It may
take me a while... Aren't minor releases supposed to be backwards
compatible?

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

* Re: [PATCH 24/24] gnu: qt: Update qt to 5.7.0.
  2016-08-19  8:56     ` David Craven
@ 2016-08-19 18:41       ` Mark H Weaver
  2016-08-19 18:45         ` David Craven
  0 siblings, 1 reply; 108+ messages in thread
From: Mark H Weaver @ 2016-08-19 18:41 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:

>> Thank you for all of this!  Hydra indicates that there are some
>> regressions.  Can you take a look?
>>
>>   https://hydra.gnu.org/eval/109084#tabs-now-fail
>>   https://hydra.gnu.org/eval/109084#tabs-new
>
> Ah, now I understand where the fun in distributing starts =P It may
> take me a while... Aren't minor releases supposed to be backwards
> compatible?

I don't know, ask the Qt developers.  Anyway, it's probably better to do
these kinds of upgrades on a separate branch to allow regressions to be
fixed without leaving 'master' broken in the meantime.  We can ask hydra
to build another branch and check for regressions there before merging
to master.

      Mark

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

* Re: [PATCH 24/24] gnu: qt: Update qt to 5.7.0.
  2016-08-19 18:41       ` Mark H Weaver
@ 2016-08-19 18:45         ` David Craven
  2016-08-20 15:40           ` David Craven
  0 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-19 18:45 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel

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

Yes, would have probably been better. I got sidetracked today, but I intend
to get it fixed over the weekend.

On Aug 19, 2016 8:42 PM, "Mark H Weaver" <mhw@netris.org> wrote:

> David Craven <david@craven.ch> writes:
>
> >> Thank you for all of this!  Hydra indicates that there are some
> >> regressions.  Can you take a look?
> >>
> >>   https://hydra.gnu.org/eval/109084#tabs-now-fail
> >>   https://hydra.gnu.org/eval/109084#tabs-new
> >
> > Ah, now I understand where the fun in distributing starts =P It may
> > take me a while... Aren't minor releases supposed to be backwards
> > compatible?
>
> I don't know, ask the Qt developers.  Anyway, it's probably better to do
> these kinds of upgrades on a separate branch to allow regressions to be
> fixed without leaving 'master' broken in the meantime.  We can ask hydra
> to build another branch and check for regressions there before merging
> to master.
>
>       Mark
>

[-- Attachment #2: Type: text/html, Size: 1572 bytes --]

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

* Re: [PATCH 24/24] gnu: qt: Update qt to 5.7.0.
  2016-08-19 18:45         ` David Craven
@ 2016-08-20 15:40           ` David Craven
  2016-08-20 18:50             ` Mark H Weaver
  0 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-20 15:40 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel

Are we still using gcc 4.9 as our default compiler? I think that some
of the build failures could be caused by that. qt 5.7 requires a c11
compiler and gcc 4.9 still defaults to the c99 standard. Any
dependency on qt 5.7 also needs to be compiled with a c11 compiler.

Are there plans to upgrade the default gcc? I look into fixing the
packages temporarily.

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

* Re: [PATCH 24/24] gnu: qt: Update qt to 5.7.0.
  2016-08-20 15:40           ` David Craven
@ 2016-08-20 18:50             ` Mark H Weaver
  2016-08-20 19:50               ` David Craven
  0 siblings, 1 reply; 108+ messages in thread
From: Mark H Weaver @ 2016-08-20 18:50 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:

> Are we still using gcc 4.9 as our default compiler? I think that some
> of the build failures could be caused by that. qt 5.7 requires a c11
> compiler and gcc 4.9 still defaults to the c99 standard. Any
> dependency on qt 5.7 also needs to be compiled with a c11 compiler.
>
> Are there plans to upgrade the default gcc? 

We can only change the default compiler in core-updates.  We tried to
upgrade gcc to 5.x in our last core-updates cycle, but had to revert
that change because the new gcc failed to bootstrap on armhf, because of
unexpected differences in stages 2 and 3.

Interestingly, in the current core-updates cycle, which modestly updates
to 4.9.4 only, is running into the same problem on armhf:

  https://hydra.gnu.org/build/1443911/nixlog/6/tail-reload

In any case, if your Qt updates depend on GCC 5.x, then we may need to
revert these upgrades on 'master' and revisit them after we've
successfully updated our default GCC.

> I look into fixing the packages temporarily.

     Thanks,
       Mark

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

* Re: [PATCH 24/24] gnu: qt: Update qt to 5.7.0.
  2016-08-20 18:50             ` Mark H Weaver
@ 2016-08-20 19:50               ` David Craven
  2016-08-20 20:06                 ` Efraim Flashner
  2016-08-20 21:59                 ` Leo Famulari
  0 siblings, 2 replies; 108+ messages in thread
From: David Craven @ 2016-08-20 19:50 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel

I fixed all regression except vlc and avidemux (locally). I'll have to
do some more debugging tomorrow. Forcing --std=gnu++11 worked in all
other cases. With the release of vlc 3 eminent, would it be acceptable
to use qt-4 with vlc 2.2.4 (if I don't manage to fix it)? That's what
arch linux is doing currently - and forcing --std=gnu++98, since
they're on gcc6.

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

* Re: [PATCH 24/24] gnu: qt: Update qt to 5.7.0.
  2016-08-20 19:50               ` David Craven
@ 2016-08-20 20:06                 ` Efraim Flashner
  2016-08-20 21:59                 ` Leo Famulari
  1 sibling, 0 replies; 108+ messages in thread
From: Efraim Flashner @ 2016-08-20 20:06 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

On Sat, Aug 20, 2016 at 09:50:51PM +0200, David Craven wrote:
> I fixed all regression except vlc and avidemux (locally). I'll have to
> do some more debugging tomorrow. Forcing --std=gnu++11 worked in all
> other cases. With the release of vlc 3 eminent, would it be acceptable
> to use qt-4 with vlc 2.2.4 (if I don't manage to fix it)? That's what
> arch linux is doing currently - and forcing --std=gnu++98, since
> they're on gcc6.
> 

you could also try using regular qt, that's still on 5.6.1-1

-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

-- 
Sent from my Android device with K-9 Mail. Please excuse my brevity.

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

* Re: [PATCH 24/24] gnu: qt: Update qt to 5.7.0.
  2016-08-20 19:50               ` David Craven
  2016-08-20 20:06                 ` Efraim Flashner
@ 2016-08-20 21:59                 ` Leo Famulari
  2016-08-21 14:08                   ` [PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7 David Craven
  1 sibling, 1 reply; 108+ messages in thread
From: Leo Famulari @ 2016-08-20 21:59 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

On Sat, Aug 20, 2016 at 09:50:51PM +0200, David Craven wrote:
> I fixed all regression except vlc and avidemux (locally). I'll have to
> do some more debugging tomorrow. Forcing --std=gnu++11 worked in all
> other cases. With the release of vlc 3 eminent, would it be acceptable
> to use qt-4 with vlc 2.2.4 (if I don't manage to fix it)? That's what
> arch linux is doing currently - and forcing --std=gnu++98, since
> they're on gcc6.

My interpretation of the Qt security policy is that qt-4 is unsupported.

"What Version of Qt are Supported?

Fixes are only guaranteed to be provided for:

    The latest released version.

    The preceding minor version.

    Fixes for earlier versions (such as 4.8, 5.0, etc.) may be provided,
    but the qt-project makes no commitment to do so. Other groups such
    as Digia may choose to make such fixes available, but that is
    outside the scope of the qt-project."

source: https://wiki.qt.io/Qt_project_security_policy

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

* [PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7.
  2016-08-20 21:59                 ` Leo Famulari
@ 2016-08-21 14:08                   ` David Craven
  2016-08-21 14:08                     ` [PATCH 02/13] gnu: kwidgetsaddons: " David Craven
                                       ` (11 more replies)
  0 siblings, 12 replies; 108+ messages in thread
From: David Craven @ 2016-08-21 14:08 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/kde-frameworks.scm (kguiaddons)[arguments]:
Remove phase 'start-xorg-server. Add phase 'check-setup.
---
 gnu/packages/kde-frameworks.scm | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm
index af81547..ba26952 100644
--- a/gnu/packages/kde-frameworks.scm
+++ b/gnu/packages/kde-frameworks.scm
@@ -539,12 +539,9 @@ infrastructure.")
     (arguments
      `(#:phases
         (modify-phases %standard-phases
-          (add-before 'check 'start-xorg-server
-            (lambda* (#:key inputs #:allow-other-keys)
-              ;; The test suite requires a running X server.
-              (system (string-append (assoc-ref inputs "xorg-server")
-                                     "/bin/Xvfb :1 &"))
-              (setenv "DISPLAY" ":1")
+          (add-before 'check 'check-setup
+            (lambda* _
+              (setenv "QT_QPA_PLATFORM" "offscreen")
              #t)))))
     (home-page "https://community.kde.org/Frameworks")
     (synopsis "Utilities for graphical user interfaces")
-- 
2.9.0

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

* [PATCH 02/13] gnu: kwidgetsaddons: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                   ` [PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7 David Craven
@ 2016-08-21 14:08                     ` David Craven
  2016-08-21 19:48                       ` Mark H Weaver
  2016-08-21 14:08                     ` [PATCH 03/13] gnu: pinentry-qt: Fix regression David Craven
                                       ` (10 subsequent siblings)
  11 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-21 14:08 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/kde-frameworks.scm (kwidgetsaddons)[arguments]: Disable tests.
---
 gnu/packages/kde-frameworks.scm | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm
index ba26952..20ab37a 100644
--- a/gnu/packages/kde-frameworks.scm
+++ b/gnu/packages/kde-frameworks.scm
@@ -816,10 +816,13 @@ represented by a QPoint or a QSize.")
     (inputs
      `(("qtbase" ,qtbase)))
     (arguments
-     `(#:phases
+     `(#:tests? #f ; FIXME: Regression after update to qt 5.7
+       #:phases
         (modify-phases %standard-phases
           (add-before 'check 'check-setup
             (lambda* _
+              (setenv "QT_QPA_PLATFORM" "offscreen") ; a better solution to Xvfb
+              (setenv "CTEST_OUTPUT_ON_FAILURE" "1") ; enable debug info
               (setenv "DBUS_FATAL_WARNINGS" "0")))
           (add-before 'check 'start-xorg-server
             (lambda* (#:key inputs #:allow-other-keys)
-- 
2.9.0

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

* [PATCH 03/13] gnu: pinentry-qt: Fix regression.
  2016-08-21 14:08                   ` [PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7 David Craven
  2016-08-21 14:08                     ` [PATCH 02/13] gnu: kwidgetsaddons: " David Craven
@ 2016-08-21 14:08                     ` David Craven
  2016-08-21 20:16                       ` Mark H Weaver
  2016-08-21 14:08                     ` [PATCH 04/13] gnu: poppler: Update to 0.47.0 David Craven
                                       ` (9 subsequent siblings)
  11 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-21 14:08 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/gnupg.scm (pinentry-qt)[native-inputs]: Add gnu-gettext.
---
 gnu/packages/gnupg.scm | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/gnu/packages/gnupg.scm b/gnu/packages/gnupg.scm
index 7df96b4..fde6018 100644
--- a/gnu/packages/gnupg.scm
+++ b/gnu/packages/gnupg.scm
@@ -28,6 +28,7 @@
   #:use-module (gnu packages)
   #:use-module (gnu packages adns)
   #:use-module (gnu packages curl)
+  #:use-module (gnu packages gettext)
   #:use-module (gnu packages openldap)
   #:use-module (gnu packages perl)
   #:use-module (gnu packages pth)
@@ -642,6 +643,8 @@ passphrase when @code{gpg} or @code{gpg2} is run and needs it.")))
   (package
     (inherit pinentry-tty)
     (name "pinentry-qt")
+    (native-inputs
+     `(("gettext" ,gnu-gettext)))
     (inputs
      `(("qtbase" ,qtbase)
        ,@(package-inputs pinentry-tty)))
-- 
2.9.0

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

* [PATCH 04/13] gnu: poppler: Update to 0.47.0.
  2016-08-21 14:08                   ` [PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7 David Craven
  2016-08-21 14:08                     ` [PATCH 02/13] gnu: kwidgetsaddons: " David Craven
  2016-08-21 14:08                     ` [PATCH 03/13] gnu: pinentry-qt: Fix regression David Craven
@ 2016-08-21 14:08                     ` David Craven
  2016-08-21 19:52                       ` Mark H Weaver
  2016-08-21 14:08                     ` [PATCH 05/13] gnu: alsa-modular-synth: Fix regression caused by update to qt 5.7 David Craven
                                       ` (8 subsequent siblings)
  11 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-21 14:08 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/pdf.scm (poppler): Update it.
(poppler)[arguments]: Require gnu++11.
---
 gnu/packages/pdf.scm | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/pdf.scm b/gnu/packages/pdf.scm
index 4ddcedd..2b3512f 100644
--- a/gnu/packages/pdf.scm
+++ b/gnu/packages/pdf.scm
@@ -57,14 +57,14 @@
 (define-public poppler
   (package
    (name "poppler")
-   (version "0.43.0")
+   (version "0.47.0")
    (source (origin
             (method url-fetch)
             (uri (string-append "https://poppler.freedesktop.org/poppler-"
                                 version ".tar.xz"))
             (sha256
              (base32
-              "0mi4zf0pz3x3fx3ir7szz1n57nywgbpd4mp2r7mvf47f4rmf4867"))))
+              "0hnjkcqqk87dw3hlda4gh4l7brkslniax9a79g772jn3iwiffwmq"))))
    (build-system gnu-build-system)
    ;; FIXME:
    ;;  use libcurl:        no
@@ -94,7 +94,8 @@
         "--enable-zlib"
 
         ;; Saves 8 MiB of .a files.
-        "--disable-static")
+        "--disable-static"
+        "CXXFLAGS=-std=gnu++11")
       #:phases
       (alist-cons-before
        'configure 'setenv
-- 
2.9.0

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

* [PATCH 05/13] gnu: alsa-modular-synth: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                   ` [PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7 David Craven
                                       ` (2 preceding siblings ...)
  2016-08-21 14:08                     ` [PATCH 04/13] gnu: poppler: Update to 0.47.0 David Craven
@ 2016-08-21 14:08                     ` David Craven
  2016-08-21 19:53                       ` Mark H Weaver
  2016-08-21 14:08                     ` [PATCH 06/13] gnu: qsynth: " David Craven
                                       ` (7 subsequent siblings)
  11 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-21 14:08 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/audio.scm (alsa-modular-synth)[arguments]: Require gnu++11.
---
 gnu/packages/audio.scm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
index a214ef3..6c0f066 100644
--- a/gnu/packages/audio.scm
+++ b/gnu/packages/audio.scm
@@ -85,7 +85,9 @@
                 "1azbrhpfk4nnybr7kgmc7w6al6xnzppg853vas8gmkh185kk11l0"))))
     (build-system gnu-build-system)
     (arguments
-     `(#:configure-flags '("--enable-qt5")
+     `(#:configure-flags
+       '("--enable-qt5"
+         "CXXFLAGS=-std=gnu++11")
        #:phases
        (modify-phases %standard-phases
          ;; Insert an extra space between linker flags.
-- 
2.9.0

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

* [PATCH 06/13] gnu: qsynth: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                   ` [PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7 David Craven
                                       ` (3 preceding siblings ...)
  2016-08-21 14:08                     ` [PATCH 05/13] gnu: alsa-modular-synth: Fix regression caused by update to qt 5.7 David Craven
@ 2016-08-21 14:08                     ` David Craven
  2016-08-21 19:55                       ` Mark H Weaver
  2016-08-21 14:08                     ` [PATCH 07/13] gnu: gpsbabel: " David Craven
                                       ` (6 subsequent siblings)
  11 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-21 14:08 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/audio.scm (qsynth)[arguments]: Require gnu++11.
---
 gnu/packages/audio.scm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
index 6c0f066..2d741d5 100644
--- a/gnu/packages/audio.scm
+++ b/gnu/packages/audio.scm
@@ -2113,7 +2113,9 @@ interface.")
         (base32 "034p6mbwrjnxd9b6h20cidxi4ilkk3cgpjp154j0jzjs1ipf7x2h"))))
     (build-system gnu-build-system)
     (arguments
-     `(#:tests? #f)) ; no "check" phase
+     `(#:tests? #f ; no "check" phase
+       #:configure-flags
+        '("CXXFLAGS=-std=gnu++11")))
     (native-inputs
      `(("qttools" ,qttools)))
     (inputs
-- 
2.9.0

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

* [PATCH 07/13] gnu: gpsbabel: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                   ` [PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7 David Craven
                                       ` (4 preceding siblings ...)
  2016-08-21 14:08                     ` [PATCH 06/13] gnu: qsynth: " David Craven
@ 2016-08-21 14:08                     ` David Craven
  2016-08-21 19:59                       ` Mark H Weaver
  2016-08-21 14:08                     ` [PATCH 08/13] gnu: synthv1: " David Craven
                                       ` (5 subsequent siblings)
  11 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-21 14:08 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/gps.scm (gpsbabel)[arguments]: Require gnu++11. Disable tests
on all platforms due to rounding error.
---
 gnu/packages/gps.scm | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/gps.scm b/gnu/packages/gps.scm
index ccd9636..8124990 100644
--- a/gnu/packages/gps.scm
+++ b/gnu/packages/gps.scm
@@ -48,7 +48,9 @@
                 "0xf7wmy2m29g2lm8lqc74yf8rf7sxfl3cfwbk7dpf0yf42pb0b6w"))))
     (build-system gnu-build-system)
     (arguments
-     `(#:configure-flags '("--with-zlib=system")
+     `(#:configure-flags
+        '("--with-zlib=system"
+          "CXXFLAGS=-std=gnu++11")
        #:phases
        (modify-phases %standard-phases
         (add-before 'configure 'pre-configure
@@ -59,7 +61,8 @@
        ;; On i686, 'raymarine.test' fails because of a rounding error:
        ;; <http://hydra.gnu.org/build/133040>.  As a workaround, disable tests
        ;; on these platforms.
-       #:tests? ,(not (string-prefix? "i686" (%current-system)))))
+       ;; On x86_64 with -std=gnu++11 tests also fail due to rounding error.
+       #:tests? #f))
     (inputs
      `(("expat" ,expat)
        ("zlib" ,zlib)
-- 
2.9.0

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

* [PATCH 08/13] gnu: synthv1: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                   ` [PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7 David Craven
                                       ` (5 preceding siblings ...)
  2016-08-21 14:08                     ` [PATCH 07/13] gnu: gpsbabel: " David Craven
@ 2016-08-21 14:08                     ` David Craven
  2016-08-21 20:01                       ` Mark H Weaver
  2016-08-21 14:08                     ` [PATCH 09/13] gnu: scribus: " David Craven
                                       ` (4 subsequent siblings)
  11 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-21 14:08 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/music.scm (synthv1)[arguments]: Require gnu++11.
---
 gnu/packages/music.scm | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm
index dfd3f77..52eb3e0 100644
--- a/gnu/packages/music.scm
+++ b/gnu/packages/music.scm
@@ -702,8 +702,10 @@ Editor.  It is compatible with Power Tab Editor 1.7 and Guitar Pro.")
                (base32
                 "0h5zja78phf9705i9g54zh61iczb24iv7rxhljyms30sjgajig1y"))))
     (build-system gnu-build-system)
-    ;; There are no tests.
-    (arguments `(#:tests? #f))
+    (arguments
+     `(#:tests? #f ; There are no tests.
+       #:configure-flags
+        '("CXXFLAGS=-std=gnu++11")))
     (inputs
      `(("jack" ,jack-1)
        ("lv2" ,lv2)
-- 
2.9.0

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

* [PATCH 09/13] gnu: scribus: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                   ` [PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7 David Craven
                                       ` (6 preceding siblings ...)
  2016-08-21 14:08                     ` [PATCH 08/13] gnu: synthv1: " David Craven
@ 2016-08-21 14:08                     ` David Craven
  2016-08-21 20:02                       ` Mark H Weaver
  2016-08-21 14:08                     ` [PATCH 10/13] gnu: v4l-utils: " David Craven
                                       ` (3 subsequent siblings)
  11 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-21 14:08 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/scribus.scm (scribus)[arguments]: Require gnu++11.
---
 gnu/packages/scribus.scm | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/scribus.scm b/gnu/packages/scribus.scm
index 68e57cd..4f3f306 100644
--- a/gnu/packages/scribus.scm
+++ b/gnu/packages/scribus.scm
@@ -49,7 +49,10 @@
                (base32
                 "0f2adwg58w37sdi3xrk8xqw486p3pcfjaypcsswjl76r2f3yd0hq"))))
     (build-system cmake-build-system)
-    (arguments `(#:tests? #f)) ; no test target
+    (arguments
+     `(#:tests? #f ; no test target
+       #:configure-flags
+        '("-DCMAKE_CXX_FLAGS=-std=gnu++11")))
     (inputs
      `(("cairo" ,cairo)
        ("cups" ,cups)
-- 
2.9.0

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

* [PATCH 10/13] gnu: v4l-utils: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                   ` [PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7 David Craven
                                       ` (7 preceding siblings ...)
  2016-08-21 14:08                     ` [PATCH 09/13] gnu: scribus: " David Craven
@ 2016-08-21 14:08                     ` David Craven
  2016-08-21 20:04                       ` Mark H Weaver
  2016-08-21 14:08                     ` [PATCH 11/13] gnu: vlc: " David Craven
                                       ` (2 subsequent siblings)
  11 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-21 14:08 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/video.scm (v4l-utils)[arguments]: Require gnu++11.
---
 gnu/packages/video.scm | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index bc6415e..28e49b3 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -1396,7 +1396,8 @@ tools, XML authoring components, and an extensible plug-in based API.")
      '(#:configure-flags
        (list (string-append "--with-udevdir="
                             (assoc-ref %outputs "out")
-                            "/lib/udev"))))
+                            "/lib/udev")
+              "CXXFLAGS=-std=gnu++11")))
     (native-inputs
      `(("pkg-config" ,pkg-config)))
     (inputs
-- 
2.9.0

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

* [PATCH 11/13] gnu: vlc: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                   ` [PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7 David Craven
                                       ` (8 preceding siblings ...)
  2016-08-21 14:08                     ` [PATCH 10/13] gnu: v4l-utils: " David Craven
@ 2016-08-21 14:08                     ` David Craven
  2016-08-21 20:10                       ` Mark H Weaver
  2016-08-21 14:08                     ` [PATCH 12/13] gnu: avidemux: " David Craven
  2016-08-21 19:44                     ` [PATCH 01/13] gnu: kguiaddons: " Mark H Weaver
  11 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-21 14:08 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/video.scm (vlc)[inputs]: Use qt 5.6.1-1. Add libinput and libxi.
---
 gnu/packages/video.scm | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 28e49b3..45f8a28 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -53,6 +53,7 @@
   #:use-module (gnu packages databases)
   #:use-module (gnu packages elf)
   #:use-module (gnu packages fontutils)
+  #:use-module (gnu packages freedesktop)
   #:use-module (gnu packages fribidi)
   #:use-module (gnu packages gettext)
   #:use-module (gnu packages ghostscript)
@@ -617,6 +618,7 @@ audio/video codec library.")
        ("gnutls" ,gnutls)
        ("libcddb" ,libcddb)
        ("libgcrypt" ,libgcrypt)
+       ("libinput" ,libinput-minimal)
        ("libkate" ,libkate)
        ("libmad" ,libmad)
        ("libogg" ,libogg)
@@ -626,6 +628,7 @@ audio/video codec library.")
        ("libvorbis" ,libvorbis)
        ("libtheora" ,libtheora)
        ("libxext" ,libxext)
+       ("libxi" ,libxi)
        ("libxinerama" ,libxinerama)
        ("libxml2" ,libxml2)
        ("libxpm" ,libxpm)
@@ -635,7 +638,7 @@ audio/video codec library.")
        ("perl" ,perl)
        ("pulseaudio" ,pulseaudio)
        ("python" ,python-wrapper)
-       ("qtbase" ,qtbase)
+       ("qtbase" ,qt)
        ;("qtx11extras" ,qtx11extras)
        ("sdl" ,sdl)
        ("sdl-image" ,sdl-image)
-- 
2.9.0

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

* [PATCH 12/13] gnu: avidemux: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                   ` [PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7 David Craven
                                       ` (9 preceding siblings ...)
  2016-08-21 14:08                     ` [PATCH 11/13] gnu: vlc: " David Craven
@ 2016-08-21 14:08                     ` David Craven
  2016-08-21 20:13                       ` Mark H Weaver
  2016-08-21 19:44                     ` [PATCH 01/13] gnu: kguiaddons: " Mark H Weaver
  11 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-21 14:08 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/video.scm (avidemux)[inputs]: Use qt 5.6.1-1.
---
 gnu/packages/video.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index 45f8a28..7c68be8 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -1144,8 +1144,8 @@ for use with HTML5 video.")
        ("perl" ,perl)
        ("pulseaudio" ,pulseaudio)
        ("python" ,python-wrapper)
-       ("qtbase" ,qtbase)
-       ("qttools" ,qttools)
+       ("qtbase" ,qt)
+       ;("qttools" ,qttools)
        ("sdl" ,sdl)
        ("sqlite" ,sqlite)
        ("yasm" ,yasm)
-- 
2.9.0

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

* Re: [PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                   ` [PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7 David Craven
                                       ` (10 preceding siblings ...)
  2016-08-21 14:08                     ` [PATCH 12/13] gnu: avidemux: " David Craven
@ 2016-08-21 19:44                     ` Mark H Weaver
  11 siblings, 0 replies; 108+ messages in thread
From: Mark H Weaver @ 2016-08-21 19:44 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:

> * gnu/packages/kde-frameworks.scm (kguiaddons)[arguments]:
> Remove phase 'start-xorg-server. Add phase 'check-setup.
> ---
>  gnu/packages/kde-frameworks.scm | 9 +++------
>  1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm
> index af81547..ba26952 100644
> --- a/gnu/packages/kde-frameworks.scm
> +++ b/gnu/packages/kde-frameworks.scm
> @@ -539,12 +539,9 @@ infrastructure.")
>      (arguments
>       `(#:phases
>          (modify-phases %standard-phases
> -          (add-before 'check 'start-xorg-server
> -            (lambda* (#:key inputs #:allow-other-keys)
> -              ;; The test suite requires a running X server.
> -              (system (string-append (assoc-ref inputs "xorg-server")
> -                                     "/bin/Xvfb :1 &"))
> -              (setenv "DISPLAY" ":1")
> +          (add-before 'check 'check-setup
> +            (lambda* _
> +              (setenv "QT_QPA_PLATFORM" "offscreen")
>               #t)))))

The #t should line up with (setenv ...), i.e. it should be moved one
column to the right.  Otherwise, looks good to me!

     Thanks,
       Mark

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

* Re: [PATCH 02/13] gnu: kwidgetsaddons: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                     ` [PATCH 02/13] gnu: kwidgetsaddons: " David Craven
@ 2016-08-21 19:48                       ` Mark H Weaver
  0 siblings, 0 replies; 108+ messages in thread
From: Mark H Weaver @ 2016-08-21 19:48 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:

> * gnu/packages/kde-frameworks.scm (kwidgetsaddons)[arguments]: Disable tests.

You did more than disable tests.  Please describe the other changes.

> ---
>  gnu/packages/kde-frameworks.scm | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm
> index ba26952..20ab37a 100644
> --- a/gnu/packages/kde-frameworks.scm
> +++ b/gnu/packages/kde-frameworks.scm
> @@ -816,10 +816,13 @@ represented by a QPoint or a QSize.")
>      (inputs
>       `(("qtbase" ,qtbase)))
>      (arguments
> -     `(#:phases
> +     `(#:tests? #f ; FIXME: Regression after update to qt 5.7
> +       #:phases
>          (modify-phases %standard-phases
>            (add-before 'check 'check-setup
>              (lambda* _
> +              (setenv "QT_QPA_PLATFORM" "offscreen") ; a better solution to Xvfb
> +              (setenv "CTEST_OUTPUT_ON_FAILURE" "1") ; enable debug info
>                (setenv "DBUS_FATAL_WARNINGS" "0")))

lambda* => lambda, and there should be a #t added after the last setenv.
Otherwise, looks okay, although I hope the test suite can be re-enabled
soon.

      Mark

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

* Re: [PATCH 04/13] gnu: poppler: Update to 0.47.0.
  2016-08-21 14:08                     ` [PATCH 04/13] gnu: poppler: Update to 0.47.0 David Craven
@ 2016-08-21 19:52                       ` Mark H Weaver
  2016-08-21 22:40                         ` David Craven
  0 siblings, 1 reply; 108+ messages in thread
From: Mark H Weaver @ 2016-08-21 19:52 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:
> * gnu/packages/pdf.scm (poppler): Update it.
> (poppler)[arguments]: Require gnu++11.

This would entail about 3000 rebuilds.  If it's important, I'd prefer to
build it out on another branch.  If it's not important, maybe save it
for core-updates?

      Mark

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

* Re: [PATCH 05/13] gnu: alsa-modular-synth: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                     ` [PATCH 05/13] gnu: alsa-modular-synth: Fix regression caused by update to qt 5.7 David Craven
@ 2016-08-21 19:53                       ` Mark H Weaver
  0 siblings, 0 replies; 108+ messages in thread
From: Mark H Weaver @ 2016-08-21 19:53 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:

> * gnu/packages/audio.scm (alsa-modular-synth)[arguments]: Require gnu++11.
> ---
>  gnu/packages/audio.scm | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
> index a214ef3..6c0f066 100644
> --- a/gnu/packages/audio.scm
> +++ b/gnu/packages/audio.scm
> @@ -85,7 +85,9 @@
>                  "1azbrhpfk4nnybr7kgmc7w6al6xnzppg853vas8gmkh185kk11l0"))))
>      (build-system gnu-build-system)
>      (arguments
> -     `(#:configure-flags '("--enable-qt5")
> +     `(#:configure-flags
> +       '("--enable-qt5"
> +         "CXXFLAGS=-std=gnu++11")
>         #:phases
>         (modify-phases %standard-phases
>           ;; Insert an extra space between linker flags.

Looks good to me.

     Thanks,
       Mark

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

* Re: [PATCH 06/13] gnu: qsynth: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                     ` [PATCH 06/13] gnu: qsynth: " David Craven
@ 2016-08-21 19:55                       ` Mark H Weaver
  0 siblings, 0 replies; 108+ messages in thread
From: Mark H Weaver @ 2016-08-21 19:55 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:

> * gnu/packages/audio.scm (qsynth)[arguments]: Require gnu++11.
> ---
>  gnu/packages/audio.scm | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
> index 6c0f066..2d741d5 100644
> --- a/gnu/packages/audio.scm
> +++ b/gnu/packages/audio.scm
> @@ -2113,7 +2113,9 @@ interface.")
>          (base32 "034p6mbwrjnxd9b6h20cidxi4ilkk3cgpjp154j0jzjs1ipf7x2h"))))
>      (build-system gnu-build-system)
>      (arguments
> -     `(#:tests? #f)) ; no "check" phase
> +     `(#:tests? #f ; no "check" phase
> +       #:configure-flags
> +        '("CXXFLAGS=-std=gnu++11")))

It's not worth redoing this patch, but for future reference, the '
should be lined up under the #, although in this case it might be better
to just have it on the same line.

Otherwise, it looks good to me.

      Thanks,
        Mark

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

* Re: [PATCH 07/13] gnu: gpsbabel: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                     ` [PATCH 07/13] gnu: gpsbabel: " David Craven
@ 2016-08-21 19:59                       ` Mark H Weaver
  2016-08-21 22:55                         ` David Craven
  0 siblings, 1 reply; 108+ messages in thread
From: Mark H Weaver @ 2016-08-21 19:59 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:

> * gnu/packages/gps.scm (gpsbabel)[arguments]: Require gnu++11. Disable tests
> on all platforms due to rounding error.
> ---
>  gnu/packages/gps.scm | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/gps.scm b/gnu/packages/gps.scm
> index ccd9636..8124990 100644
> --- a/gnu/packages/gps.scm
> +++ b/gnu/packages/gps.scm
> @@ -48,7 +48,9 @@
>                  "0xf7wmy2m29g2lm8lqc74yf8rf7sxfl3cfwbk7dpf0yf42pb0b6w"))))
>      (build-system gnu-build-system)
>      (arguments
> -     `(#:configure-flags '("--with-zlib=system")
> +     `(#:configure-flags
> +        '("--with-zlib=system"
> +          "CXXFLAGS=-std=gnu++11")

The ' should be under the #, so the two lines above should be moved one
column to the left.

>         #:phases
>         (modify-phases %standard-phases
>          (add-before 'configure 'pre-configure
> @@ -59,7 +61,8 @@
>         ;; On i686, 'raymarine.test' fails because of a rounding error:
>         ;; <http://hydra.gnu.org/build/133040>.  As a workaround, disable tests
>         ;; on these platforms.
> -       #:tests? ,(not (string-prefix? "i686" (%current-system)))))
> +       ;; On x86_64 with -std=gnu++11 tests also fail due to rounding error.
> +       #:tests? #f))

Please add a FIXME to this comment.  We should not be in the habit of
simply disabling test suites that fail and forgetting about them.

I don't see why there should be a rounding error on x86_64.  Unlike
i686, double-rounding doesn't happen on that platform, and it's what
developers are mostly testing on.

Anyway, okay for now with the FIXME comment.

       Mark

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

* Re: [PATCH 08/13] gnu: synthv1: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                     ` [PATCH 08/13] gnu: synthv1: " David Craven
@ 2016-08-21 20:01                       ` Mark H Weaver
  0 siblings, 0 replies; 108+ messages in thread
From: Mark H Weaver @ 2016-08-21 20:01 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:

> * gnu/packages/music.scm (synthv1)[arguments]: Require gnu++11.
> ---
>  gnu/packages/music.scm | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm
> index dfd3f77..52eb3e0 100644
> --- a/gnu/packages/music.scm
> +++ b/gnu/packages/music.scm
> @@ -702,8 +702,10 @@ Editor.  It is compatible with Power Tab Editor 1.7 and Guitar Pro.")
>                 (base32
>                  "0h5zja78phf9705i9g54zh61iczb24iv7rxhljyms30sjgajig1y"))))
>      (build-system gnu-build-system)
> -    ;; There are no tests.
> -    (arguments `(#:tests? #f))
> +    (arguments
> +     `(#:tests? #f ; There are no tests.
> +       #:configure-flags
> +        '("CXXFLAGS=-std=gnu++11")))

For future reference, the ' should be under the #.
Otherwise looks okay.

      Thanks,
        Mark

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

* Re: [PATCH 09/13] gnu: scribus: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                     ` [PATCH 09/13] gnu: scribus: " David Craven
@ 2016-08-21 20:02                       ` Mark H Weaver
  0 siblings, 0 replies; 108+ messages in thread
From: Mark H Weaver @ 2016-08-21 20:02 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:

> * gnu/packages/scribus.scm (scribus)[arguments]: Require gnu++11.
> ---
>  gnu/packages/scribus.scm | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/scribus.scm b/gnu/packages/scribus.scm
> index 68e57cd..4f3f306 100644
> --- a/gnu/packages/scribus.scm
> +++ b/gnu/packages/scribus.scm
> @@ -49,7 +49,10 @@
>                 (base32
>                  "0f2adwg58w37sdi3xrk8xqw486p3pcfjaypcsswjl76r2f3yd0hq"))))
>      (build-system cmake-build-system)
> -    (arguments `(#:tests? #f)) ; no test target
> +    (arguments
> +     `(#:tests? #f ; no test target
> +       #:configure-flags
> +        '("-DCMAKE_CXX_FLAGS=-std=gnu++11")))

The ' should be under the #, but otherwise okay.

     Thanks,
       Mark

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

* Re: [PATCH 10/13] gnu: v4l-utils: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                     ` [PATCH 10/13] gnu: v4l-utils: " David Craven
@ 2016-08-21 20:04                       ` Mark H Weaver
  0 siblings, 0 replies; 108+ messages in thread
From: Mark H Weaver @ 2016-08-21 20:04 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:

> * gnu/packages/video.scm (v4l-utils)[arguments]: Require gnu++11.
> ---
>  gnu/packages/video.scm | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
> index bc6415e..28e49b3 100644
> --- a/gnu/packages/video.scm
> +++ b/gnu/packages/video.scm
> @@ -1396,7 +1396,8 @@ tools, XML authoring components, and an extensible plug-in based API.")
>       '(#:configure-flags
>         (list (string-append "--with-udevdir="
>                              (assoc-ref %outputs "out")
> -                            "/lib/udev"))))
> +                            "/lib/udev")
> +              "CXXFLAGS=-std=gnu++11")))

The " should be lined up under the "(", so that last line should be
moved one column to the left.

Also, although I neglected to mention it, in general when the change
made is simple enough to describe in the summary line, better to do
that, so instead of writing "Fix regression caused by update to qt 5.7",
please write "Require gnu++11.".

Otherwise looks good to me.

     Thanks,
       Mark
       

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

* Re: [PATCH 11/13] gnu: vlc: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                     ` [PATCH 11/13] gnu: vlc: " David Craven
@ 2016-08-21 20:10                       ` Mark H Weaver
  2016-08-21 23:06                         ` David Craven
  0 siblings, 1 reply; 108+ messages in thread
From: Mark H Weaver @ 2016-08-21 20:10 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:
> * gnu/packages/video.scm (vlc)[inputs]: Use qt 5.6.1-1. Add libinput and libxi.

The summary line is a bit misleading, because it leads me to guess that
this commit gets vlc working with Qt 5.7.  Better to write "gnu: vlc:
Use Qt 5.6.1."

Is it necessary to add 'libinput' and 'libxi' to fix this regression, or
is that an unrelated change to add new features?  If the latter, please
add those in a separate commit.

> @@ -635,7 +638,7 @@ audio/video codec library.")
>         ("perl" ,perl)
>         ("pulseaudio" ,pulseaudio)
>         ("python" ,python-wrapper)
> -       ("qtbase" ,qtbase)
> +       ("qtbase" ,qt)

Please change the string to "qt" as well.

      Thanks,
        Mark

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

* Re: [PATCH 12/13] gnu: avidemux: Fix regression caused by update to qt 5.7.
  2016-08-21 14:08                     ` [PATCH 12/13] gnu: avidemux: " David Craven
@ 2016-08-21 20:13                       ` Mark H Weaver
  2016-08-21 21:08                         ` David Craven
  0 siblings, 1 reply; 108+ messages in thread
From: Mark H Weaver @ 2016-08-21 20:13 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:

> * gnu/packages/video.scm (avidemux)[inputs]: Use qt 5.6.1-1.

In the summary line and in the line above, how about writing
"Use monolithic Qt 5.6."?

> ---
>  gnu/packages/video.scm | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
> index 45f8a28..7c68be8 100644
> --- a/gnu/packages/video.scm
> +++ b/gnu/packages/video.scm
> @@ -1144,8 +1144,8 @@ for use with HTML5 video.")
>         ("perl" ,perl)
>         ("pulseaudio" ,pulseaudio)
>         ("python" ,python-wrapper)
> -       ("qtbase" ,qtbase)
> -       ("qttools" ,qttools)
> +       ("qtbase" ,qt)
> +       ;("qttools" ,qttools)

Please change "qtbase" => "qt" and add a FIXME comment reminding us to
get this working with modular Qt 5.7+.

What do you think?

      Mark

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

* Re: [PATCH 03/13] gnu: pinentry-qt: Fix regression.
  2016-08-21 14:08                     ` [PATCH 03/13] gnu: pinentry-qt: Fix regression David Craven
@ 2016-08-21 20:16                       ` Mark H Weaver
  2016-08-21 22:37                         ` David Craven
  0 siblings, 1 reply; 108+ messages in thread
From: Mark H Weaver @ 2016-08-21 20:16 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:

> * gnu/packages/gnupg.scm (pinentry-qt)[native-inputs]: Add gnu-gettext.

The summary line should say "Add gnu-gettext to native-inputs" instead
of "Fix regression".

> ---
>  gnu/packages/gnupg.scm | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/gnu/packages/gnupg.scm b/gnu/packages/gnupg.scm
> index 7df96b4..fde6018 100644
> --- a/gnu/packages/gnupg.scm
> +++ b/gnu/packages/gnupg.scm
> @@ -28,6 +28,7 @@
>    #:use-module (gnu packages)
>    #:use-module (gnu packages adns)
>    #:use-module (gnu packages curl)
> +  #:use-module (gnu packages gettext)
>    #:use-module (gnu packages openldap)
>    #:use-module (gnu packages perl)
>    #:use-module (gnu packages pth)
> @@ -642,6 +643,8 @@ passphrase when @code{gpg} or @code{gpg2} is run and needs it.")))
>    (package
>      (inherit pinentry-tty)
>      (name "pinentry-qt")
> +    (native-inputs
> +     `(("gettext" ,gnu-gettext)))

It would be good to understand why this is needed, because it shouldn't
be needed for a tarball release.  Can you add a comment explaining the
reason, possibly with "FIXME" in it?

     Thanks,
       Mark

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

* Re: [PATCH 12/13] gnu: avidemux: Fix regression caused by update to qt 5.7.
  2016-08-21 20:13                       ` Mark H Weaver
@ 2016-08-21 21:08                         ` David Craven
  0 siblings, 0 replies; 108+ messages in thread
From: David Craven @ 2016-08-21 21:08 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel

The problem isn't with qt, when adding -std=gnu++11 the build fails.
And I'm not really a C/C++ developer, so sometimes the error messages
look a little intimidating. But yes I'll make the changes...

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

* Re: [PATCH 03/13] gnu: pinentry-qt: Fix regression.
  2016-08-21 20:16                       ` Mark H Weaver
@ 2016-08-21 22:37                         ` David Craven
  0 siblings, 0 replies; 108+ messages in thread
From: David Craven @ 2016-08-21 22:37 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel

I removed gettext and added

(arguments
  `(#:configure-flags '("CXXFLAGS=-std=gnu++11")))

instead. I don't know why adding gettext as a native-input fixed the
build failure.

On Sun, Aug 21, 2016 at 10:16 PM, Mark H Weaver <mhw@netris.org> wrote:
> David Craven <david@craven.ch> writes:
>
>> * gnu/packages/gnupg.scm (pinentry-qt)[native-inputs]: Add gnu-gettext.
>
> The summary line should say "Add gnu-gettext to native-inputs" instead
> of "Fix regression".
>
>> ---
>>  gnu/packages/gnupg.scm | 3 +++
>>  1 file changed, 3 insertions(+)
>>
>> diff --git a/gnu/packages/gnupg.scm b/gnu/packages/gnupg.scm
>> index 7df96b4..fde6018 100644
>> --- a/gnu/packages/gnupg.scm
>> +++ b/gnu/packages/gnupg.scm
>> @@ -28,6 +28,7 @@
>>    #:use-module (gnu packages)
>>    #:use-module (gnu packages adns)
>>    #:use-module (gnu packages curl)
>> +  #:use-module (gnu packages gettext)
>>    #:use-module (gnu packages openldap)
>>    #:use-module (gnu packages perl)
>>    #:use-module (gnu packages pth)
>> @@ -642,6 +643,8 @@ passphrase when @code{gpg} or @code{gpg2} is run and needs it.")))
>>    (package
>>      (inherit pinentry-tty)
>>      (name "pinentry-qt")
>> +    (native-inputs
>> +     `(("gettext" ,gnu-gettext)))
>
> It would be good to understand why this is needed, because it shouldn't
> be needed for a tarball release.  Can you add a comment explaining the
> reason, possibly with "FIXME" in it?
>
>      Thanks,
>        Mark

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

* Re: [PATCH 04/13] gnu: poppler: Update to 0.47.0.
  2016-08-21 19:52                       ` Mark H Weaver
@ 2016-08-21 22:40                         ` David Craven
  0 siblings, 0 replies; 108+ messages in thread
From: David Craven @ 2016-08-21 22:40 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel

> This would entail about 3000 rebuilds.  If it's important, I'd prefer to
> build it out on another branch.  If it's not important, maybe save it
> for core-updates?

I didn't realize it was such an important package. No it's not important,
updating a package is just the first thing I do when something doesn't
work that's all. I'll add the gnu++11 to poppler-qt5 only, that should
require less rebuilds.

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

* Re: [PATCH 07/13] gnu: gpsbabel: Fix regression caused by update to qt 5.7.
  2016-08-21 19:59                       ` Mark H Weaver
@ 2016-08-21 22:55                         ` David Craven
  0 siblings, 0 replies; 108+ messages in thread
From: David Craven @ 2016-08-21 22:55 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel

Here is a sample of the test failures, they all look similar. This is
what I'd call a rounding error, but maybe it's something else?

       <trkpt lat="51.728263833" lon="8.736165997">
-        <ele>95.195312</ele>
+        <ele>95.195313</ele>
         <time>2013-08-15T15:19:01Z</time>
         <speed>522.500000</speed>
         <name>TP8440</name>
@@ -50705,7 +50705,7 @@
         <name>TP8445</name>
       </trkpt>
       <trkpt lat="51.728286721" lon="8.736141201">
-        <ele>95.101562</ele>
+        <ele>95.101563</ele>
         <time>2013-08-15T15:19:01Z</time>
         <speed>511.944458</speed>
         <name>TP8446</name>

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

* Re: [PATCH 11/13] gnu: vlc: Fix regression caused by update to qt 5.7.
  2016-08-21 20:10                       ` Mark H Weaver
@ 2016-08-21 23:06                         ` David Craven
  2016-08-21 23:16                           ` Leo Famulari
  0 siblings, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-21 23:06 UTC (permalink / raw)
  To: Mark H Weaver; +Cc: guix-devel

libxi is required for it to build, I don't know why it worked before.
libinput is not strictly required, but I saw libinput.h not found
falling back to ... and since libinput is already in qt's dependency
graph, we can add it. But I'll make it a separate commit.

I left the qtbase string as documentation, as has been done in other
places. Do you think it's a bad idea (to not use the same string as
the package name)?

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

* Re: [PATCH 11/13] gnu: vlc: Fix regression caused by update to qt 5.7.
  2016-08-21 23:06                         ` David Craven
@ 2016-08-21 23:16                           ` Leo Famulari
  2016-08-21 23:19                             ` David Craven
  0 siblings, 1 reply; 108+ messages in thread
From: Leo Famulari @ 2016-08-21 23:16 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

On Mon, Aug 22, 2016 at 01:06:39AM +0200, David Craven wrote:
> I left the qtbase string as documentation, as has been done in other
> places. Do you think it's a bad idea (to not use the same string as
> the package name)?

My understanding is that the 'qt' package is not the same thing as the
'qtbase' package. 'qt' is the all-in-one Qt, whereas 'qtbase' is the
base package of the modularized Qt that Efraim has been working on.

So, I do think it's a good idea to use that string as documentation but,
in this case, I think that using "qtbase" is inappropriate. What do you
think?

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

* Re: [PATCH 11/13] gnu: vlc: Fix regression caused by update to qt 5.7.
  2016-08-21 23:16                           ` Leo Famulari
@ 2016-08-21 23:19                             ` David Craven
  2016-08-21 23:24                               ` Leo Famulari
  2016-08-21 23:24                               ` David Craven
  0 siblings, 2 replies; 108+ messages in thread
From: David Craven @ 2016-08-21 23:19 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

> So, I do think it's a good idea to use that string as documentation but,
> in this case, I think that using "qtbase" is inappropriate. What do you
> think?

So what would be a better string? "needs qtbase"?

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

* Re: [PATCH 11/13] gnu: vlc: Fix regression caused by update to qt 5.7.
  2016-08-21 23:19                             ` David Craven
@ 2016-08-21 23:24                               ` Leo Famulari
  2016-08-21 23:24                               ` David Craven
  1 sibling, 0 replies; 108+ messages in thread
From: Leo Famulari @ 2016-08-21 23:24 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

On Mon, Aug 22, 2016 at 01:19:05AM +0200, David Craven wrote:
> > So, I do think it's a good idea to use that string as documentation but,
> > in this case, I think that using "qtbase" is inappropriate. What do you
> > think?
> 
> So what would be a better string? "needs qtbase"?

I would just use "qt", and mention that it should instead use qtbase in
a code comment.

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

* Re: [PATCH 11/13] gnu: vlc: Fix regression caused by update to qt 5.7.
  2016-08-21 23:19                             ` David Craven
  2016-08-21 23:24                               ` Leo Famulari
@ 2016-08-21 23:24                               ` David Craven
  2016-08-22 12:21                                 ` Mark H Weaver
  1 sibling, 1 reply; 108+ messages in thread
From: David Craven @ 2016-08-21 23:24 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

I ended up doing it like this:
 ("qt" ,qt) ; FIXME: reenable modular qt after update - requires building
 ;("qtbase" ,qtbase) with -std=gnu++11.
 ;("qtx11extras" ,qtx11extras)

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

* Re: [PATCH 11/13] gnu: vlc: Fix regression caused by update to qt 5.7.
  2016-08-21 23:24                               ` David Craven
@ 2016-08-22 12:21                                 ` Mark H Weaver
  0 siblings, 0 replies; 108+ messages in thread
From: Mark H Weaver @ 2016-08-22 12:21 UTC (permalink / raw)
  To: David Craven; +Cc: guix-devel

David Craven <david@craven.ch> writes:

> I ended up doing it like this:
>  ("qt" ,qt) ; FIXME: reenable modular qt after update - requires building
>  ;("qtbase" ,qtbase) with -std=gnu++11.
>  ;("qtx11extras" ,qtx11extras)

Sounds good.  Thanks for taking care of these regressions!

     Mark

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

end of thread, other threads:[~2016-08-22 12:21 UTC | newest]

Thread overview: 108+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-08-16 18:39 [PATCH 01/24] gnu: qt: Add qtquickcontrols David Craven
2016-08-16 18:39 ` [PATCH 02/24] gnu: qt: Add qtquickcontrols2 David Craven
2016-08-16 18:39 ` [PATCH 03/24] gnu: qt: Add qtgraphicaleffects David Craven
2016-08-16 19:29   ` Danny Milosavljevic
2016-08-16 18:39 ` [PATCH 04/24] gnu: databases: Add lmdb David Craven
2016-08-16 18:39 ` [PATCH 05/24] gnu: xml: Add yaml-cpp David Craven
2016-08-16 19:58   ` Alex Kost
2016-08-16 18:39 ` [PATCH 06/24] gnu: xml: Add jsoncpp David Craven
2016-08-16 19:28   ` Danny Milosavljevic
2016-08-16 19:59     ` Danny Milosavljevic
2016-08-16 20:00   ` Alex Kost
2016-08-16 18:39 ` [PATCH 07/24] gnu: compression: Add snappy David Craven
2016-08-16 19:45   ` Alex Kost
2016-08-16 18:39 ` [PATCH 08/24] gnu: xorg: Add lndir David Craven
2016-08-16 19:53   ` Alex Kost
2016-08-16 18:39 ` [PATCH 09/24] gnu: freedesktop: Add wayland-protocols David Craven
2016-08-16 19:51   ` Danny Milosavljevic
2016-08-16 20:02   ` Alex Kost
2016-08-16 18:39 ` [PATCH 10/24] gnu: networkmanager-qt: Propagate network-manager David Craven
2016-08-16 20:07   ` Alex Kost
2016-08-16 18:39 ` [PATCH 11/24] gnu: modemmanager-qt: Propagate modem-manager David Craven
2016-08-16 18:39 ` [PATCH 12/24] gnu: gnome: Add mobile-broadband-provider-info David Craven
2016-08-16 19:53   ` Danny Milosavljevic
2016-08-17  9:33   ` Mark H Weaver
2016-08-17  9:37     ` David Craven
2016-08-16 18:39 ` [PATCH 13/24] gnu: polkit-qt: Use modular qt David Craven
2016-08-16 19:17   ` Danny Milosavljevic
2016-08-16 21:00     ` Leo Famulari
2016-08-16 18:39 ` [PATCH 14/24] gnu: gstreamer: Use license: prefix David Craven
2016-08-16 19:18   ` Danny Milosavljevic
2016-08-16 20:15   ` Alex Kost
2016-08-16 20:29     ` David Craven
2016-08-16 22:38     ` Eric Bavier
2016-08-17  8:02       ` David Craven
2016-08-17  8:47         ` Danny Milosavljevic
2016-08-17  8:53           ` David Craven
2016-08-17  9:17             ` Danny Milosavljevic
2016-08-17  9:20               ` David Craven
2016-08-16 18:39 ` [PATCH 15/24] gnu: gstreamer: Symlink gstconfig.h into include directory David Craven
2016-08-17  9:28   ` Mark H Weaver
2016-08-17  9:35     ` David Craven
2016-08-17  9:57       ` Mark H Weaver
2016-08-17 10:10         ` David Craven
2016-08-16 18:39 ` [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer David Craven
2016-08-16 21:08   ` Leo Famulari
2016-08-16 22:28   ` Eric Bavier
2016-08-16 18:39 ` [PATCH 17/24] gnu: kwidgetsaddons: Fix test failure David Craven
2016-08-16 19:22   ` Danny Milosavljevic
2016-08-16 18:39 ` [PATCH 18/24] gnu: qsynth: Use modular qt David Craven
2016-08-16 19:16   ` Danny Milosavljevic
2016-08-16 18:39 ` [PATCH 19/24] gnu: fritzing: " David Craven
2016-08-16 19:17   ` Danny Milosavljevic
2016-08-16 18:39 ` [PATCH 20/24] gnu: bitcoin-core: " David Craven
2016-08-16 19:23   ` Danny Milosavljevic
2016-08-16 18:39 ` [PATCH 21/24] gnu: gst-plugins-bad: " David Craven
2016-08-16 19:23   ` Danny Milosavljevic
2016-08-16 18:39 ` [PATCH 22/24] gnu: poppler-qt5: " David Craven
2016-08-16 19:24   ` Danny Milosavljevic
2016-08-16 18:39 ` [PATCH 23/24] gnu: qtkeychain: " David Craven
2016-08-16 19:25   ` Danny Milosavljevic
2016-08-16 18:39 ` [PATCH 24/24] gnu: qt: Update qt to 5.7.0 David Craven
2016-08-16 19:41   ` Danny Milosavljevic
2016-08-19  0:28   ` Mark H Weaver
2016-08-19  8:56     ` David Craven
2016-08-19 18:41       ` Mark H Weaver
2016-08-19 18:45         ` David Craven
2016-08-20 15:40           ` David Craven
2016-08-20 18:50             ` Mark H Weaver
2016-08-20 19:50               ` David Craven
2016-08-20 20:06                 ` Efraim Flashner
2016-08-20 21:59                 ` Leo Famulari
2016-08-21 14:08                   ` [PATCH 01/13] gnu: kguiaddons: Fix regression caused by update to qt 5.7 David Craven
2016-08-21 14:08                     ` [PATCH 02/13] gnu: kwidgetsaddons: " David Craven
2016-08-21 19:48                       ` Mark H Weaver
2016-08-21 14:08                     ` [PATCH 03/13] gnu: pinentry-qt: Fix regression David Craven
2016-08-21 20:16                       ` Mark H Weaver
2016-08-21 22:37                         ` David Craven
2016-08-21 14:08                     ` [PATCH 04/13] gnu: poppler: Update to 0.47.0 David Craven
2016-08-21 19:52                       ` Mark H Weaver
2016-08-21 22:40                         ` David Craven
2016-08-21 14:08                     ` [PATCH 05/13] gnu: alsa-modular-synth: Fix regression caused by update to qt 5.7 David Craven
2016-08-21 19:53                       ` Mark H Weaver
2016-08-21 14:08                     ` [PATCH 06/13] gnu: qsynth: " David Craven
2016-08-21 19:55                       ` Mark H Weaver
2016-08-21 14:08                     ` [PATCH 07/13] gnu: gpsbabel: " David Craven
2016-08-21 19:59                       ` Mark H Weaver
2016-08-21 22:55                         ` David Craven
2016-08-21 14:08                     ` [PATCH 08/13] gnu: synthv1: " David Craven
2016-08-21 20:01                       ` Mark H Weaver
2016-08-21 14:08                     ` [PATCH 09/13] gnu: scribus: " David Craven
2016-08-21 20:02                       ` Mark H Weaver
2016-08-21 14:08                     ` [PATCH 10/13] gnu: v4l-utils: " David Craven
2016-08-21 20:04                       ` Mark H Weaver
2016-08-21 14:08                     ` [PATCH 11/13] gnu: vlc: " David Craven
2016-08-21 20:10                       ` Mark H Weaver
2016-08-21 23:06                         ` David Craven
2016-08-21 23:16                           ` Leo Famulari
2016-08-21 23:19                             ` David Craven
2016-08-21 23:24                               ` Leo Famulari
2016-08-21 23:24                               ` David Craven
2016-08-22 12:21                                 ` Mark H Weaver
2016-08-21 14:08                     ` [PATCH 12/13] gnu: avidemux: " David Craven
2016-08-21 20:13                       ` Mark H Weaver
2016-08-21 21:08                         ` David Craven
2016-08-21 19:44                     ` [PATCH 01/13] gnu: kguiaddons: " Mark H Weaver
  -- strict thread matches above, loose matches on Subject: below --
2016-08-17 11:14 [PATCH 16/24] gnu: gstreamer: Add qt-gstreamer David Craven
2016-08-17 15:09 ` Eric Bavier
2016-08-18  0:18   ` David Craven

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.