guix-devel
[Top][All Lists]
Advanced

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

Re: [PATCH 3/3] gnu: Add mupdf


From: Ludovic Courtès
Subject: Re: [PATCH 3/3] gnu: Add mupdf
Date: Mon, 15 Sep 2014 08:52:51 +0200
User-agent: Gnus/5.130011 (Ma Gnus v0.11) Emacs/24.3 (gnu/linux)

Dušan Xmetov <address@hidden> skribis:

> On 9/15/14, Alex Kost <address@hidden> wrote:
>> 5 * trailing space.
>>
>
> They were generated by diff, won't removing them break the patch?

I’m not sure, and that’s why I think it’s OK to keep trailing spaces in
diffs (and because the tools generate them this way.)

>>
>>> +    (inputs
>>> +      `(("curl" ,curl)
>>> +        ("freetype", freetype)
>>
>> ("freetype" ,freetype)
>>
>
> Yup.
>
>>
>>> +    (synopsis "A lightweight PDF viewer and toolkit.")
>>> +    (description
>>> +      "MuPDF is a C library that implements a PDF and XPS parsing and
>>> rendering
>>> +engine. It is used primarily to render pages into bitmaps, but also
>>> provides
>>> +support for other operations such as searching and listing the table of
>>> +contents and hyperlinks.
>>> +
>>> +The library ships with a rudimentary X11 viewer, and a set of command
>>> line tools
>>> +for batch rendering (pdfdraw), examining the file structure (pdfshow),
>>> and
>>> +rewriting files (pdfclean).")
>>> +    (license license:agpl3+)))
>>
>> I think people usually avoid a period in a synopsis and use a double
>> space after a period in a description.  Also the description should be
>> wrapped by 72 (or 78?) characters in a line.

Indeed.  Actually ‘guix lint mupdf’ would tell you that the synopsis
shouldn’t start with “A” and shouldn’t end with a period.

Thanks in advance!

Ludo’.



reply via email to

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