GNU bug report logs -
#49533
[PATCH] gnu: Add omins-lv2.
Previous Next
Reported by: Thorsten Wilms <t_w_ <at> freenet.de>
Date: Mon, 12 Jul 2021 09:46:02 UTC
Severity: normal
Tags: patch
Done: Ricardo Wurmus <rekado <at> elephly.net>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 49533 in the body.
You can then email your comments to 49533 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
guix-patches <at> gnu.org
:
bug#49533
; Package
guix-patches
.
(Mon, 12 Jul 2021 09:46:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Thorsten Wilms <t_w_ <at> freenet.de>
:
New bug report received and forwarded. Copy sent to
guix-patches <at> gnu.org
.
(Mon, 12 Jul 2021 09:46:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
* gnu/packages/audio.scm (omins-lv2): New variable.
---
gnu/packages/audio.scm | 38 +++++++++++++++++++++++++++++++++++++-
1 file changed, 37 insertions(+), 1 deletion(-)
diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
index 3939d4bc7e..8eceb839ad 100644
--- a/gnu/packages/audio.scm
+++ b/gnu/packages/audio.scm
@@ -15,7 +15,7 @@
;;; Copyright © 2018 Clément Lassieur <clement <at> lassieur.org>
;;; Copyright © 2018 Brett Gilio <brettg <at> gnu.org>
;;; Copyright © 2018, 2019 Marius Bakke <mbakke <at> fastmail.com>
-;;; Copyright © 2018 Thorsten Wilms <t_w_ <at> freenet.de>
+;;; Copyright © 2018, 2021 Thorsten Wilms <t_w_ <at> freenet.de>
;;; Copyright © 2018 Eric Bavier <bavier <at> member.fsf.org>
;;; Copyright © 2018 Brendan Tildesley <mail <at> brendan.scot>
;;; Copyright © 2019, 2021 Pierre Langlois <pierre.langlois <at> gmx.com>
@@ -5410,3 +5410,39 @@ device. There is support for mono and/or stereo and 8 or 16 bit samples.")
"MDA-LV2 is an LV2 port of the MDA plugins. It includes effects and a few
instrument plugins.")
(license license:gpl3+)))
+
+(define-public omins-lv2
+ (package
+ (name "omins-lv2")
+ ;; No release despite being perfectly usable. 0.0.0 seems to be the only
+ ;; version the author ever specified:
+ (version "0.0.0")
+ (source
+ (origin
+ (method git-fetch)
+ (uri
+ (git-reference
+ (url "https://git.drobilla.net/cgit.cgi/omins.lv2.git/")
+ (commit "058f341053067b69a84d4081107fda5058290ff9")))
+ (file-name (git-file-name name version))
+ (sha256
+ (base32 "01hnx4hhbz3ap3bw15s42q4q1mw1mhdjwygq4550wvjfg6k4ga8w"))))
+ (build-system waf-build-system)
+ (arguments
+ `(#:tests? #f ; There are no tests.
+ #:configure-flags
+ (list (string-append "--prefix="
+ (assoc-ref %outputs "out")))))
+ (inputs
+ `(("lv2" ,lv2)))
+ (native-inputs
+ `(("pkg-config" ,pkg-config)))
+ (native-search-paths
+ (list (search-path-specification
+ (variable "LV2_PATH")
+ (files '("lib/lv2")))))
+ (home-page "https://git.drobilla.net/cgit.cgi/omins.lv2.git/")
+ (synopsis "LV2 audio plugins for modular synthesis")
+ (description
+ "Omins-lv2 is a small collection of LV2 audio plugins for modular synthesis.")
+ (license (list license:gpl2+ license:gpl2+))))
--
2.32.0
Information forwarded
to
guix-patches <at> gnu.org
:
bug#49533
; Package
guix-patches
.
(Thu, 22 Jul 2021 04:16:02 GMT)
Full text and
rfc822 format available.
Message #8 received at 49533 <at> debbugs.gnu.org (full text, mbox):
Hello,
Thanks for the patch. Just a quick comment...
Thorsten Wilms <t_w_ <at> freenet.de> writes:
> * gnu/packages/audio.scm (omins-lv2): New variable.
> ---
> gnu/packages/audio.scm | 38 +++++++++++++++++++++++++++++++++++++-
> 1 file changed, 37 insertions(+), 1 deletion(-)
>
> diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
> index 3939d4bc7e..8eceb839ad 100644
> --- a/gnu/packages/audio.scm
> +++ b/gnu/packages/audio.scm
> @@ -15,7 +15,7 @@
> ;;; Copyright © 2018 Clément Lassieur <clement <at> lassieur.org>
> ;;; Copyright © 2018 Brett Gilio <brettg <at> gnu.org>
> ;;; Copyright © 2018, 2019 Marius Bakke <mbakke <at> fastmail.com>
> -;;; Copyright © 2018 Thorsten Wilms <t_w_ <at> freenet.de>
> +;;; Copyright © 2018, 2021 Thorsten Wilms <t_w_ <at> freenet.de>
> ;;; Copyright © 2018 Eric Bavier <bavier <at> member.fsf.org>
> ;;; Copyright © 2018 Brendan Tildesley <mail <at> brendan.scot>
> ;;; Copyright © 2019, 2021 Pierre Langlois <pierre.langlois <at> gmx.com>
> @@ -5410,3 +5410,39 @@ device. There is support for mono and/or stereo and 8 or 16 bit samples.")
> "MDA-LV2 is an LV2 port of the MDA plugins. It includes effects and a few
> instrument plugins.")
> (license license:gpl3+)))
> +
> +(define-public omins-lv2
> + (package
> + (name "omins-lv2")
> + ;; No release despite being perfectly usable. 0.0.0 seems to be the only
> + ;; version the author ever specified:
> + (version "0.0.0")
> + (source
> + (origin
> + (method git-fetch)
> + (uri
> + (git-reference
> + (url "https://git.drobilla.net/cgit.cgi/omins.lv2.git/")
> + (commit "058f341053067b69a84d4081107fda5058290ff9")))
> + (file-name (git-file-name name version))
> + (sha256
> + (base32 "01hnx4hhbz3ap3bw15s42q4q1mw1mhdjwygq4550wvjfg6k4ga8w"))))
> + (build-system waf-build-system)
> + (arguments
> + `(#:tests? #f ; There are no tests.
> + #:configure-flags
> + (list (string-append "--prefix="
> + (assoc-ref %outputs "out")))))
Setting --prefix shouldn't be necessary, as the build systems usually
add that themselves. (And yep, just tried building without it, and the
outputs are identical.)
> + (inputs
> + `(("lv2" ,lv2)))
> + (native-inputs
> + `(("pkg-config" ,pkg-config)))
> + (native-search-paths
> + (list (search-path-specification
> + (variable "LV2_PATH")
> + (files '("lib/lv2")))))
> + (home-page "https://git.drobilla.net/cgit.cgi/omins.lv2.git/")
> + (synopsis "LV2 audio plugins for modular synthesis")
> + (description
> + "Omins-lv2 is a small collection of LV2 audio plugins for modular synthesis.")
> + (license (list license:gpl2+ license:gpl2+))))
--
Sarah
Information forwarded
to
guix-patches <at> gnu.org
:
bug#49533
; Package
guix-patches
.
(Sat, 24 Jul 2021 19:42:02 GMT)
Full text and
rfc822 format available.
Message #11 received at 49533 <at> debbugs.gnu.org (full text, mbox):
* gnu/packages/audio.scm (omins-lv2): New variable.
---
Thanks to Sarah for having a look and noticing the superfluous --prefix.
gnu/packages/audio.scm | 33 +++++++++++++++++++++++++++++++++
1 file changed, 33 insertions(+)
diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
index 3939d4bc7e..b5471c9d3d 100644
--- a/gnu/packages/audio.scm
+++ b/gnu/packages/audio.scm
@@ -5410,3 +5410,36 @@ device. There is support for mono and/or stereo and 8 or 16 bit samples.")
"MDA-LV2 is an LV2 port of the MDA plugins. It includes effects and a few
instrument plugins.")
(license license:gpl3+)))
+
+(define-public omins-lv2
+ (package
+ (name "omins-lv2")
+ ;; No release despite being perfectly usable. 0.0.0 seems to be the only
+ ;; version the author ever specified:
+ (version "0.0.0")
+ (source
+ (origin
+ (method git-fetch)
+ (uri
+ (git-reference
+ (url "https://git.drobilla.net/cgit.cgi/omins.lv2.git/")
+ (commit "058f341053067b69a84d4081107fda5058290ff9")))
+ (file-name (git-file-name name version))
+ (sha256
+ (base32 "01hnx4hhbz3ap3bw15s42q4q1mw1mhdjwygq4550wvjfg6k4ga8w"))))
+ (build-system waf-build-system)
+ (arguments
+ `(#:tests? #f)) ; There are no tests.
+ (inputs
+ `(("lv2" ,lv2)))
+ (native-inputs
+ `(("pkg-config" ,pkg-config)))
+ (native-search-paths
+ (list (search-path-specification
+ (variable "LV2_PATH")
+ (files '("lib/lv2")))))
+ (home-page "https://git.drobilla.net/cgit.cgi/omins.lv2.git/")
+ (synopsis "LV2 audio plugins for modular synthesis")
+ (description
+ "Omins-lv2 is a small collection of LV2 audio plugins for modular synthesis.")
+ (license (list license:gpl2+ license:gpl2+))))
--
2.32.0
Information forwarded
to
guix-patches <at> gnu.org
:
bug#49533
; Package
guix-patches
.
(Thu, 29 Jul 2021 19:54:01 GMT)
Full text and
rfc822 format available.
Message #14 received at 49533 <at> debbugs.gnu.org (full text, mbox):
Hi Thorsten,
Thorsten Wilms <t_w_ <at> freenet.de> skribis:
> * gnu/packages/audio.scm (omins-lv2): New variable.
It LGTM, modulo minor issues:
> +(define-public omins-lv2
> + (package
> + (name "omins-lv2")
Please consider passing it through ./etc/indent-code.el:
https://guix.gnu.org/manual/en/html_node/Formatting-Code.html
> + ;; No release despite being perfectly usable. 0.0.0 seems to be the only
> + ;; version the author ever specified:
> + (version "0.0.0")
> + (source
> + (origin
> + (method git-fetch)
> + (uri
> + (git-reference
> + (url "https://git.drobilla.net/cgit.cgi/omins.lv2.git/")
> + (commit "058f341053067b69a84d4081107fda5058290ff9")))
Please check out
<https://guix.gnu.org/manual/en/html_node/Version-Numbers.html> on how
to compute version numbers in such cases.
> + (native-search-paths
> + (list (search-path-specification
> + (variable "LV2_PATH")
> + (files '("lib/lv2")))))
I’m surprised this is here (in the plugin) rather than in the package
that loads this plugin. There are other LV2 plugins doing that though.
Ricardo, WDYT?
> + (license (list license:gpl2+ license:gpl2+))))
(license license:gpl2+) should be enough. :-)
Could you send an updated patch?
Thank you, and thanks, Sarah!
Ludo’.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#49533
; Package
guix-patches
.
(Wed, 01 Sep 2021 21:30:02 GMT)
Full text and
rfc822 format available.
Message #17 received at 49533 <at> debbugs.gnu.org (full text, mbox):
Hi Thorsten,
Did you have a chance to look into this?
Thanks,
Ludo’.
Ludovic Courtès <ludo <at> gnu.org> skribis:
> Hi Thorsten,
>
> Thorsten Wilms <t_w_ <at> freenet.de> skribis:
>
>> * gnu/packages/audio.scm (omins-lv2): New variable.
>
> It LGTM, modulo minor issues:
>
>> +(define-public omins-lv2
>> + (package
>> + (name "omins-lv2")
>
> Please consider passing it through ./etc/indent-code.el:
>
> https://guix.gnu.org/manual/en/html_node/Formatting-Code.html
>
>> + ;; No release despite being perfectly usable. 0.0.0 seems to be the only
>> + ;; version the author ever specified:
>> + (version "0.0.0")
>> + (source
>> + (origin
>> + (method git-fetch)
>> + (uri
>> + (git-reference
>> + (url "https://git.drobilla.net/cgit.cgi/omins.lv2.git/")
>> + (commit "058f341053067b69a84d4081107fda5058290ff9")))
>
> Please check out
> <https://guix.gnu.org/manual/en/html_node/Version-Numbers.html> on how
> to compute version numbers in such cases.
>
>> + (native-search-paths
>> + (list (search-path-specification
>> + (variable "LV2_PATH")
>> + (files '("lib/lv2")))))
>
> I’m surprised this is here (in the plugin) rather than in the package
> that loads this plugin. There are other LV2 plugins doing that though.
> Ricardo, WDYT?
>
>> + (license (list license:gpl2+ license:gpl2+))))
>
> (license license:gpl2+) should be enough. :-)
>
> Could you send an updated patch?
>
> Thank you, and thanks, Sarah!
>
> Ludo’.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#49533
; Package
guix-patches
.
(Thu, 02 Sep 2021 07:58:02 GMT)
Full text and
rfc822 format available.
Message #20 received at 49533 <at> debbugs.gnu.org (full text, mbox):
Ludovic Courtès <ludo <at> gnu.org> writes:
>> + (native-search-paths
>> + (list (search-path-specification
>> + (variable "LV2_PATH")
>> + (files '("lib/lv2")))))
>
> I’m surprised this is here (in the plugin) rather than in the
> package
> that loads this plugin. There are other LV2 plugins doing that
> though.
> Ricardo, WDYT?
Yes, this looks wrong.
Plugins don’t need to set LV2_PATH; only hosts (such as Ardour,
jalv, and others) do. This must have been an early mistake that
has been copied for other plugins. None of the LV2 hosts that we
offer set LV2_PATH.
--
Ricardo
Information forwarded
to
guix-patches <at> gnu.org
:
bug#49533
; Package
guix-patches
.
(Thu, 02 Sep 2021 08:05:02 GMT)
Full text and
rfc822 format available.
Message #23 received at 49533 <at> debbugs.gnu.org (full text, mbox):
On Wed, 01 Sep 2021 23:29:18 +0200
Ludovic Courtès <ludo <at> gnu.org> wrote:
> Did you have a chance to look into this?
Hi Ludovic, I missed to notice your earlier email. Thanks for it and
this reminder!
> > Please consider passing it through ./etc/indent-code.el:
> >
> > https://guix.gnu.org/manual/en/html_node/Formatting-Code.html
I assumed Emacs would do the right thing as is. My guix-installed Emacs
does not offer a guix-devel-mode. There’s also no ./etc/indent-code.el
on this foreign distro.
> > Please check out
> > <https://guix.gnu.org/manual/en/html_node/Version-Numbers.html> on
> > how to compute version numbers in such cases.
Done. Assuming the very last example shows the pattern to use, I
suspect that there are a lot of packages that should, but do not use
it. At least in audio.scm.
> >> + (native-search-paths
> >> + (list (search-path-specification
> >> + (variable "LV2_PATH")
> >> + (files '("lib/lv2")))))
> >
> > I’m surprised this is here (in the plugin) rather than in the
> > package that loads this plugin. There are other LV2 plugins doing
> > that though. Ricardo, WDYT?
I mean to recall that this was necessary for the first lv2 package I
wrote, but from then on, I may have been cargo-culting. Since my
lilv and sord are still broken, no lv2-host will run. So I can’t even
test this now.
> >> + (license (list license:gpl2+ license:gpl2+))))
> >
> > (license license:gpl2+) should be enough. :-)
Sheesh, I did it again!
New patch version follows in separate email.
Thanks!
--
Thorsten Wilms <t_w_ <at> freenet.de>
Information forwarded
to
guix-patches <at> gnu.org
:
bug#49533
; Package
guix-patches
.
(Thu, 02 Sep 2021 08:58:02 GMT)
Full text and
rfc822 format available.
Message #26 received at 49533 <at> debbugs.gnu.org (full text, mbox):
* gnu/packages/audio.scm (omins-lv2): New variable.
---
gnu/packages/audio.scm | 33 ++++++++++++++++++++++++++++++++-
1 file changed, 32 insertions(+), 1 deletion(-)
diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
index 80cd521773..d135bfdf3d 100644
--- a/gnu/packages/audio.scm
+++ b/gnu/packages/audio.scm
@@ -15,7 +15,7 @@
;;; Copyright © 2018 Clément Lassieur <clement <at> lassieur.org>
;;; Copyright © 2018 Brett Gilio <brettg <at> gnu.org>
;;; Copyright © 2018, 2019 Marius Bakke <mbakke <at> fastmail.com>
-;;; Copyright © 2018 Thorsten Wilms <t_w_ <at> freenet.de>
+;;; Copyright © 2018, 2021 Thorsten Wilms <t_w_ <at> freenet.de>
;;; Copyright © 2018 Eric Bavier <bavier <at> member.fsf.org>
;;; Copyright © 2018 Brendan Tildesley <mail <at> brendan.scot>
;;; Copyright © 2019, 2021 Pierre Langlois <pierre.langlois <at> gmx.com>
@@ -5410,3 +5410,34 @@ device. There is support for mono and/or stereo and 8 or 16 bit samples.")
"MDA-LV2 is an LV2 port of the MDA plugins. It includes effects and a few
instrument plugins.")
(license license:gpl3+)))
+
+(define-public omins-lv2
+ (let ((commit "058f341053067b69a84d4081107fda5058290ff9")
+ (revision "1"))
+ (package
+ (name "omins-lv2")
+ ;; No release despite being perfectly usable. 0.0.0 seems to be the only
+ ;; version the author ever specified:
+ (version (git-version "0.0.0" revision commit))
+ (source
+ (origin
+ (method git-fetch)
+ (uri
+ (git-reference
+ (url "https://git.drobilla.net/cgit.cgi/omins.lv2.git/")
+ (commit commit)))
+ (file-name (git-file-name name version))
+ (sha256
+ (base32 "01hnx4hhbz3ap3bw15s42q4q1mw1mhdjwygq4550wvjfg6k4ga8w"))))
+ (build-system waf-build-system)
+ (arguments
+ `(#:tests? #f)) ; There are no tests.
+ (inputs
+ `(("lv2" ,lv2)))
+ (native-inputs
+ `(("pkg-config" ,pkg-config)))
+ (home-page "https://git.drobilla.net/cgit.cgi/omins.lv2.git/")
+ (synopsis "LV2 audio plugins for modular synthesis")
+ (description
+ "Omins-lv2 is a small collection of LV2 audio plugins for modular synthesis.")
+ (license license:gpl2+))))
--
2.33.0
Information forwarded
to
guix-patches <at> gnu.org
:
bug#49533
; Package
guix-patches
.
(Thu, 02 Sep 2021 09:47:01 GMT)
Full text and
rfc822 format available.
Message #29 received at 49533 <at> debbugs.gnu.org (full text, mbox):
Hi!
Thorsten Wilms <t_w_ <at> freenet.de> skribis:
> On Wed, 01 Sep 2021 23:29:18 +0200
> Ludovic Courtès <ludo <at> gnu.org> wrote:
[...]
>> > Please consider passing it through ./etc/indent-code.el:
>> >
>> > https://guix.gnu.org/manual/en/html_node/Formatting-Code.html
>
> I assumed Emacs would do the right thing as is. My guix-installed Emacs
> does not offer a guix-devel-mode. There’s also no ./etc/indent-code.el
> on this foreign distro.
‘guix-devel-mode’ is part of the ‘emacs-guix’ package (aka. guix.el).
(If you have that, you don’t need indent-code.el, which is itself in the
Guix source tree.)
Thanks,
Ludo’.
Information forwarded
to
guix-patches <at> gnu.org
:
bug#49533
; Package
guix-patches
.
(Tue, 26 Oct 2021 12:48:01 GMT)
Full text and
rfc822 format available.
Message #32 received at 49533 <at> debbugs.gnu.org (full text, mbox):
* gnu/packages/audio.scm (omins-lv2): New Variable.
---
This time with indentation via guix-devel-mode.
gnu/packages/audio.scm | 33 ++++++++++++++++++++++++++++++++-
1 file changed, 32 insertions(+), 1 deletion(-)
diff --git a/gnu/packages/audio.scm b/gnu/packages/audio.scm
index faf106fc68..425af448a0 100644
--- a/gnu/packages/audio.scm
+++ b/gnu/packages/audio.scm
@@ -15,7 +15,7 @@
;;; Copyright © 2018 Clément Lassieur <clement <at> lassieur.org>
;;; Copyright © 2018 Brett Gilio <brettg <at> gnu.org>
;;; Copyright © 2018, 2019 Marius Bakke <mbakke <at> fastmail.com>
-;;; Copyright © 2018 Thorsten Wilms <t_w_ <at> freenet.de>
+;;; Copyright © 2018, 2021 Thorsten Wilms <t_w_ <at> freenet.de>
;;; Copyright © 2018 Eric Bavier <bavier <at> member.fsf.org>
;;; Copyright © 2018 Brendan Tildesley <mail <at> brendan.scot>
;;; Copyright © 2019, 2021 Pierre Langlois <pierre.langlois <at> gmx.com>
@@ -5566,3 +5566,34 @@ (define-public mda-lv2
"MDA-LV2 is an LV2 port of the MDA plugins. It includes effects and a few
instrument plugins.")
(license license:gpl3+)))
+
+(define-public omins-lv2
+ (let ((commit "058f341053067b69a84d4081107fda5058290ff9")
+ (revision "1"))
+ (package
+ (name "omins-lv2")
+ ;; No release despite being perfectly usable. 0.0.0 seems to be the only
+ ;; version the author ever specified:
+ (version (git-version "0.0.0" revision commit))
+ (source
+ (origin
+ (method git-fetch)
+ (uri
+ (git-reference
+ (url "https://git.drobilla.net/cgit.cgi/omins.lv2.git/")
+ (commit commit)))
+ (file-name (git-file-name name version))
+ (sha256
+ (base32 "01hnx4hhbz3ap3bw15s42q4q1mw1mhdjwygq4550wvjfg6k4ga8w"))))
+ (build-system waf-build-system)
+ (arguments
+ `(#:tests? #f)) ; There are no tests.
+ (inputs
+ `(("lv2" ,lv2)))
+ (native-inputs
+ `(("pkg-config" ,pkg-config)))
+ (home-page "https://git.drobilla.net/cgit.cgi/omins.lv2.git/")
+ (synopsis "LV2 audio plugins for modular synthesis")
+ (description
+ "Omins-lv2 is a small collection of LV2 audio plugins for modular synthesis.")
+ (license license:gpl2+))))
--
2.33.1
Reply sent
to
Ricardo Wurmus <rekado <at> elephly.net>
:
You have taken responsibility.
(Sun, 25 Feb 2024 12:18:02 GMT)
Full text and
rfc822 format available.
Notification sent
to
Thorsten Wilms <t_w_ <at> freenet.de>
:
bug acknowledged by developer.
(Sun, 25 Feb 2024 12:18:02 GMT)
Full text and
rfc822 format available.
Message #37 received at 49533-done <at> debbugs.gnu.org (full text, mbox):
I'm sorry for the long delay.
I adjusted the package definition and pushed to the master branch with
commit 386080366c642d887f9156059e4e894cd2d4e9b1.
--
Ricardo
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Wed, 27 Mar 2024 11:24:12 GMT)
Full text and
rfc822 format available.
This bug report was last modified 1 year and 43 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.