all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH 2/2] gnu: Add gnome-mpv.
@ 2016-07-07  0:04 Alex Griffin
  2016-07-07  7:10 ` Ricardo Wurmus
  2016-07-12 15:34 ` ng0
  0 siblings, 2 replies; 7+ messages in thread
From: Alex Griffin @ 2016-07-07  0:04 UTC (permalink / raw)
  To: guix-devel

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

This patch adds gnome-mpv, a simple GTK+ frontend to the mpv video
player.
-- 
Alex Griffin

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0002-gnu-Add-gnome-mpv.patch --]
[-- Type: text/x-patch; name="0002-gnu-Add-gnome-mpv.patch", Size: 1965 bytes --]

From b72a82b9abfe01fa9374d22893cb5a0564eee97e Mon Sep 17 00:00:00 2001
From: Alex Griffin <a@ajgrf.com>
Date: Wed, 6 Jul 2016 18:58:17 -0500
Subject: [PATCH 2/2] gnu: Add gnome-mpv.

* gnu/packages/video.scm (gnome-mpv): New variable.
---
 gnu/packages/video.scm | 28 ++++++++++++++++++++++++++++
 1 file changed, 28 insertions(+)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index ae1813d..e9088fd 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -33,6 +33,7 @@
   #:use-module (guix git-download)
   #:use-module (guix build-system cmake)
   #:use-module (guix build-system gnu)
+  #:use-module (guix build-system glib-or-gtk)
   #:use-module (guix build-system python)
   #:use-module (guix build-system waf)
   #:use-module (gnu packages)
@@ -829,6 +830,33 @@ fork of mplayer2 and MPlayer.  It shares some features with the former
 projects while introducing many more.")
     (license license:gpl2+)))
 
+(define-public gnome-mpv
+  (package
+    (name "gnome-mpv")
+    (version "0.9")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "https://github.com/gnome-mpv/gnome-mpv/releases/download/v"
+                           version "/gnome-mpv-" version ".tar.xz"))
+       (sha256
+        (base32
+         "06pgxl6f3kkgxv8nlmyl7gy3pg55sqf8vgr8m6426mlpm4p3qdn0"))))
+    (native-inputs
+     `(("intltool" ,intltool)
+       ("pkg-config" ,pkg-config)))
+    (inputs
+     `(("gtk+" ,gtk+)
+       ("libepoxy" ,libepoxy)
+       ("mpv" ,mpv)))
+    (build-system glib-or-gtk-build-system)
+    (home-page "https://github.com/gnome-mpv/gnome-mpv")
+    (synopsis "Simple GTK+ frontend for mpv")
+    (description "GNOME MPV is a simple GTK+ frontend for mpv.  GNOME MPV
+interacts with mpv via the client API exported by libmpv, allowing access to
+mpv's powerful playback capabilities.")
+    (license license:gpl3+)))
+
 (define-public libvpx
   (package
     (name "libvpx")
-- 
2.9.0


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH 2/2] gnu: Add gnome-mpv.
  2016-07-07  0:04 [PATCH 2/2] gnu: Add gnome-mpv Alex Griffin
@ 2016-07-07  7:10 ` Ricardo Wurmus
  2016-07-07 14:39   ` Alex Griffin
  2016-07-12 15:34 ` ng0
  1 sibling, 1 reply; 7+ messages in thread
From: Ricardo Wurmus @ 2016-07-07  7:10 UTC (permalink / raw)
  To: Alex Griffin; +Cc: guix-devel


Hi Alex,

> This patch adds gnome-mpv, a simple GTK+ frontend to the mpv video
> player.

Thank you for the patch!

> +(define-public gnome-mpv
> +  (package
> +    (name "gnome-mpv")
> +    (version "0.9")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append "https://github.com/gnome-mpv/gnome-mpv/releases/download/v"
> +                           version "/gnome-mpv-" version ".tar.xz"))

This line looks a bit long to me.  Have you checked with “guix lint
gnome-mpv”?

> +       (sha256
> +        (base32
> +         "06pgxl6f3kkgxv8nlmyl7gy3pg55sqf8vgr8m6426mlpm4p3qdn0"))))
> +    (native-inputs
> +     `(("intltool" ,intltool)
> +       ("pkg-config" ,pkg-config)))
> +    (inputs
> +     `(("gtk+" ,gtk+)
> +       ("libepoxy" ,libepoxy)
> +       ("mpv" ,mpv)))
> +    (build-system glib-or-gtk-build-system)
> +    (home-page "https://github.com/gnome-mpv/gnome-mpv")
> +    (synopsis "Simple GTK+ frontend for mpv")
> +    (description "GNOME MPV is a simple GTK+ frontend for mpv.  GNOME MPV
> +interacts with mpv via the client API exported by libmpv, allowing access to
> +mpv's powerful playback capabilities.")
> +    (license license:gpl3+)))
> +

The rest looks good to me.

~~ Ricardo

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 2/2] gnu: Add gnome-mpv.
  2016-07-07  7:10 ` Ricardo Wurmus
@ 2016-07-07 14:39   ` Alex Griffin
  2016-07-07 17:07     ` Eric Bavier
                       ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: Alex Griffin @ 2016-07-07 14:39 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: guix-devel

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

On Thu, Jul 7, 2016, at 02:10 AM, Ricardo Wurmus wrote:
> > +       (uri (string-append "https://github.com/gnome-mpv/gnome-mpv/releases/download/v"
> > +                           version "/gnome-mpv-" version ".tar.xz"))
> 
> This line looks a bit long to me.  Have you checked with “guix lint
> gnome-mpv”?

Actually, yes I did, and it didn't catch it. Maybe there's a bug in the
linter?

Anyway, here's an updated patch. In addition to correcting the long
line, I also tweaked the synopsis and description to make clear that
this is a media player, for the benefit of anyone who hasn't heard of
mpv.
-- 
Alex Griffin

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0002-gnu-Add-gnome-mpv.patch --]
[-- Type: text/x-patch; name="0002-gnu-Add-gnome-mpv.patch", Size: 2023 bytes --]

