unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* bug#26500: [PATCH] gnu: Add mcomix.
@ 2017-04-14 17:50 nee
  2017-04-15  8:36 ` Arun Isaac
  0 siblings, 1 reply; 11+ messages in thread
From: nee @ 2017-04-14 17:50 UTC (permalink / raw)
  To: 26500

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

Hello, here is mcomix.
I tested it with zip, CBZ, 7zip pdf files, and a directory with normal
images, they all work fine.

I also tried unrar, but couldn't extract any archive with it, neither
through mcomix, nor manually through the commandline tool, so this might
be a separate problem.

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

From 7aee659716fab9cfcb9c22eed7ad302696ded374 Mon Sep 17 00:00:00 2001
From: nee <nee.git@cock.li>
Date: Fri, 14 Apr 2017 19:09:40 +0200
Subject: [PATCH] gnu: Add mcomix.

* gnu/packages/image-viewers.scm (mcomix): New variable.
---
 gnu/packages/image-viewers.scm | 30 ++++++++++++++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git a/gnu/packages/image-viewers.scm b/gnu/packages/image-viewers.scm
index 830ce10ea..d1e14f053 100644
--- a/gnu/packages/image-viewers.scm
+++ b/gnu/packages/image-viewers.scm
@@ -28,6 +28,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 +47,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))
 
 (define-public feh
@@ -360,3 +362,31 @@ imaging.  It supports several HDR and LDR image formats, and it can:
 @item Copy EXIF data between sets of images.
 @end itemize\n")
     (license license:gpl2+)))
+
+(define-public mcomix
+  (package
+    (name "mcomix")
+    (version "1.2.1")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "mirror://sourceforge/mcomix/MComix-" version
+                           "/mcomix-" version ".tar.bz2"))
+       (sha256
+        (base32
+         "0fzsf9pklhfs1rzwzj64c0v30b74nk94p93h371rpg45qnfiahvy"))))
+    (build-system python-build-system)
+    ;; Python 2.5 or newer (Python 3 and up is not supported)
+    (arguments `(#:python ,python-2))
+    (propagated-inputs `(("python2-pygtk" ,python2-pygtk)
+                         ("gtk+" ,gtk+)
+                         ("python2-pillow" ,python2-pillow)))
+    (home-page "https://sourceforge.net/p/mcomix/wiki/Home/")
+    (synopsis "Image viewer for comics")
+    (description "A customizable image viewer that specializes as
+a comic and manga reader.  It supports a variety of container formats
+including CBR, CBZ, CB7, CBT, LHA.
+For 7z support install the p7zip package.
+For rar support install the unrar package.
+For pdf support install the mupdf package.")
+    (license license:gpl2)))
-- 
2.12.2


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

* bug#26500: [PATCH] gnu: Add mcomix.
  2017-04-14 17:50 bug#26500: [PATCH] gnu: Add mcomix nee
@ 2017-04-15  8:36 ` Arun Isaac
  2017-04-15  9:09   ` ng0
  0 siblings, 1 reply; 11+ messages in thread
From: Arun Isaac @ 2017-04-15  8:36 UTC (permalink / raw)
  To: 26500


> Hello, here is mcomix.

Thanks for the patch!

> I tested it with zip, CBZ, 7zip pdf files, and a directory with normal
> images, they all work fine.
>
> I also tried unrar, but couldn't extract any archive with it, neither
> through mcomix, nor manually through the commandline tool, so this might
> be a separate problem.

I'm yet to actually test running the application. Will do so and get
back in a while. Meanwhile, other changes.

> +    ;; Python 2.5 or newer (Python 3 and up is not supported)
> +    (arguments `(#:python ,python-2))

You can remove the comment. It's more or less obvious from the
'arguments' section that mcomix needs python 2.

> +    (propagated-inputs `(("python2-pygtk" ,python2-pygtk)
> +                         ("gtk+" ,gtk+)
> +                         ("python2-pillow" ,python2-pillow)))

These are inputs not propagated-inputs. propagated-inputs are only
required for python libraries. For python applications such as mcomix,
the executable gets wrapped in a script which sets the correct
PYTHONPATH variable. Look at $out/bin/mcomix and $out/bin/.mcomix-real

Also, is the gtk+ dependency required? mcomix seems to build and run
fine on my system without it.

> +    (description "A customizable image viewer that specializes as
> +a comic and manga reader.  It supports a variety of container formats
> +including CBR, CBZ, CB7, CBT, LHA.

Please make this "MComix is a ... ". See
https://www.gnu.org/software/guix/manual/html_node/Synopses-and-Descriptions.html

> +    (license license:gpl2)))

This should be license:gpl2+. When no license version is explicitly
specified in the source headers, it is of the "or later" variant. See
https://lists.gnu.org/archive/html/guix-devel/2017-02/msg00026.html

Also, add a copyright header for yourself at the top of the source file
(gnu/packages/image-viewers.scm in this case).

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

* bug#26500: [PATCH] gnu: Add mcomix.
  2017-04-15  8:36 ` Arun Isaac
@ 2017-04-15  9:09   ` ng0
  2017-04-16  8:53     ` Arun Isaac
  0 siblings, 1 reply; 11+ messages in thread
From: ng0 @ 2017-04-15  9:09 UTC (permalink / raw)
  To: Arun Isaac; +Cc: 26500

Arun Isaac transcribed 1.8K bytes:
> 
> > Hello, here is mcomix.
> 
> Thanks for the patch!
> 
> > I tested it with zip, CBZ, 7zip pdf files, and a directory with normal
> > images, they all work fine.
> >
> > I also tried unrar, but couldn't extract any archive with it, neither
> > through mcomix, nor manually through the commandline tool, so this might
> > be a separate problem.
> 
> I'm yet to actually test running the application. Will do so and get
> back in a while. Meanwhile, other changes.
> 
> > +    ;; Python 2.5 or newer (Python 3 and up is not supported)
> > +    (arguments `(#:python ,python-2))
> 
> You can remove the comment. It's more or less obvious from the
> 'arguments' section that mcomix needs python 2.

No, I think a short comment behind or above the #:python keyword is
necessary, in case someone in the future wonders "Hey, why do we package
this for python 2 only and not for python 3" even if both languages are
developed in parallel.
 
> > +    (propagated-inputs `(("python2-pygtk" ,python2-pygtk)
> > +                         ("gtk+" ,gtk+)
> > +                         ("python2-pillow" ,python2-pillow)))
> 
> These are inputs not propagated-inputs. propagated-inputs are only
> required for python libraries. For python applications such as mcomix,
> the executable gets wrapped in a script which sets the correct
> PYTHONPATH variable. Look at $out/bin/mcomix and $out/bin/.mcomix-real
> 
> Also, is the gtk+ dependency required? mcomix seems to build and run
> fine on my system without it.
> 
> > +    (description "A customizable image viewer that specializes as
> > +a comic and manga reader.  It supports a variety of container formats
> > +including CBR, CBZ, CB7, CBT, LHA.
> 
> Please make this "MComix is a ... ". See
> https://www.gnu.org/software/guix/manual/html_node/Synopses-and-Descriptions.html
> 
> > +    (license license:gpl2)))
> 
> This should be license:gpl2+. When no license version is explicitly
> specified in the source headers, it is of the "or later" variant. See
> https://lists.gnu.org/archive/html/guix-devel/2017-02/msg00026.html
> 
> Also, add a copyright header for yourself at the top of the source file
> (gnu/packages/image-viewers.scm in this case).
> 
> 
> 

-- 
PGP and more: https://people.pragmatique.xyz/ng0/

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

* bug#26500: [PATCH] gnu: Add mcomix.
  2017-04-15  9:09   ` ng0
@ 2017-04-16  8:53     ` Arun Isaac
  2017-04-17  4:45       ` nee
  0 siblings, 1 reply; 11+ messages in thread
From: Arun Isaac @ 2017-04-16  8:53 UTC (permalink / raw)
  To: 26500


> No, I think a short comment behind or above the #:python keyword is
> necessary, in case someone in the future wonders "Hey, why do we package
> this for python 2 only and not for python 3" even if both languages are
> developed in parallel.

Ok, it doesn't hurt to keep the comment. Let's keep it.

> +    (description "A customizable image viewer that specializes as
> +a comic and manga reader.  It supports a variety of container formats
> +including CBR, CBZ, CB7, CBT, LHA.
> +For 7z support install the p7zip package.
> +For rar support install the unrar package.
> +For pdf support install the mupdf package.")

Instead of mentioning p7zip, unrar and mupdf in the description, it
would be better if we could make them inputs and patch all references to
the executables in the source with their absolute paths. Could you try this?

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

* bug#26500: [PATCH] gnu: Add mcomix.
  2017-04-16  8:53     ` Arun Isaac
@ 2017-04-17  4:45       ` nee
  2017-04-17 19:21         ` Arun Isaac
  0 siblings, 1 reply; 11+ messages in thread
From: nee @ 2017-04-17  4:45 UTC (permalink / raw)
  To: Arun Isaac, 26500

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

Am 16.04.2017 um 10:53 schrieb Arun Isaac:
> Instead of mentioning p7zip, unrar and mupdf in the description, it
> would be better if we could make them inputs and patch all references to
> the executables in the source with their absolute paths. Could you try this?
I could try that, but mupdf is a pretty heavy dependency. guix size says
it's self-size is 208MiB and total-size is 491MiB.
I personally never needed it with mcomix, so I like that it is optional.
Is there some way to define a package with optional inputs?

For now I'm appending my WIP that should address your previous remarks.

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

From 493a6e0c1c5dd9ef6b454b31be9b80646f201fb4 Mon Sep 17 00:00:00 2001
From: nee <nee.git@cock.li>
Date: Fri, 14 Apr 2017 19:09:40 +0200
Subject: [PATCH] gnu: Add mcomix.

* gnu/packages/image-viewers.scm (mcomix): New variable.
---
 gnu/packages/image-viewers.scm | 30 ++++++++++++++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git a/gnu/packages/image-viewers.scm b/gnu/packages/image-viewers.scm
index 830ce10ea..bfcad99f1 100644
--- a/gnu/packages/image-viewers.scm
+++ b/gnu/packages/image-viewers.scm
@@ -6,6 +6,7 @@
 ;;; Copyright © 2017 Alex Griffin <a@ajgrf.com>
 ;;; Copyright © 2017 ng0 <contact.ng0@cryptolab.net>
 ;;; Copyright © 2017 Mathieu Othacehe <m.othacehe@gmail.com>
+;;; Copyright © 2017 nee <nee-git@hidamari.blue>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -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))
 
 (define-public feh
@@ -360,3 +363,30 @@ imaging.  It supports several HDR and LDR image formats, and it can:
 @item Copy EXIF data between sets of images.
 @end itemize\n")
     (license license:gpl2+)))
