From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54888) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1d0bK1-0007VX-IC for guix-patches@gnu.org; Tue, 18 Apr 2017 18:10:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1d0bJy-0008Nh-AP for guix-patches@gnu.org; Tue, 18 Apr 2017 18:10:05 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:57103) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1d0bJy-0008Nb-1S for guix-patches@gnu.org; Tue, 18 Apr 2017 18:10:02 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1d0bJx-0004Dq-Pm for guix-patches@gnu.org; Tue, 18 Apr 2017 18:10:01 -0400 Subject: bug#26500: [PATCH] gnu: Add mcomix. Resent-Message-ID: References: <633024d3.AEEAJmX48o8AAAAAAAAAAAO1C4gAAAACwQwAAAAAAAW9WABY9RXB@mailjet.com> From: nee Message-ID: <7d9f2380-156d-986b-e38c-2aa5d6f45341@cock.li> Date: Wed, 19 Apr 2017 00:09:41 +0200 MIME-Version: 1.0 In-Reply-To: <633024d3.AEEAJmX48o8AAAAAAAAAAAO1C4gAAAACwQwAAAAAAAW9WABY9RXB@mailjet.com> Content-Type: multipart/mixed; boundary="------------4DA0616580476CFC1F33417E" List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: 26500@debbugs.gnu.org This is a multi-part message in MIME format. --------------4DA0616580476CFC1F33417E Content-Type: text/plain; charset=iso-8859-15 Content-Transfer-Encoding: 8bit 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/ --------------4DA0616580476CFC1F33417E Content-Type: text/x-patch; name="0001-gnu-Add-mcomix.patch" Content-Transfer-Encoding: quoted-printable Content-Disposition: attachment; filename="0001-gnu-Add-mcomix.patch" =46rom 6ecc38ecc8ce93d21cb4fa854b6332230c4ea4c9 Mon Sep 17 00:00:00 2001 From: nee 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 =C2=A9 2017 Alex Griffin ;;; Copyright =C2=A9 2017 ng0 ;;; Copyright =C2=A9 2017 Mathieu Othacehe +;;; Copyright =C2=A9 2017 nee ;;; ;;; 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)) =20 (define-public feh @@ -360,3 +363,40 @@ imaging. It supports several HDR and LDR image form= ats, 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 =3D -1") + (string-append "_7z_executable =3D 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+))) --=20 2.12.2 --------------4DA0616580476CFC1F33417E--