From e3f40673b6883bba023eec9eb8995df97d9245fd Mon Sep 17 00:00:00 2001
From: Alex Griffin <a@ajgrf.com>
Date: Wed, 6 Jul 2016 18:58:17 -0500
Subject: [PATCH 2/2] gnu: Add gnome-mpv.

* gnu/packages/video.scm (gnome-mpv): New variable.
---
 gnu/packages/video.scm | 29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)

diff --git a/gnu/packages/video.scm b/gnu/packages/video.scm
index ae1813d..6888e36 100644
--- a/gnu/packages/video.scm
+++ b/gnu/packages/video.scm
@@ -33,6 +33,7 @@
   #:use-module (guix git-download)
   #:use-module (guix build-system cmake)
   #:use-module (guix build-system gnu)
+  #:use-module (guix build-system glib-or-gtk)
   #:use-module (guix build-system python)
   #:use-module (guix build-system waf)
   #:use-module (gnu packages)
@@ -829,6 +830,34 @@ fork of mplayer2 and MPlayer.  It shares some features with the former
 projects while introducing many more.")
     (license license:gpl2+)))
 
+(define-public gnome-mpv
+  (package
+    (name "gnome-mpv")
+    (version "0.9")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "https://github.com/gnome-mpv/gnome-mpv/releases"
+                           "/download/v" version "/gnome-mpv-" version
+                           ".tar.xz"))
+       (sha256
+        (base32
+         "06pgxl6f3kkgxv8nlmyl7gy3pg55sqf8vgr8m6426mlpm4p3qdn0"))))
+    (native-inputs
+     `(("intltool" ,intltool)
+       ("pkg-config" ,pkg-config)))
+    (inputs
+     `(("gtk+" ,gtk+)
+       ("libepoxy" ,libepoxy)
+       ("mpv" ,mpv)))
+    (build-system glib-or-gtk-build-system)
+    (home-page "https://github.com/gnome-mpv/gnome-mpv")
+    (synopsis "GTK+ frontend for the mpv media player")
+    (description "GNOME MPV is a simple GTK+ frontend for the mpv media player.
+GNOME MPV interacts with mpv via the client API exported by libmpv, allowing
+access to mpv's powerful playback capabilities.")
+    (license license:gpl3+)))
+
 (define-public libvpx
   (package
     (name "libvpx")
-- 
2.9.0


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH 2/2] gnu: Add gnome-mpv.
  2016-07-07 14:39   ` Alex Griffin
@ 2016-07-07 17:07     ` Eric Bavier
  2016-07-07 17:17     ` Efraim Flashner
  2016-07-10  4:13     ` Efraim Flashner
  2 siblings, 0 replies; 7+ messages in thread
From: Eric Bavier @ 2016-07-07 17:07 UTC (permalink / raw)
  To: Alex Griffin, Ricardo Wurmus; +Cc: guix-devel



On July 7, 2016 9:39:48 AM CDT, Alex Griffin <a@ajgrf.com> wrote:
>On Thu, Jul 7, 2016, at 02:10 AM, Ricardo Wurmus wrote:
>> > +       (uri (string-append
>"https://github.com/gnome-mpv/gnome-mpv/releases/download/v"
>> > +                           version "/gnome-mpv-" version
>".tar.xz"))
>> 
>> This line looks a bit long to me.  Have you checked with “guix lint
>> gnome-mpv”?
>
>Actually, yes I did, and it didn't catch it. Maybe there's a bug in the
>linter?

The linter is slightly tolerant and will accept lines under 90 characters IIRC. 

>Anyway, here's an updated patch. In addition to correcting the long
>line, I also tweaked the synopsis and description to make clear that
>this is a media player, for the benefit of anyone who hasn't heard of
>mpv.

-- 
Sent from my Android device with K-9 Mail. Please excuse my brevity.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 2/2] gnu: Add gnome-mpv.
  2016-07-07 14:39   ` Alex Griffin
  2016-07-07 17:07     ` Eric Bavier