+
+(define-public mcomix
+  (package
+    (name "mcomix")
+    (version "1.2.1")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "mirror://sourceforge/mcomix/MComix-" version
+                           "/mcomix-" version ".tar.bz2"))
+       (sha256
+        (base32
+         "0fzsf9pklhfs1rzwzj64c0v30b74nk94p93h371rpg45qnfiahvy"))))
+    (build-system python-build-system)
+    ;; Python 2.5 or newer (Python 3 and up is not supported)
+    (arguments `(#:python ,python-2))
+    (inputs `(("python2-pygtk" ,python2-pygtk)
+              ("python2-pillow" ,python2-pillow)))
+    (home-page "https://sourceforge.net/p/mcomix/wiki/Home/")
+    (synopsis "Image viewer for comics")
+    (description "MComix is a customizable image viewer that specializes as
+a comic and manga reader.  It supports a variety of container formats
+including CBR, CBZ, CB7, CBT, LHA.
+For 7z support install the p7zip package.
+For rar support install the unrar package.
+For pdf support install the mupdf package.")
+    (license license:gpl2+)))
-- 
2.12.2


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

* bug#26500: [PATCH] gnu: Add mcomix.
  2017-04-17  4:45       ` nee
@ 2017-04-17 19:21         ` Arun Isaac
  2017-04-18 22:09           ` nee
  0 siblings, 1 reply; 11+ messages in thread
From: Arun Isaac @ 2017-04-17 19:21 UTC (permalink / raw)
  To: 26500


>> Instead of mentioning p7zip, unrar and mupdf in the description, it
>> would be better if we could make them inputs and patch all references to
>> the executables in the source with their absolute paths. Could you try this?
> I could try that, but mupdf is a pretty heavy dependency. guix size says
> it's self-size is 208MiB and total-size is 491MiB.
> I personally never needed it with mcomix, so I like that it is optional.
> Is there some way to define a package with optional inputs?

As far as I understand, there is no way to specify optional inputs
(yet?). Here are two of many threads discussing this.

https://lists.gnu.org/archive/html/guix-devel/2014-11/msg00549.html
https://lists.gnu.org/archive/html/guix-devel/2015-05/msg00379.html

If mupdf is a heavy dependency and it is not commonly needed, you can
drop it and just package with 7z and rar as dependencies.

> +    (arguments `(#:python ,python-2))
> +    (inputs `(("python2-pygtk" ,python2-pygtk)
> +              ("python2-pillow" ,python2-pillow)))

Sort these inputs alphabetically, as I mentioned in the crawl thread.

LGTM, otherwise! Please send an updated patch. Thanks!

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

* bug#26500: [PATCH] gnu: Add mcomix.
  2017-04-17 19:21         ` Arun Isaac
@ 2017-04-18 22:09           ` nee
  2017-04-19 15:53             ` Arun Isaac
  0 siblings, 1 reply; 11+ messages in thread
From: nee @ 2017-04-18 22:09 UTC (permalink / raw)
  To: 26500

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

Am 17.04.2017 um 21:21 schrieb Arun Isaac:
> 
> If mupdf is a heavy dependency and it is not commonly needed, you can
> drop it and just package with 7z and rar as dependencies.

I have successfully patched in the path for 7zip.
I init the variable that stores it with a unicode string. I make the
assumption that the store path will never contain a ' character.


I read a little bit about unrar and it's issues with mcomix.
Unrar is very old and only supports RAR2, as someone says in the mcomix
issues it is unable to unpack most rar archives created after 2002¹. On
other distro mcomix currently relies on an unfree rar package², and
there is an open ticket³ about switching to a free library like
libarchiver (which supports RAR5 and is also used by file-roller, which
works fine on GuixSD).

I tried unrar on the commandline and it would list file names, but
always fail on unpacking. When called by mcomix, it would always try to
read a password from stdin and freeze the gui.
I will omit rar support for now, since unrar is mostly useless.

> 
> Sort these inputs alphabetically, as I mentioned in the crawl thread.
> 
> LGTM, otherwise! Please send an updated patch. Thanks!
> 
Okay.

(1) https://sourceforge.net/p/mcomix/feature-requests/82/#2bc5
(2) https://sourceforge.net/p/mcomix/feature-requests/82/
(3) https://sourceforge.net/p/mcomix/feature-requests/85/

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

From 6ecc38ecc8ce93d21cb4fa854b6332230c4ea4c9 Mon Sep 17 00:00:00 2001
From: nee <nee.git@cock.li>
Date: Fri, 14 Apr 2017 19:09:40 +0200
Subject: [PATCH] gnu: Add mcomix.

* gnu/packages/image-viewers.scm (mcomix): New variable.
---
 gnu/packages/image-viewers.scm | 40 ++++++++++++++++++++++++++++++++++++++++
 1 file changed, 40 insertions(+)

diff --git a/gnu/packages/image-viewers.scm b/gnu/packages/image-viewers.scm
index 830ce10ea..83ca1ee46 100644
--- a/gnu/packages/image-viewers.scm
+++ b/gnu/packages/image-viewers.scm
@@ -6,6 +6,7 @@
 ;;; Copyright © 2017 Alex Griffin <a@ajgrf.com>
 ;;; Copyright © 2017 ng0 <contact.ng0@cryptolab.net>
 ;;; Copyright © 2017 Mathieu Othacehe <m.othacehe@gmail.com>
+;;; Copyright © 2017 nee <nee-git@hidamari.blue>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -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))
 
 (define-public feh
@@ -360,3 +363,40 @@ imaging.  It supports several HDR and LDR image formats, and it can:
 @item Copy EXIF data between sets of images.
 @end itemize\n")
     (license license:gpl2+)))
