unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Jaft <jaft.r@outlook.com>
To: Nicolas Goaziou <mail@nicolasgoaziou.fr>
Cc: "52113@debbugs.gnu.org" <52113@debbugs.gnu.org>
Subject: [bug#52113] [PATCH] gnu: Add pnmixer
Date: Thu, 16 Dec 2021 21:40:09 +0000 (UTC)	[thread overview]
Message-ID: <BN8PR19MB2449625F3B00273797F0CF4599779@BN8PR19MB2449.namprd19.prod.outlook.com> (raw)
In-Reply-To: <87v902ail3.fsf@nicolasgoaziou.fr>

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

> On Sunday, December 5, 2021, 02:43:27 PM CST, Nicolas Goaziou <mail@nicolasgoaziou.fr> wrote: 
>
>
>
>
>
> Hello,
>
> Jaft <jaft.r@outlook.com> writes:
> 
> > * gnu/packages/gtk.scm (pnmixer):Add PNMixer
>
> Thank you. Some comments follow.
>
> > +(define-public pnmixer
> > +  (let ([version "0.7.2"])
>
> Nitpick: we don't use square brackets for let. Besides, you do not need
> a binding here, you just need to hard-code it in the version field.

Bah; sorry about that. It was mentioned to me, before, and I must've forgotten to change it before I sent this out. I probably should check my other patch submissions around this time as I probably forgot for those, as well…

> Could you send an updated patch?
>
> Regards,
> -- 
> Nicolas Goaziou

Thanks a ton for looking at my patch; all updates should've been applied and I've attached the new patch to this E-mail.

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: pnmixer2.patch --]
[-- Type: text/x-patch, Size: 2625 bytes --]

* gnu/packages/gtk.scm (pnmixer): Add PNMixer
---
gnu/packages/gtk.scm | 40 ++++++++++++++++++++++++++++++++++++++++
1 file changed, 40 insertions(+)
diff --git a/gnu/packages/gtk.scm b/gnu/packages/gtk.scm
index a7e5d10e10..8b3184389a 100644
--- a/gnu/packages/gtk.scm
+++ b/gnu/packages/gtk.scm
@@ -28,6 +28,7 @@
 ;;; Copyright © 2021 Leo Famulari <leo@famulari.name>
 ;;; Copyright © 2021 Simon Streit <simon@netpanic.org>
 ;;; Copyright © 2021 Maxime Devos <maximedevos@telenet.be>
+;;; Copyright © 2021 Wamm K. D. <jaft.r@outlook.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -52,6 +53,7 @@ (define-module (gnu packages gtk)
   #:use-module (guix download)
   #:use-module (guix git-download)
   #:use-module ((guix build utils) #:select (alist-replace))
+  #:use-module (guix build-system cmake)
   #:use-module (guix build-system glib-or-gtk)
   #:use-module (guix build-system gnu)
   #:use-module (guix build-system meson)
@@ -2833,3 +2835,41 @@ (define-public volctl
     ;; XXX: 'setup.py' says "GPLv2" but nothing says "version 2 only".  Is
     ;; GPLv2+ intended?
     (license license:gpl2)))
+
+(define-public pnmixer
+  (package
+    (name "pnmixer")
+    (version "0.7.2")
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/nicklan/pnmixer/")
+             (commit (string-append "v" version))))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32 "0416pa933ddf4b7ph9zxhk5jppkk7ppcq1aqph6xsrfnka4yb148"))))
+  (build-system cmake-build-system)
+  (arguments `(#:tests? #f))  ; no check target
+  (native-inputs
+   `(("gettext" ,gettext-minimal)
+     ("pkg-config" ,pkg-config)))
+  (inputs
+   `(("alsa-lib" ,alsa-lib)
+     ("glib" ,glib)
+     ("gtk+" ,gtk+)
+     ("libnotify" ,libnotify)
+     ("libx11" ,libx11)))
+  (home-page "https://github.com/nicklan/pnmixer/")
+  (synopsis "Simple mixer application designed to run in system tray")
+  (description "PNMixer is a simple mixer application designed to run in your
+system tray.  It integrates nicely into desktop environments that don't have a
+panel that supports applets and therefore can't run a mixer applet.  In
+particular, it's been used quite a lot with fbpanel and tint2 but should run
+fine in any system tray.
+
+PNMixer is designed to work on systems that use ALSA for sound management.  Any
+other sound driver like OSS or FFADO are, currently, not supported.  There is no
+official PulseAudio support, at the moment, but it seems that PNMixer behaves
+quite well anyway when PA is running.")
+  (license license:gpl3)))

  reply	other threads:[~2021-12-16 22:27 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <529632162.5471310.1637869688182.ref@mail.yahoo.com>
2021-11-25 19:48 ` [bug#52113] [PATCH] gnu: Add pnmixer Jaft
2021-12-05 20:43   ` Nicolas Goaziou
2021-12-16 21:40     ` Jaft [this message]
2021-12-17  8:06       ` bug#52113: " Nicolas Goaziou
2021-12-17 10:34         ` [bug#52113] " Jaft

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=BN8PR19MB2449625F3B00273797F0CF4599779@BN8PR19MB2449.namprd19.prod.outlook.com \
    --to=jaft.r@outlook.com \
    --cc=52113@debbugs.gnu.org \
    --cc=mail@nicolasgoaziou.fr \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).