@ 2016-07-07 17:17     ` Efraim Flashner
  2016-07-10  4:13     ` Efraim Flashner
  2 siblings, 0 replies; 7+ messages in thread
From: Efraim Flashner @ 2016-07-07 17:17 UTC (permalink / raw)
  To: Alex Griffin; +Cc: guix-devel

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

On Thu, Jul 07, 2016 at 09:39:48AM -0500, Alex Griffin wrote:
> On Thu, Jul 7, 2016, at 02:10 AM, Ricardo Wurmus wrote:
> > > +       (uri (string-append "https://github.com/gnome-mpv/gnome-mpv/releases/download/v"
> > > +                           version "/gnome-mpv-" version ".tar.xz"))
> > 
> > This line looks a bit long to me.  Have you checked with “guix lint
> > gnome-mpv”?
> 
> Actually, yes I did, and it didn't catch it. Maybe there's a bug in the
> linter?

The linkter checks that there aren't any lines over 90 characters, to
allow for incredibly long urls and super-indented code.

> 
> Anyway, here's an updated patch. In addition to correcting the long
> line, I also tweaked the synopsis and description to make clear that
> this is a media player, for the benefit of anyone who hasn't heard of
> mpv.
> -- 
> Alex Griffin


-- 
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: 819 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 2/2] gnu: Add gnome-mpv.
  2016-07-07 14:39   ` Alex Griffin
  2016-07-07 17:07     ` Eric Bavier
  2016-07-07 17:17     ` Efraim Flashner
@ 2016-07-10  4:13     ` Efraim Flashner
  2 siblings, 0 replies; 7+ messages in thread
From: Efraim Flashner @ 2016-07-10  4:13 UTC (permalink / raw)
  To: Alex Griffin; +Cc: guix-devel

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

On Thu, Jul 07, 2016 at 09:39:48AM -0500, Alex Griffin wrote:
> On Thu, Jul 7, 2016, at 02:10 AM, Ricardo Wurmus wrote:
> > > +       (uri (string-append "https://github.com/gnome-mpv/gnome-mpv/releases/download/v"
> > > +                           version "/gnome-mpv-" version ".tar.xz"))
> > 
> > This line looks a bit long to me.  Have you checked with “guix lint
> > gnome-mpv”?
> 
> Actually, yes I did, and it didn't catch it. Maybe there's a bug in the
> linter?
> 
> Anyway, here's an updated patch. In addition to correcting the long
> line, I also tweaked the synopsis and description to make clear that
> this is a media player, for the benefit of anyone who hasn't heard of
> mpv.
> -- 
> Alex Griffin

packaged pushed!

-- 
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: 819 bytes --]

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 2/2] gnu: Add gnome-mpv.
  2016-07-07  0:04 [PATCH 2/2] gnu: Add gnome-mpv Alex Griffin
  2016-07-07  7:10 ` Ricardo Wurmus
@ 2016-07-12 15:34 ` ng0
  1 sibling, 0 replies; 7+ messages in thread
From: ng0 @ 2016-07-12 15:34 UTC (permalink / raw)
  To: guix-devel

Alex Griffin <a@ajgrf.com> writes:

> This patch adds gnome-mpv, a simple GTK+ frontend to the mpv video
> player.

Thanks!

I notified upstream about the availibility:
https://github.com/gnome-mpv/gnome-mpv/issues/203
-- 
♥Ⓐ  ng0
For non-prism friendly talk find me on http://www.psyced.org
SecuShare – http://secushare.org

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2016-07-12 15:34 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-07-07  0:04 [PATCH 2/2] gnu: Add gnome-mpv Alex Griffin
2016-07-07  7:10 ` Ricardo Wurmus
2016-07-07 14:39   ` Alex Griffin
2016-07-07 17:07     ` Eric Bavier
2016-07-07 17:17     ` Efraim Flashner
2016-07-10  4:13     ` Efraim Flashner
2016-07-12 15:34 ` ng0

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.