+
+(define-public mcomix
+  (package
+    (name "mcomix")
+    (version "1.2.1")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "mirror://sourceforge/mcomix/MComix-" version
+                                  "/mcomix-" version ".tar.bz2"))
+              (sha256
+               (base32
+                "0fzsf9pklhfs1rzwzj64c0v30b74nk94p93h371rpg45qnfiahvy"))))
+    (build-system python-build-system)
+    (arguments
+     ;; Python 2.5 or newer (Python 3 and up is not supported)
+     `(#:python ,python-2
+       #:phases
+       (modify-phases %standard-phases
+         (add-after 'unpack 'patch
+           (lambda* (#:key inputs #:allow-other-keys)
+             (let ((p7zip (assoc-ref inputs "p7zip")))
+               ;; insert absolute path to 7z executable
+               (substitute* "mcomix/archive/sevenzip_external.py"
+                 (("_7z_executable = -1")
+                  (string-append "_7z_executable = u'" p7zip "/bin/7z'"))))
+             #t)))))
+    (inputs `(("python2-pillow" ,python2-pillow)
+              ("python2-pygtk" ,python2-pygtk)
+              ("p7zip" ,p7zip)))
+    (home-page "https://sourceforge.net/p/mcomix/wiki/Home/")
+    (synopsis "Image viewer for comics")
+    (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.")
+    (license license:gpl2+)))
-- 
2.12.2


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

