* [bug#32431] [PATCH] gnu: clementine: Drop dependency on qtwebkit. @ 2018-08-13 19:26 Pierre Langlois 2018-08-13 20:58 ` Leo Famulari 0 siblings, 1 reply; 9+ messages in thread From: Pierre Langlois @ 2018-08-13 19:26 UTC (permalink / raw) To: 32431 [-- Attachment #1: Type: text/plain, Size: 276 bytes --] Hello guix! I realised clementine wasn't building because qtwebkit isn't building either. But, it turns out that while it's an optional dependency according to CMakeList.txt, it isn't actually used anywhere in the code. Here's a patch that removes qtwebkit from its inputs. [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #2: 0001-gnu-clementine-Drop-dependency-on-qtwebkit.patch --] [-- Type: text/x-patch, Size: 931 bytes --] From c32d30001676d80bc9c80ece3ee8c31ec9b72592 Mon Sep 17 00:00:00 2001 From: Pierre Langlois <pierre.langlois@gmx.com> Date: Mon, 13 Aug 2018 09:05:17 +0100 Subject: [PATCH] gnu: clementine: Drop dependency on qtwebkit. QtWebkit is optionally included in CMakeList.txt but isn't actually used anywhere in the code. * gnu/packages/music.scm (clementine)[inputs]: Remove qtwebkit. --- gnu/packages/music.scm | 1 - 1 file changed, 1 deletion(-) diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm index c5ee4293b..cf1e0f82f 100644 --- a/gnu/packages/music.scm +++ b/gnu/packages/music.scm @@ -281,7 +281,6 @@ score, keyboard, guitar, drum and controller views.") ("pulseaudio" ,pulseaudio) ("qtbase" ,qtbase) ("qtx11extras" ,qtx11extras) - ("qtwebkit" ,qtwebkit) ("sqlite" ,sqlite-with-fts3) ("sparsehash" ,sparsehash) ("taglib" ,taglib))) -- 2.18.0 [-- Attachment #3: Type: text/plain, Size: 17 bytes --] Thanks! Pierre ^ permalink raw reply related [flat|nested] 9+ messages in thread
* [bug#32431] [PATCH] gnu: clementine: Drop dependency on qtwebkit. 2018-08-13 19:26 [bug#32431] [PATCH] gnu: clementine: Drop dependency on qtwebkit Pierre Langlois @ 2018-08-13 20:58 ` Leo Famulari 2018-08-13 21:12 ` Pierre Langlois 0 siblings, 1 reply; 9+ messages in thread From: Leo Famulari @ 2018-08-13 20:58 UTC (permalink / raw) To: Pierre Langlois; +Cc: 32431 [-- Attachment #1: Type: text/plain, Size: 1765 bytes --] On Mon, Aug 13, 2018 at 08:26:21PM +0100, Pierre Langlois wrote: > Hello guix! > > I realised clementine wasn't building because qtwebkit isn't building > either. But, it turns out that while it's an optional dependency > according to CMakeList.txt, it isn't actually used anywhere in the > code. Here's a patch that removes qtwebkit from its inputs. > > From c32d30001676d80bc9c80ece3ee8c31ec9b72592 Mon Sep 17 00:00:00 2001 > From: Pierre Langlois <pierre.langlois@gmx.com> > Date: Mon, 13 Aug 2018 09:05:17 +0100 > Subject: [PATCH] gnu: clementine: Drop dependency on qtwebkit. > > QtWebkit is optionally included in CMakeList.txt but isn't actually used > anywhere in the code. > > * gnu/packages/music.scm (clementine)[inputs]: Remove qtwebkit. Thanks! Does clementine work for you with this change? For me, it fails like this: $ clementine 16:58:17.043 INFO main:315 Clementine-qt5 1.3.1 16:58:17.065 WARN Database:275 Couldn't register FTS3 tokenizer : QSqlError("1", "Unable to fetch row", "fts3tokenize disabled") 16:58:17.066 DEBUG Database:449 Applying database schema update 13 from ":/schema/schema-13.sql" 16:58:17.066 ERROR Database:577 db error: QSqlError("1", "Unable to fetch row", "unknown tokenizer: unicode") 16:58:17.066 ERROR Database:578 faulty query: "CREATE VIRTUAL TABLE songs_fts USING fts3(\n ftstitle, ftsalbum, ftsartist, ftsalbumartist, ftscomposer, ftsgenre, ftscomment,\n tokenize=unicode\n)" 16:58:17.066 ERROR Database:579 bound values: QMap() __logging_message__16:58:17.066 ERROR unknown Unable to update music library database [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* [bug#32431] [PATCH] gnu: clementine: Drop dependency on qtwebkit. 2018-08-13 20:58 ` Leo Famulari @ 2018-08-13 21:12 ` Pierre Langlois 2018-08-14 2:31 ` Leo Famulari 0 siblings, 1 reply; 9+ messages in thread From: Pierre Langlois @ 2018-08-13 21:12 UTC (permalink / raw) To: Leo Famulari; +Cc: 32431 Hi Leo, Leo Famulari writes: > On Mon, Aug 13, 2018 at 08:26:21PM +0100, Pierre Langlois wrote: >> Hello guix! >> >> I realised clementine wasn't building because qtwebkit isn't building >> either. But, it turns out that while it's an optional dependency >> according to CMakeList.txt, it isn't actually used anywhere in the >> code. Here's a patch that removes qtwebkit from its inputs. >> > >> From c32d30001676d80bc9c80ece3ee8c31ec9b72592 Mon Sep 17 00:00:00 2001 >> From: Pierre Langlois <pierre.langlois@gmx.com> >> Date: Mon, 13 Aug 2018 09:05:17 +0100 >> Subject: [PATCH] gnu: clementine: Drop dependency on qtwebkit. >> >> QtWebkit is optionally included in CMakeList.txt but isn't actually used >> anywhere in the code. >> >> * gnu/packages/music.scm (clementine)[inputs]: Remove qtwebkit. > > Thanks! > > Does clementine work for you with this change? For me, it fails like > this: > > > $ clementine > 16:58:17.043 INFO main:315 Clementine-qt5 1.3.1 > 16:58:17.065 WARN Database:275 Couldn't register FTS3 tokenizer : QSqlError("1", "Unable to fetch row", "fts3tokenize disabled") > 16:58:17.066 DEBUG Database:449 Applying database schema update 13 from ":/schema/schema-13.sql" > 16:58:17.066 ERROR Database:577 db error: QSqlError("1", "Unable to fetch row", "unknown tokenizer: unicode") > 16:58:17.066 ERROR Database:578 faulty query: "CREATE VIRTUAL TABLE songs_fts USING fts3(\n ftstitle, ftsalbum, ftsartist, ftsalbumartist, ftscomposer, ftsgenre, ftscomment,\n tokenize=unicode\n)" > 16:58:17.066 ERROR Database:579 bound values: QMap() > __logging_message__16:58:17.066 ERROR unknown Unable to update music library database That's strange, it works OK for me. I'm on master, which branch are you on? IIRC this was the error I was getting when sqlite wasn't built with FTS3 support, hence the sqlite-with-fts3 package (although I'm not sure what it is). Thanks, Pierre ^ permalink raw reply [flat|nested] 9+ messages in thread
* [bug#32431] [PATCH] gnu: clementine: Drop dependency on qtwebkit. 2018-08-13 21:12 ` Pierre Langlois @ 2018-08-14 2:31 ` Leo Famulari 2018-08-14 9:24 ` Pierre Langlois 0 siblings, 1 reply; 9+ messages in thread From: Leo Famulari @ 2018-08-14 2:31 UTC (permalink / raw) To: Pierre Langlois; +Cc: 32431 [-- Attachment #1: Type: text/plain, Size: 1884 bytes --] On Mon, Aug 13, 2018 at 10:12:14PM +0100, Pierre Langlois wrote: > That's strange, it works OK for me. I'm on master, which branch are you > on? IIRC this was the error I was getting when sqlite wasn't built with > FTS3 support, hence the sqlite-with-fts3 package (although I'm not sure > what it is). I'm also working on the master branch, Guix on Debian. I can reproduce the error on commit 9285f8bbae9d71139873430c71b7ca7634bda31b like this: ------ $ ./pre-inst-env guix environment --pure --ad-hoc clementine -- clementine 22:30:46.067 INFO main:315 Clementine-qt5 1.3.1 __logging_message__22:30:46.071 WARN unknown QStandardPaths: XDG_RUNTIME_DIR not set, defaulting to '/tmp/runtime-leo' __logging_message__22:30:46.083 WARN unknown QStandardPaths: XDG_RUNTIME_DIR not set, defaulting to '/tmp/runtime-leo' 22:30:46.091 WARN Database:275 Couldn't register FTS3 tokenizer : QSqlError("1", "Unable to fetch row", "fts3tokenize disabled") 22:30:46.092 DEBUG Database:449 Applying database schema update 13 from ":/schema/schema-13.sql" 22:30:46.092 ERROR Database:577 db error: QSqlError("1", "Unable to fetch row", "unknown tokenizer: unicode") 22:30:46.092 ERROR Database:578 faulty query: "CREATE VIRTUAL TABLE songs_fts USING fts3(\n ftstitle, ftsalbum, ftsartist, ftsalbumartist, ftscomposer, ftsgenre, ftscomment,\n tokenize=unicode\n)" 22:30:46.092 ERROR Database:579 bound values: QMap() __logging_message__22:30:46.092 ERROR unknown Unable to update music library database ------ If it works for you on GuixSD or another distro, we should go ahead and push the patch, and find out why it doesn't work for me later. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* [bug#32431] [PATCH] gnu: clementine: Drop dependency on qtwebkit. 2018-08-14 2:31 ` Leo Famulari @ 2018-08-14 9:24 ` Pierre Langlois 2018-08-14 21:32 ` Pierre Langlois 0 siblings, 1 reply; 9+ messages in thread From: Pierre Langlois @ 2018-08-14 9:24 UTC (permalink / raw) To: Leo Famulari; +Cc: 32431 Leo Famulari writes: > On Mon, Aug 13, 2018 at 10:12:14PM +0100, Pierre Langlois wrote: >> That's strange, it works OK for me. I'm on master, which branch are you >> on? IIRC this was the error I was getting when sqlite wasn't built with >> FTS3 support, hence the sqlite-with-fts3 package (although I'm not sure >> what it is). > > I'm also working on the master branch, Guix on Debian. I can reproduce > the error on commit 9285f8bbae9d71139873430c71b7ca7634bda31b like this: > > ------ > $ ./pre-inst-env guix environment --pure --ad-hoc clementine -- clementine > 22:30:46.067 INFO main:315 Clementine-qt5 1.3.1 > __logging_message__22:30:46.071 WARN unknown QStandardPaths: XDG_RUNTIME_DIR not set, defaulting to '/tmp/runtime-leo' > __logging_message__22:30:46.083 WARN unknown QStandardPaths: XDG_RUNTIME_DIR not set, defaulting to '/tmp/runtime-leo' > 22:30:46.091 WARN Database:275 Couldn't register FTS3 tokenizer : QSqlError("1", "Unable to fetch row", "fts3tokenize disabled") > 22:30:46.092 DEBUG Database:449 Applying database schema update 13 from ":/schema/schema-13.sql" > 22:30:46.092 ERROR Database:577 db error: QSqlError("1", "Unable to fetch row", "unknown tokenizer: unicode") > 22:30:46.092 ERROR Database:578 faulty query: "CREATE VIRTUAL TABLE songs_fts USING fts3(\n ftstitle, ftsalbum, ftsartist, ftsalbumartist, ftscomposer, ftsgenre, ftscomment,\n tokenize=unicode\n)" > 22:30:46.092 ERROR Database:579 bound values: QMap() > __logging_message__22:30:46.092 ERROR unknown Unable to update music library database > ------ > > If it works for you on GuixSD or another distro, we should go ahead and > push the patch, and find out why it doesn't work for me later. Actually, now I can reproduce the error. I already had a database created in my home directory by the previous working installation so I didn't see the error. If I remove '.config/Clementine' it crashes the same way. I don't think it's because of this patch though, which means the problem has probably been there a while :-(, I'll investigate when I get the time tonight! Thanks for finding that bug! Pierre ^ permalink raw reply [flat|nested] 9+ messages in thread
* [bug#32431] [PATCH] gnu: clementine: Drop dependency on qtwebkit. 2018-08-14 9:24 ` Pierre Langlois @ 2018-08-14 21:32 ` Pierre Langlois 2018-08-15 0:42 ` Leo Famulari 0 siblings, 1 reply; 9+ messages in thread From: Pierre Langlois @ 2018-08-14 21:32 UTC (permalink / raw) To: Leo Famulari; +Cc: 32431 [-- Attachment #1: Type: text/plain, Size: 2799 bytes --] Hi Leo, Pierre Langlois writes: > Leo Famulari writes: > >> On Mon, Aug 13, 2018 at 10:12:14PM +0100, Pierre Langlois wrote: >>> That's strange, it works OK for me. I'm on master, which branch are you >>> on? IIRC this was the error I was getting when sqlite wasn't built with >>> FTS3 support, hence the sqlite-with-fts3 package (although I'm not sure >>> what it is). >> >> I'm also working on the master branch, Guix on Debian. I can reproduce >> the error on commit 9285f8bbae9d71139873430c71b7ca7634bda31b like this: >> >> ------ >> $ ./pre-inst-env guix environment --pure --ad-hoc clementine -- clementine >> 22:30:46.067 INFO main:315 Clementine-qt5 1.3.1 >> __logging_message__22:30:46.071 WARN unknown QStandardPaths: XDG_RUNTIME_DIR not set, defaulting to '/tmp/runtime-leo' >> __logging_message__22:30:46.083 WARN unknown QStandardPaths: XDG_RUNTIME_DIR not set, defaulting to '/tmp/runtime-leo' >> 22:30:46.091 WARN Database:275 Couldn't register FTS3 tokenizer : QSqlError("1", "Unable to fetch row", "fts3tokenize disabled") >> 22:30:46.092 DEBUG Database:449 Applying database schema update 13 from ":/schema/schema-13.sql" >> 22:30:46.092 ERROR Database:577 db error: QSqlError("1", "Unable to fetch row", "unknown tokenizer: unicode") >> 22:30:46.092 ERROR Database:578 faulty query: "CREATE VIRTUAL TABLE songs_fts USING fts3(\n ftstitle, ftsalbum, ftsartist, ftsalbumartist, ftscomposer, ftsgenre, ftscomment,\n tokenize=unicode\n)" >> 22:30:46.092 ERROR Database:579 bound values: QMap() >> __logging_message__22:30:46.092 ERROR unknown Unable to update music library database >> ------ >> >> If it works for you on GuixSD or another distro, we should go ahead and >> push the patch, and find out why it doesn't work for me later. > > Actually, now I can reproduce the error. I already had a database > created in my home directory by the previous working installation so I > didn't see the error. If I remove '.config/Clementine' it crashes the > same way. I don't think it's because of this patch though, which means > the problem has probably been there a while :-(, I'll investigate when I > get the time tonight! OK, it's working now :-). It was indeed unrelated to qtwebkit so here is a follow-up patch that fixes the issue. I'm not sure of the details, but there is a pull request [0] that allows us to patch clementine to fix this, and on top of this, it allows us to use the generic sqlite! It seems OpenSUSE have also patched clementine this way. WDYT? Thanks, Pierre [0]: https://github.com/clementine-player/Clementine/pull/5669 [-- Attachment #2: 0001-gnu-clementine-Fix-creating-initial-database.patch --] [-- Type: text/x-patch, Size: 5854 bytes --] From 304f288f3be2438fe4ecf536728a6dc5579f04d0 Mon Sep 17 00:00:00 2001 From: Pierre Langlois <pierre.langlois@gmx.com> Date: Tue, 14 Aug 2018 22:17:24 +0100 Subject: [PATCH] gnu: clementine: Fix creating initial database. It seems a recent version of sqlite broke Clementine's first startup. It turns out we can patch clementine to fix the problem instead of providing a different sqlite package: https://github.com/clementine-player/Clementine/pull/5669 * gnu/packages/databases.scm (sqlite-with-fts3): Remove. * gnu/packages/music.scm (clementine)[inputs]: Replace sqlite-with-fts3 with sqlite. [source][patches]: Add clementine-fix-sqlite.patch. * gnu/packages/patches/clementine-fix-sqlite.patch: New file. * gnu/local.mk (dist_patch_DATA): Add it. --- gnu/local.mk | 1 + gnu/packages/databases.scm | 16 +------------ gnu/packages/music.scm | 5 ++-- .../patches/clementine-fix-sqlite.patch | 23 +++++++++++++++++++ 4 files changed, 28 insertions(+), 17 deletions(-) create mode 100644 gnu/packages/patches/clementine-fix-sqlite.patch diff --git a/gnu/local.mk b/gnu/local.mk index 15e7beac6..ffcd7eb85 100644 --- a/gnu/local.mk +++ b/gnu/local.mk @@ -611,6 +611,7 @@ dist_patch_DATA = \ %D%/packages/patches/clang-runtime-asan-build-fixes.patch \ %D%/packages/patches/clang-runtime-esan-build-fixes.patch \ %D%/packages/patches/classpath-aarch64-support.patch \ + %D%/packages/patches/clementine-fix-sqlite.patch \ %D%/packages/patches/clementine-remove-crypto++-dependency.patch \ %D%/packages/patches/clementine-use-openssl.patch \ %D%/packages/patches/clisp-glibc-2.26.patch \ diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm index c148ba499..ba3a3233a 100644 --- a/gnu/packages/databases.scm +++ b/gnu/packages/databases.scm @@ -27,7 +27,7 @@ ;;; Copyright © 2017 Alex Vong <alexvong1995@gmail.com> ;;; Copyright © 2017, 2018 Ben Woodcroft <donttrustben@gmail.com> ;;; Copyright © 2017 Rutger Helling <rhelling@mykolab.com> -;;; Copyright © 2017 Pierre Langlois <pierre.langlois@gmx.com> +;;; Copyright © 2017, 2018 Pierre Langlois <pierre.langlois@gmx.com> ;;; Copyright © 2015, 2017, 2018 Ricardo Wurmus <rekado@elephly.net> ;;; Copyright © 2017 Kristofer Buffington <kristoferbuffington@gmail.com> ;;; Copyright © 2018 Amirouche Boubekki <amirouche@hypermove.net> @@ -1069,20 +1069,6 @@ is in the public domain.") ((#:configure-flags flags) `(cons "--enable-fts5" ,flags)))))) -;; This is used by Clementine. -(define-public sqlite-with-fts3 - (package (inherit sqlite) - (name "sqlite-with-fts3") - (arguments - (substitute-keyword-arguments (package-arguments sqlite) - ((#:configure-flags flags) - `(list (string-append "CFLAGS=-O2 -DSQLITE_SECURE_DELETE " - "-DSQLITE_ENABLE_UNLOCK_NOTIFY " - "-DSQLITE_ENABLE_DBSTAT_VTAB " - "-DSQLITE_ENABLE_FTS3 " - "-DSQLITE_ENABLE_FTS3_PARENTHESIS " - "-DSQLITE_ENABLE_FTS3_TOKENIZER"))))))) - (define-public tdb (package (name "tdb") diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm index cf1e0f82f..7d4aeff99 100644 --- a/gnu/packages/music.scm +++ b/gnu/packages/music.scm @@ -238,7 +238,8 @@ score, keyboard, guitar, drum and controller views.") "tinysvcmdns")) #t)) (patches (search-patches "clementine-use-openssl.patch" - "clementine-remove-crypto++-dependency.patch")))) + "clementine-remove-crypto++-dependency.patch" + "clementine-fix-sqlite.patch")))) (build-system cmake-build-system) (arguments '(#:test-target "clementine_test" @@ -281,7 +282,7 @@ score, keyboard, guitar, drum and controller views.") ("pulseaudio" ,pulseaudio) ("qtbase" ,qtbase) ("qtx11extras" ,qtx11extras) - ("sqlite" ,sqlite-with-fts3) + ("sqlite" ,sqlite) ("sparsehash" ,sparsehash) ("taglib" ,taglib))) (home-page "http://clementine-player.org") diff --git a/gnu/packages/patches/clementine-fix-sqlite.patch b/gnu/packages/patches/clementine-fix-sqlite.patch new file mode 100644 index 000000000..f9d44f907 --- /dev/null +++ b/gnu/packages/patches/clementine-fix-sqlite.patch @@ -0,0 +1,23 @@ +Patch downloaded from https://github.com/clementine-player/Clementine/pull/5669 . + +diff -ruN clementine-1.3.1.565.gd20c2244a.orig/src/core/database.cpp clementine-1.3.1.565.gd20c2244a/src/core/database.cpp +--- clementine-1.3.1.565.gd20c2244a.orig/src/core/database.cpp 2018-07-07 23:59:24.018540126 +0200 ++++ clementine-1.3.1.565.gd20c2244a/src/core/database.cpp 2018-07-08 00:04:47.991551728 +0200 +@@ -265,6 +265,17 @@ + StaticInit(); + + { ++ ++#ifdef SQLITE_DBCONFIG_ENABLE_FTS3_TOKENIZER ++ // In case sqlite>=3.12 is compiled without -DSQLITE_ENABLE_FTS3_TOKENIZER ++ // (generally a good idea due to security reasons) the fts3 support should be enabled explicitly. ++ QVariant v = db.driver()->handle(); ++ if (v.isValid() && qstrcmp(v.typeName(), "sqlite3*") == 0) { ++ sqlite3 *handle = *static_cast<sqlite3**>(v.data()); ++ if (handle) sqlite3_db_config(handle, SQLITE_DBCONFIG_ENABLE_FTS3_TOKENIZER, 1, NULL); ++ } ++#endif ++ + QSqlQuery set_fts_tokenizer(db); + set_fts_tokenizer.prepare("SELECT fts3_tokenizer(:name, :pointer)"); + set_fts_tokenizer.bindValue(":name", "unicode"); -- 2.18.0 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* [bug#32431] [PATCH] gnu: clementine: Drop dependency on qtwebkit. 2018-08-14 21:32 ` Pierre Langlois @ 2018-08-15 0:42 ` Leo Famulari 2018-08-15 6:57 ` Pierre Langlois 0 siblings, 1 reply; 9+ messages in thread From: Leo Famulari @ 2018-08-15 0:42 UTC (permalink / raw) To: Pierre Langlois; +Cc: 32431 [-- Attachment #1.1: Type: text/plain, Size: 816 bytes --] On Tue, Aug 14, 2018 at 10:32:11PM +0100, Pierre Langlois wrote: > * gnu/packages/databases.scm (sqlite-with-fts3): Remove. > * gnu/packages/music.scm (clementine)[inputs]: Replace sqlite-with-fts3 with > sqlite. > [source][patches]: Add clementine-fix-sqlite.patch. > * gnu/packages/patches/clementine-fix-sqlite.patch: New file. > * gnu/local.mk (dist_patch_DATA): Add it. Thanks! With this, I could start Clementine but I couldn't actually play any MP3 or FLAC files. It would fail with a dialog that said: "GStreamer could not create the element: autoaudiosink. Please make sure that you have installed all necessary GStreamer plugins (e.g. OGG and MP3)" But, when I rebuilt Clementine with the missing plugins (as attached), it works. If that change looks good to you I'll push it along with your patches. [-- Attachment #1.2: clementine.diff --] [-- Type: text/plain, Size: 524 bytes --] diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm index 7d4aeff99..2b8ff451b 100644 --- a/gnu/packages/music.scm +++ b/gnu/packages/music.scm @@ -272,6 +272,8 @@ score, keyboard, guitar, drum and controller views.") ("glu" ,glu) ("gstreamer" ,gstreamer) ("gst-plugins-base" ,gst-plugins-base) + ("gst-plugins-good" ,gst-plugins-good) + ("gst-libav" ,gst-libav) ("libcdio" ,libcdio) ("libmygpo-qt" ,libmygpo-qt) ;; TODO: Package libgpod. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply related [flat|nested] 9+ messages in thread
* [bug#32431] [PATCH] gnu: clementine: Drop dependency on qtwebkit. 2018-08-15 0:42 ` Leo Famulari @ 2018-08-15 6:57 ` Pierre Langlois 2018-08-15 18:35 ` bug#32431: " Leo Famulari 0 siblings, 1 reply; 9+ messages in thread From: Pierre Langlois @ 2018-08-15 6:57 UTC (permalink / raw) To: Leo Famulari; +Cc: 32431 Leo Famulari writes: > On Tue, Aug 14, 2018 at 10:32:11PM +0100, Pierre Langlois wrote: >> * gnu/packages/databases.scm (sqlite-with-fts3): Remove. >> * gnu/packages/music.scm (clementine)[inputs]: Replace sqlite-with-fts3 with >> sqlite. >> [source][patches]: Add clementine-fix-sqlite.patch. >> * gnu/packages/patches/clementine-fix-sqlite.patch: New file. >> * gnu/local.mk (dist_patch_DATA): Add it. > > Thanks! > > With this, I could start Clementine but I couldn't actually play > any MP3 or FLAC files. It would fail with a dialog that said: > > "GStreamer could not create the element: autoaudiosink. Please make sure > that you have installed all necessary GStreamer plugins (e.g. OGG and > MP3)" Ah yes, that's expected. when packaging this we agreed not to add any of the gst-plugins-{good,bad,ugly} to the inputs and let the user install what they want instead [0]. Although, while we defenetely shouldn't add 'bad' and 'ugly', I don't see any reason not to add 'good' to make sure it all works out of the box. But then, the dialog hints the user should install them so that's not so bad as it is. Basically, I don't mind either way :-). Thanks for testing! Pierre [0]: https://lists.gnu.org/archive/html/guix-patches/2017-02/msg00192.html > > But, when I rebuilt Clementine with the missing plugins (as attached), > it works. > > If that change looks good to you I'll push it along with your patches. > diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm > index 7d4aeff99..2b8ff451b 100644 > --- a/gnu/packages/music.scm > +++ b/gnu/packages/music.scm > @@ -272,6 +272,8 @@ score, keyboard, guitar, drum and controller views.") > ("glu" ,glu) > ("gstreamer" ,gstreamer) > ("gst-plugins-base" ,gst-plugins-base) > + ("gst-plugins-good" ,gst-plugins-good) > + ("gst-libav" ,gst-libav) > ("libcdio" ,libcdio) > ("libmygpo-qt" ,libmygpo-qt) > ;; TODO: Package libgpod. ^ permalink raw reply [flat|nested] 9+ messages in thread
* bug#32431: [PATCH] gnu: clementine: Drop dependency on qtwebkit. 2018-08-15 6:57 ` Pierre Langlois @ 2018-08-15 18:35 ` Leo Famulari 0 siblings, 0 replies; 9+ messages in thread From: Leo Famulari @ 2018-08-15 18:35 UTC (permalink / raw) To: Pierre Langlois; +Cc: 32431-done [-- Attachment #1: Type: text/plain, Size: 910 bytes --] On Wed, Aug 15, 2018 at 07:57:07AM +0100, Pierre Langlois wrote: > Ah yes, that's expected. when packaging this we agreed not to add any of > the gst-plugins-{good,bad,ugly} to the inputs and let the user install > what they want instead [0]. Although, while we defenetely shouldn't add > 'bad' and 'ugly', I don't see any reason not to add 'good' to make sure > it all works out of the box. But then, the dialog hints the user should > install them so that's not so bad as it is. > > Basically, I don't mind either way :-). Previously, MP3 decoding in GStreamer was in gst-plugins-ugly because MP3 was patented, but the patents expired and now gst-plugins-good includes MP3. I think the package should work as-is for its primary use case (music playback), so I pushed your patches and also added gst-plugins-good and gst-libav, ending with commit 8473f8024ceb8e7249702289b206782de7d4c71c. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2018-08-15 18:36 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2018-08-13 19:26 [bug#32431] [PATCH] gnu: clementine: Drop dependency on qtwebkit Pierre Langlois 2018-08-13 20:58 ` Leo Famulari 2018-08-13 21:12 ` Pierre Langlois 2018-08-14 2:31 ` Leo Famulari 2018-08-14 9:24 ` Pierre Langlois 2018-08-14 21:32 ` Pierre Langlois 2018-08-15 0:42 ` Leo Famulari 2018-08-15 6:57 ` Pierre Langlois 2018-08-15 18:35 ` bug#32431: " Leo Famulari
Code repositories for project(s) associated with this public inbox https://git.savannah.gnu.org/cgit/guix.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).