[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 2/2] gnu: Add gnome-mpv.

From: Eric Bavier
Subject: Re: [PATCH 2/2] gnu: Add gnome-mpv.
Date: Thu, 07 Jul 2016 12:07:01 -0500
User-agent: K-9 Mail for Android

On July 7, 2016 9:39:48 AM CDT, Alex Griffin <address@hidden> wrote:
>On Thu, Jul 7, 2016, at 02:10 AM, Ricardo Wurmus wrote:
>> > +       (uri (string-append
>> > +                           version "/gnome-mpv-" version
>> 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

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

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

reply via email to

[Prev in Thread] Current Thread [Next in Thread]