[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
>"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.