* bug#26500: [PATCH] gnu: Add mcomix.
  2017-04-18 22:09           ` nee
@ 2017-04-19 15:53             ` Arun Isaac
  2017-04-19 19:54               ` ng0
  2017-04-21  9:00               ` Ludovic Courtès
  0 siblings, 2 replies; 11+ messages in thread
From: Arun Isaac @ 2017-04-19 15:53 UTC (permalink / raw)
  To: 26500


nee writes:

>> If mupdf is a heavy dependency and it is not commonly needed, you can
>> drop it and just package with 7z and rar as dependencies.
>
> I have successfully patched in the path for 7zip.
> I init the variable that stores it with a unicode string. I make the
> assumption that the store path will never contain a ' character.

> I read a little bit about unrar and it's issues with mcomix.
> Unrar is very old and only supports RAR2, as someone says in the mcomix
> issues it is unable to unpack most rar archives created after 2002¹. On
> other distro mcomix currently relies on an unfree rar package², and
> there is an open ticket³ about switching to a free library like
> libarchiver (which supports RAR5 and is also used by file-roller, which
> works fine on GuixSD).
>
> I tried unrar on the commandline and it would list file names, but
> always fail on unpacking. When called by mcomix, it would always try to
> read a password from stdin and freeze the gui.
> I will omit rar support for now, since unrar is mostly useless.

