Subject: Re: bug#26500: [PATCH] gnu: Add mcomix.

From: [email-masked] (Ludovic Courtès)
Date: Fri, 21 Apr 2017 11:00:27 +0200

Hello!

Arun Isaac <[email-masked]> skribis:

> nee writes:

[...]

>> @@ -28,6 +29,7 @@
>>    #:use-module (guix packages)
>>    #:use-module (guix build-system gnu)
>>    #:use-module (guix build-system cmake)
>> +  #:use-module (guix build-system python)
>>    #:use-module (gnu packages autotools)
>>    #:use-module (gnu packages algebra)
>>    #:use-module (gnu packages base)
>> @@ -46,6 +48,7 @@
>>    #:use-module (gnu packages pkg-config)
>>    #:use-module (gnu packages qt)
>>    #:use-module (gnu packages xorg)
>> +  #:use-module (gnu packages python)
>>    #:use-module (gnu packages))
>
> Since these modules are already in alphabetical order, it would be good
> to keep them that way. Insert (gnu packages python) at an appropriate
> position.
>
>> +    (inputs `(("python2-pillow" ,python2-pillow)
>> +              ("python2-pygtk" ,python2-pygtk)
>> +              ("p7zip" ,p7zip)))
>
> Please put these in alphabetical order too. That means p7zip will go on
> the top.
>
>> +    (description "MComix is a customizable image viewer that specializes as
>> +a comic and manga reader.  It supports a variety of container formats
>> +including CBZ, CB7, CBT, LHA.
>> +CBR and rar are currently unsupported, due to unfree dependencies.
>> +For optional pdf support, you can install the mupdf package.")
>
> Can we move the information about RAR and PDF support to a comment above
> the package definition, as done for the octave package?

I agree with the comments, though I also think this shouldn’t block the
patch or drive nee away.  :-)

Ludo’.