[bug#48717] [PATCH] Add lingot

[bug#48717] [PATCH] Add lingot

[Rovanion Luckey]

[-- Attachment #1.1: Type: text/plain, Size: 49 bytes --]

Added a tuner program to the package collection.

[-- Attachment #1.2: Type: text/html, Size: 74 bytes --]

[-- Attachment #2: 0001-gnu-lingot-Add-package-lingot.patch --]
[-- Type: text/x-patch, Size: 2262 bytes --]

From 248c6525564dabe71e130e4f6a48005201d2efdc Mon Sep 17 00:00:00 2001
From: Rovanion Luckey <rovanion.luckey@gmail.com>
Date: Fri, 28 May 2021 17:19:56 +0200
Subject: [PATCH] gnu: lingot: Add package lingot.

* gnu/packages/music.scm (lingot): Add package
---
 gnu/packages/music.scm | 37 +++++++++++++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)

diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm
index 7840e8399a..423c9bd3c7 100644
--- a/gnu/packages/music.scm
+++ b/gnu/packages/music.scm
@@ -6639,3 +6639,40 @@ It is provided as an LV2 plugin and as a standalone Jack application.")
   framework.")
       (home-page "http://shiru.untergrund.net/software.shtml")
       (license license:wtfpl2))))
+
+(define-public lingot
+    (package
+      (name "lingot")
+      (version "1.1.1")
+      (source (origin
+                (method git-fetch)
+                (uri (git-reference
+                      (url "https://github.com/ibancg/lingot")
+                      (commit (string-append "v" version))))
+                (file-name (git-file-name name version))
+                (sha256
+                 (base32
+                  "04lcjzfhddbyskxr2068z609y6x0s2gjx1wl78w0dkxdi459zrn9"))))
+    (build-system gnu-build-system)
+    (native-inputs
+     `(("autoconf" ,autoconf)
+       ("automake" ,automake)
+       ("intltool" ,intltool)
+       ("cunit" ,cunit)
+       ("pkg-config" ,pkg-config)
+       ("libtool" ,libtool)))
+    (inputs
+     `(
+       ("alsa-lib" ,alsa-lib) ; for libasound2
+       ("fftw" ,fftw)
+       ("gtk+" ,gtk+)
+       ("glib" ,glib "bin") ; for GLIB_COMPILE_RESOURCES
+       ("jack" ,jack-2)
+       ("json-c" ,json-c)
+       ("pulseaudio" ,pulseaudio)))
+    (home-page "http://lingot.nongnu.org/")
+    (synopsis "An accurate, configurable musical instrument tuner")
+    (description "LINGOT is a musical instrument tuner. It’s accurate, easy to use, and highly configurable. Originally conceived to tune electric guitars, it can now be used to tune other instruments.
+
+It looks like an analogue tuner, with a gauge indicating the relative shift to a certain note, determined automatically as the closest note to the estimated frequency.")
+    (license license:gpl2)))
-- 
2.30.0

[bug#48717] [PATCH] Add lingot

[Tobias Geerinckx-Rice via Guix-patches via]

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

Rovanion Luckey 写道：
> Added a tuner program to the package collection.

Thank you!

I should add a copyright line for you at the top of the file, once 
you confirm which name & e-mail address to use.

I made the following changes (pay close attention, they might be 
on the test):
- moved glib:bin to native-inputs, which means it can be run at 
  build time even when cross-building (and ‘guix gc --references’ 
  indicates it won't be run at run time anyway),
- the licence is ‘GPL2 or later’,
- removed the leading ‘An’ from the synopsis (in a list, it's just 
  noise),
- wrapped all text to 80 characters and added a double space after 
  full stops.

Be sure to run ‘guix lint’ on new packages, it will warn you about 
some of these and other cool mistakes.

These minor nitpicks will not be on the test but I applied them 
anyway:
- reindented the package (specifically the source form was off?),
- ordered the inputs alphabetically, and put the package somewhere 
  similar,
- removed the libasound comment because that's 99% of why packages 
  use alsa-lib.  Probably 100.  Could the same be said for 
  glib:bin, which I kept?  Perhaps!  I am whimsy.

Kind regards,

T G-R

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 247 bytes --]

[bug#48717] [PATCH] Add lingot

[Rovanion Luckey]

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

You can use the following for the copyright line:

Rovanion Luckey <rovanion.luckey@gmail.com>

I gleaned the license info from GitHub, but I guess it was wrong. Should
really have run guix lint on the package though. Thank you for your
patience.

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

[bug#48717] [PATCH] Add lingot

[Rovanion Luckey]

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

This patch has been included and this issue closed right?

Den fre 28 maj 2021 kl 21:18 skrev Rovanion Luckey <
rovanion.luckey@gmail.com>:

> You can use the following for the copyright line:
>
> Rovanion Luckey <rovanion.luckey@gmail.com>
>
> I gleaned the license info from GitHub, but I guess it was wrong. Should
> really have run guix lint on the package though. Thank you for your
> patience.
>

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

bug#48717: [PATCH] Add lingot

[Efraim Flashner]

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

On Fri, Jul 30, 2021 at 09:03:11PM +0200, Rovanion Luckey wrote:
> This patch has been included and this issue closed right?

Looks like it was committed on May 30th. Closing the bug.


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

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]