* [bug#38129] [PATCH] gnu: Add sherlock-lv2. @ 2019-11-08 11:05 Alexandros Theodotou 2019-11-10 21:31 ` Marius Bakke 0 siblings, 1 reply; 4+ messages in thread From: Alexandros Theodotou @ 2019-11-08 11:05 UTC (permalink / raw) To: 38129 From 5207795bcce201e019c33bb0274e06a460cf6c3f Mon Sep 17 00:00:00 2001 From: Alexandros Theodotou <alex@zrythm.org> Date: Fri, 8 Nov 2019 10:59:52 +0000 Subject: [PATCH] gnu: Add sherlock-lv2. * gnu/packages/music.scm: (sherlock-lv2): New variable. --- gnu/packages/music.scm | 34 ++++++++++++++++++++++++++++++++++ 1 file changed, 34 insertions(+) diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm index a30765e87f..95cc0ff93c 100644 --- a/gnu/packages/music.scm +++ b/gnu/packages/music.scm @@ -24,6 +24,7 @@ ;;; Copyright © 2019 Jakob L. Kreuze <zerodaysfordays@sdf.lonestar.org> ;;; Copyright © 2019 raingloom <raingloom@protonmail.com> ;;; Copyright © 2019 David Wilson <david@daviwil.com> +;;; Copyright © 2019 Alexandros Theodotou <alex@zrythm.org> ;;; ;;; This file is part of GNU Guix. ;;; @@ -4699,3 +4700,36 @@ You can also get metadata about the playing track such as the artist and title for integration into status line generators or other command-line tools.") (home-page "https://github.com/altdesktop/playerctl") (license license:lgpl3+))) + +(define-public sherlock-lv2 + (package + (name "sherlock-lv2") + (version "0.20.0") + (source + (origin + (method url-fetch) + (uri (string-append + "https://git.open-music-kontrollers.ch/lv2/" + "sherlock.lv2/snapshot/sherlock.lv2-" + version ".tar.xz")) + (sha256 + (base32 + "1c5xajpss9h8lbyx160bbzg8va50n2d74qwnxig9sf468rzmha1y")))) + (build-system meson-build-system) + (inputs + `(("glibc" ,glibc) + ("libx11" ,libx11) + ("mesa" ,mesa) + ("serd" ,serd) + ("sratom" ,sratom))) + (native-inputs + `(("flex" ,flex) + ("lv2" ,lv2) + ("sord" ,sord) + ("pkg-config" ,pkg-config))) + (synopsis "Investigative LV2 plugin bundle") + (description "The Sherlock plugin bundle contains LV2 plugins for +visualizing LV2 atom, MIDI and OSC events. They can be used for monitoring and +debugging of event signal flows inside plugin graphs.") + (home-page "https://open-music-kontrollers.ch/lv2/sherlock/") + (license license:artistic2.0))) -- 2.24.0 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* [bug#38129] [PATCH] gnu: Add sherlock-lv2. 2019-11-08 11:05 [bug#38129] [PATCH] gnu: Add sherlock-lv2 Alexandros Theodotou @ 2019-11-10 21:31 ` Marius Bakke 2019-11-11 1:35 ` Alexandros Theodotou 0 siblings, 1 reply; 4+ messages in thread From: Marius Bakke @ 2019-11-10 21:31 UTC (permalink / raw) To: Alexandros Theodotou, 38129 [-- Attachment #1: Type: text/plain, Size: 1862 bytes --] Alexandros, Thanks for these patches. I think your MUA might have mangled these patches somehow, as many of them are "corrupt" according to git. Can you try to resend these? I'll comment on the patches individually meanwhile. Alexandros Theodotou <alex@zrythm.org> writes: > From 5207795bcce201e019c33bb0274e06a460cf6c3f Mon Sep 17 00:00:00 2001 > From: Alexandros Theodotou <alex@zrythm.org> > Date: Fri, 8 Nov 2019 10:59:52 +0000 > Subject: [PATCH] gnu: Add sherlock-lv2. > > * gnu/packages/music.scm: (sherlock-lv2): New variable. [...] > +(define-public sherlock-lv2 > + (package > + (name "sherlock-lv2") > + (version "0.20.0") > + (source > + (origin > + (method url-fetch) > + (uri (string-append > + "https://git.open-music-kontrollers.ch/lv2/" > + "sherlock.lv2/snapshot/sherlock.lv2-" > + version ".tar.xz")) > + (sha256 > + (base32 > + "1c5xajpss9h8lbyx160bbzg8va50n2d74qwnxig9sf468rzmha1y")))) > + (build-system meson-build-system) > + (inputs > + `(("glibc" ,glibc) I don't think this is needed? > + ("libx11" ,libx11) > + ("mesa" ,mesa) > + ("serd" ,serd) > + ("sratom" ,sratom))) > + (native-inputs > + `(("flex" ,flex) > + ("lv2" ,lv2) > + ("sord" ,sord) I don't know anything about LV2 and sord, are these inputs really only used for the build process itself? :-) > + ("pkg-config" ,pkg-config))) > + (synopsis "Investigative LV2 plugin bundle") > + (description "The Sherlock plugin bundle contains LV2 plugins for > +visualizing LV2 atom, MIDI and OSC events. They can be used for > monitoring and > +debugging of event signal flows inside plugin graphs.") Two spaces after periods, as per 'guix lint'. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 487 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
* [bug#38129] [PATCH] gnu: Add sherlock-lv2. 2019-11-10 21:31 ` Marius Bakke @ 2019-11-11 1:35 ` Alexandros Theodotou 2019-11-13 22:23 ` bug#38129: " Marius Bakke 0 siblings, 1 reply; 4+ messages in thread From: Alexandros Theodotou @ 2019-11-11 1:35 UTC (permalink / raw) To: Marius Bakke, 38129 [-- Attachment #1.1: Type: text/plain, Size: 1810 bytes --] Hi Marius, > I think your MUA might have mangled these > patches somehow, as many of them are "corrupt" according to git. Can > you try to resend these? I'll comment on the patches individually > meanwhile. Sorry about that. I got Gnome evolution sorted out now so hopefully it's better. send-mail doesn't work for me at the moment so I'll add the patches as attachments. > > + `(("glibc" ,glibc) > > I don't think this is needed? I guess it's not, I removed it. > > + ("libx11" ,libx11) > > + ("mesa" ,mesa) > > + ("serd" ,serd) > > + ("sratom" ,sratom))) > > + (native-inputs > > + `(("flex" ,flex) > > + ("lv2" ,lv2) > > + ("sord" ,sord) > > I don't know anything about LV2 and sord, are these inputs really > only > used for the build process itself? :-) Sord is an input for sratom (which is in the inputs above), but the build fails if I remove this from the native-inputs. For LV2, technically, it's only needed in the build process since it's just a header file with specifications. Since this is an LV2 plugin, a host that will be loading it will include LV2 itself and will have it as an input, so the plugin doesn't really need it. It doesn't hurt to add it as an input either, but I think having it as a build input is more "correct". By the way, the Arch packagers do the same for most LV2 plugins. > > + ("pkg-config" ,pkg-config))) > > + (synopsis "Investigative LV2 plugin bundle") > > + (description "The Sherlock plugin bundle contains LV2 plugins > > for > > +visualizing LV2 atom, MIDI and OSC events. They can be used for > > monitoring and > > +debugging of event signal flows inside plugin graphs.") > > Two spaces after periods, as per 'guix lint'. Fixed, thank you! [-- Attachment #1.2: 0001-gnu-Add-sherlock-lv2.patch --] [-- Type: text/x-patch, Size: 1796 bytes --] From 8bda974a93874bcf366ba78bdf3329f73245e90f Mon Sep 17 00:00:00 2001 From: Alexandros Theodotou <alex@zrythm.org> Date: Fri, 8 Nov 2019 10:59:52 +0000 Subject: [PATCH] gnu: Add sherlock-lv2. * gnu/packages/music.scm: (sherlock-lv2): New variable. --- gnu/packages/music.scm | 31 +++++++++++++++++++++++++++++++ 1 file changed, 31 insertions(+) diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm index 2f023685df..690434202e 100644 --- a/gnu/packages/music.scm +++ b/gnu/packages/music.scm @@ -4737,3 +4737,34 @@ for integration into status line generators or other command-line tools.") effects. It contains a bitcrusher, delay, distortion, equalizer, compressor, and reverb.") (license license:gpl2+))) + +(define-public sherlock-lv2 + (package + (name "sherlock-lv2") + (version "0.20.0") + (source + (origin + (method url-fetch) + (uri (string-append + "https://git.open-music-kontrollers.ch/lv2/" + "sherlock.lv2/snapshot/sherlock.lv2-" + version ".tar.xz")) + (sha256 + (base32 + "1c5xajpss9h8lbyx160bbzg8va50n2d74qwnxig9sf468rzmha1y")))) + (build-system meson-build-system) + (inputs + `(("libx11" ,libx11) + ("mesa" ,mesa) + ("sratom" ,sratom))) + (native-inputs + `(("flex" ,flex) + ("lv2" ,lv2) + ("sord" ,sord) + ("pkg-config" ,pkg-config))) + (synopsis "Investigative LV2 plugin bundle") + (description "The Sherlock plugin bundle contains LV2 plugins for +visualizing LV2 atom, MIDI and OSC events. They can be used for monitoring +and debugging of event signal flows inside plugin graphs.") + (home-page "https://open-music-kontrollers.ch/lv2/sherlock/") + (license license:artistic2.0))) -- 2.24.0 [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply related [flat|nested] 4+ messages in thread
* bug#38129: [PATCH] gnu: Add sherlock-lv2. 2019-11-11 1:35 ` Alexandros Theodotou @ 2019-11-13 22:23 ` Marius Bakke 0 siblings, 0 replies; 4+ messages in thread From: Marius Bakke @ 2019-11-13 22:23 UTC (permalink / raw) To: Alexandros Theodotou, 38129-done [-- Attachment #1: Type: text/plain, Size: 2251 bytes --] Alexandros Theodotou <alex@zrythm.org> writes: > Hi Marius, > >> I think your MUA might have mangled these >> patches somehow, as many of them are "corrupt" according to git. Can >> you try to resend these? I'll comment on the patches individually >> meanwhile. > Sorry about that. I got Gnome evolution sorted out now so hopefully > it's better. send-mail doesn't work for me at the moment so I'll add > the patches as attachments. Thanks! >> > + ("libx11" ,libx11) >> > + ("mesa" ,mesa) >> > + ("serd" ,serd) >> > + ("sratom" ,sratom))) >> > + (native-inputs >> > + `(("flex" ,flex) >> > + ("lv2" ,lv2) >> > + ("sord" ,sord) >> >> I don't know anything about LV2 and sord, are these inputs really >> only >> used for the build process itself? :-) > Sord is an input for sratom (which is in the inputs above), but the > build fails if I remove this from the native-inputs. Right, I think I see the problem. sratom's pkg-config file has this line: Requires: lv2 serd-0 sord-0 Which means that these need to be available when build systems run e.g. 'pkg-config --cflags sratom-0'. This is the most common use of propagated-inputs in Guix. I went ahead and propagated these from 'sratom', and removed the 'lv2' and 'sord' inputs. Let me know if you think something is off! > For LV2, technically, it's only needed in the build process since it's > just a header file with specifications. Since this is an LV2 plugin, a > host that will be loading it will include LV2 itself and will have it > as an input, so the plugin doesn't really need it. It doesn't hurt to > add it as an input either, but I think having it as a build input is > more "correct". By the way, the Arch packagers do the same for most LV2 > plugins. Right. Header files are usually architecture-independent, so having them as native-inputs should be safe. > From 8bda974a93874bcf366ba78bdf3329f73245e90f Mon Sep 17 00:00:00 2001 > From: Alexandros Theodotou <alex@zrythm.org> > Date: Fri, 8 Nov 2019 10:59:52 +0000 > Subject: [PATCH] gnu: Add sherlock-lv2. > > * gnu/packages/music.scm: (sherlock-lv2): New variable. Applied (with the dropped inputs), thanks! [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 487 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-11-13 22:24 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-11-08 11:05 [bug#38129] [PATCH] gnu: Add sherlock-lv2 Alexandros Theodotou 2019-11-10 21:31 ` Marius Bakke 2019-11-11 1:35 ` Alexandros Theodotou 2019-11-13 22:23 ` bug#38129: " Marius Bakke
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).