Ok, that's fine.

> @@ -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?

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

* bug#26500: [PATCH] gnu: Add mcomix.
  2017-04-19 15:53             ` Arun Isaac
@ 2017-04-19 19:54               ` ng0
  2017-04-21  9:00               ` Ludovic Courtès
  1 sibling, 0 replies; 11+ messages in thread
From: ng0 @ 2017-04-19 19:54 UTC (permalink / raw)
  To: Arun Isaac; +Cc: 26500

Arun Isaac transcribed 2.4K bytes:
> 
> nee writes:
> 
> >> If mupdf is a heavy dependency and it is not commonly needed, you can
> >> drop it and just package with 7z and rar as dependencies.
> >
> > I have successfully patched in the path for 7zip.
> > I init the variable that stores it with a unicode string. I make the
> > assumption that the store path will never contain a ' character.
> 
> > I read a little bit about unrar and it's issues with mcomix.
> > Unrar is very old and only supports RAR2, as someone says in the mcomix
> > issues it is unable to unpack most rar archives created after 2002¹. On
> > other distro mcomix currently relies on an unfree rar package², and
> > there is an open ticket³ about switching to a free library like
> > libarchiver (which supports RAR5 and is also used by file-roller, which
> > works fine on GuixSD).
> >
> > I tried unrar on the commandline and it would list file names, but
> > always fail on unpacking. When called by mcomix, it would always try to
> > read a password from stdin and freeze the gui.
> > I will omit rar support for now, since unrar is mostly useless.

Oh... this could be why zathura-cb has massive issues with most .cbr comics
I have! I had some ideas, but thanks for explaining this issue.

> Ok, that's fine.
> 
> > @@ -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?
> 
> 
> 

-- 
PGP and more: https://people.pragmatique.xyz/ng0/

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

* bug#26500: [PATCH] gnu: Add mcomix.
  2017-04-19 15:53             ` Arun Isaac
  2017-04-19 19:54               ` ng0
@ 2017-04-21  9:00               ` Ludovic Courtès
  2017-04-21 17:57                 ` Arun Isaac
  1 sibling, 1 reply; 11+ messages in thread
From: Ludovic Courtès @ 2017-04-21  9:00 UTC (permalink / raw)
  To: Arun Isaac, nee; +Cc: 26500

Hello!

Arun Isaac <arunisaac@systemreboot.net> 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’.

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

* bug#26500: [PATCH] gnu: Add mcomix.
  2017-04-21  9:00               ` Ludovic Courtès
@ 2017-04-21 17:57                 ` Arun Isaac
  0 siblings, 0 replies; 11+ messages in thread
From: Arun Isaac @ 2017-04-21 17:57 UTC (permalink / raw)
  To: 26500-done


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

Ok, pushed with changes! 

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

end of thread, other threads:[~2017-04-21 17:59 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-04-14 17:50 bug#26500: [PATCH] gnu: Add mcomix nee
2017-04-15  8:36 ` Arun Isaac
2017-04-15  9:09   ` ng0
2017-04-16  8:53     ` Arun Isaac
2017-04-17  4:45       ` nee
2017-04-17 19:21         ` Arun Isaac
2017-04-18 22:09           ` nee
2017-04-19 15:53             ` Arun Isaac
2017-04-19 19:54               ` ng0
2017-04-21  9:00               ` Ludovic Courtès
2017-04-21 17:57                 ` Arun Isaac

Code repositories for project(s) associated with this public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).