GNU bug report logs - #54558
mcomix refuses to run, missing GTK libraries

Previous Next

Package: guix;

Reported by: Natalie <batalie <at> riseup.net>

Date: Thu, 24 Mar 2022 22:01:01 UTC

Severity: normal

Done: Liliana Marie Prikler <liliana.prikler <at> gmail.com>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 54558 in the body.
You can then email your comments to 54558 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-guix <at> gnu.org:
bug#54558; Package guix. (Thu, 24 Mar 2022 22:01:01 GMT) Full text and rfc822 format available.

Acknowledgement sent to Natalie <batalie <at> riseup.net>:
New bug report received and forwarded. Copy sent to bug-guix <at> gnu.org. (Thu, 24 Mar 2022 22:01:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Natalie <batalie <at> riseup.net>
To: bug-guix <at> gnu.org
Subject: mcomix refuses to run, missing GTK libraries
Date: Thu, 24 Mar 2022 13:08:56 -0700
Hi there,

I've just tried to install mcomix on Guix System, but have had no luck
running it.

Upon installing and trying to run it in the shell, it fails with:

> [MainThread] ERROR: You do not have the required versions of GTK+ 3.0 installed.

Does GTK need to be added to its inputs?




Information forwarded to bug-guix <at> gnu.org:
bug#54558; Package guix. (Fri, 25 Mar 2022 22:49:02 GMT) Full text and rfc822 format available.

Message #8 received at 54558 <at> debbugs.gnu.org (full text, mbox):

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: 54558 <at> debbugs.gnu.org
Cc: batalie <at> riseup.net
Subject: [PATCH 2/2] gnu: mcomix: Wrap missing paths.
Date: Fri, 25 Mar 2022 23:45:51 +0100
* gnu/packages/image-viewers.scm (mcomix)[inputs]: Add gtk+.
[arguments]<#:imported-modules,#:modules>: New arguments.
<#:phases>: Add ‘glib-or-gtk-wrap’ and ‘gi-wrap’.
---
 gnu/packages/image-viewers.scm | 19 +++++++++++++++++--
 1 file changed, 17 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/image-viewers.scm b/gnu/packages/image-viewers.scm
index 1c325cd8a7..134ffc563a 100644
--- a/gnu/packages/image-viewers.scm
+++ b/gnu/packages/image-viewers.scm
@@ -675,9 +675,14 @@ (define-public mcomix
          "187ca815vxb2in1ryvfiaf1zapi0bc9jxdac3c1bky0kr6x7xyap"))))
     (build-system python-build-system)
     (inputs
-     (list p7zip python python-pillow python-pygobject python-pycairo))
+     (list p7zip python python-pillow python-pygobject python-pycairo gtk+))
     (arguments
      (list
+      #:imported-modules `(,@%python-build-system-modules
+                           (guix build glib-or-gtk-build-system))
+      #:modules '((guix build python-build-system)
+                  ((guix build glib-or-gtk-build-system) #:prefix glib-or-gtk:)
+                  (guix build utils))
       #:phases
       #~(modify-phases %standard-phases
           (add-after 'unpack 'patch-source
@@ -699,7 +704,17 @@ (define-public mcomix
                              "/lib/python"
                              #$(version-major+minor
                                 (package-version (this-package-input "python")))
-                             "/site-packages/mcomix/images")))))))
+                             "/site-packages/mcomix/images"))))
+         (add-after 'glib-or-gtk-compile-schemas 'glib-or-gtk-wrap
+           (assoc-ref glib-or-gtk:%standard-phases 'glib-or-gtk-wrap))
+         (add-after 'wrap 'gi-wrap
+           (lambda* (#:key outputs #:allow-other-keys)
+             (let ((bin (string-append (assoc-ref outputs "out") "/bin")))
+               (for-each
+                (lambda (prog)
+                  (wrap-program prog
+                    `("GI_TYPELIB_PATH" = (,(getenv "GI_TYPELIB_PATH")))))
+                (list (string-append bin "/mcomix")))))))))
     (home-page "https://sourceforge.net/p/mcomix/wiki/Home/")
     (synopsis "Image viewer for comics")
     (description "MComix is a customizable image viewer that specializes as
-- 
2.34.0





Information forwarded to bug-guix <at> gnu.org:
bug#54558; Package guix. (Fri, 25 Mar 2022 22:49:02 GMT) Full text and rfc822 format available.

Message #11 received at 54558 <at> debbugs.gnu.org (full text, mbox):

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: 54558 <at> debbugs.gnu.org
Cc: batalie <at> riseup.net
Subject: [PATCH 1/2] gnu: mcomix: Update to 2.0.1.
Date: Fri, 25 Mar 2022 21:37:01 +0100
* gnu/packages/image-viewers.scm (mcomix): Update to 2.0.1.
[source]: Use sourceforge once again.
[inputs]: Add ‘python’.
[arguments]: Convert to a list of G-Expressions.
<#:tests?>: Drop.
<#:phases>: Rename ‘configure’ to ‘patch-sources’.  Also fix an assertion
error in "mcomix/image_tools.py".
No longer replace ‘build’ or ‘install’.  Add ‘install-data’ phase after
‘install’.  Use (this-package-input) to refer to the python input.
---
 gnu/packages/image-viewers.scm | 123 +++++++++++----------------------
 1 file changed, 42 insertions(+), 81 deletions(-)

diff --git a/gnu/packages/image-viewers.scm b/gnu/packages/image-viewers.scm
index 6ab6cf3307..1c325cd8a7 100644
--- a/gnu/packages/image-viewers.scm
+++ b/gnu/packages/image-viewers.scm
@@ -662,91 +662,52 @@ (define-public luminance-hdr
 
 ;; CBR and RAR are currently unsupported, due to non-free dependencies.
 (define-public mcomix
-  ;; Official mcomix hasn't been updated since 2016, it's broken with
-  ;; python-pillow 6+ and only supports Python 2.  We use fork instead.
-  (let ((commit "fea55a7a9369569eefed72209eed830409c4af98"))
-    (package
-      (name "mcomix")
-      (version (git-version "1.2.1" "1" commit))
-      (source
-       (origin
-         (method git-fetch)
-         (uri (git-reference
-               (url "https://github.com/multiSnow/mcomix3")
-               (commit commit)))
-         (file-name (git-file-name name version))
-         (sha256
-          (base32
-           "05zl0dkjwbdcm2zlk4nz9w33amlqj8pbf32a8ymshc2356fqhhi5"))))
-      (build-system python-build-system)
-      (inputs
-       (list p7zip python-pillow python-pygobject python-pycairo))
-      (arguments
-       `(#:tests? #f                    ; FIXME: How do we run tests?
-         #:phases
-         (modify-phases %standard-phases
-           (add-after 'unpack 'configure
-             (lambda* (#:key inputs #:allow-other-keys)
-               (let ((p7zip (assoc-ref inputs "p7zip")))
-                 ;; insert absolute path to 7z executable
-                 (substitute* "mcomix/mcomix/archive/sevenzip_external.py"
-                   (("_7z_executable = -1")
-                    (string-append "_7z_executable = u'" p7zip "/bin/7z'"))))
-               #t))
-           (replace 'build
-             (lambda* (#:key outputs #:allow-other-keys)
-               (let* ((out (assoc-ref outputs "out"))
-                      (pyver ,(version-major+minor (package-version python)))
-                      (lib (string-append out "/lib/python" pyver)))
-                 (invoke (which "python") "installer.py" "--srcdir=mcomix"
-                         (string-append "--target=" lib))
-                 (rename-file (string-append lib "/mcomix")
-                              (string-append lib "/site-packages"))
-                 #t)))
-           (replace 'install
-             (lambda* (#:key outputs #:allow-other-keys)
-               (let* ((out (assoc-ref outputs "out"))
-                      (share (string-append out "/share"))
-                      (bin (string-append out "/bin"))
-                      (pyver ,(version-major+minor (package-version python)))
-                      (lib (string-append out "/lib/python" pyver "/site-packages")))
-                 (mkdir-p bin)
-                 (rename-file (string-append lib "/mcomixstarter.py")
-                              (string-append bin "/mcomix"))
-                 (rename-file (string-append lib "/comicthumb.py")
-                              (string-append bin "/comicthumb"))
-                 (install-file "mime/mcomix.desktop"
-                               (string-append share "/applications"))
-                 (install-file "mime/mcomix.appdata.xml"
-                               (string-append share "/metainfo"))
-                 (install-file "mime/mcomix.xml"
-                               (string-append share "/mime/packages"))
-                 (install-file "mime/comicthumb.thumbnailer"
-                               (string-append share "/thumbnailers"))
-                 (install-file "man/mcomix.1" (string-append share "/man/man1"))
-                 (install-file "man/comicthumb.1" (string-append share "/man/man1"))
-                 (for-each
-                  (lambda (size)
-                    (install-file
-                     (format #f "mcomix/mcomix/images/~sx~s/mcomix.png" size size)
-                     (format #f "~a/icons/hicolor/~sx~s/apps/" share size size))
-                    (for-each
-                     (lambda (ext)
-                       (install-file
-                        (format #f "mime/icons/~sx~s/application-x-~a.png" size size ext)
-                        (format #f "~a/icons/hicolor/~sx~s/mimetypes/"
-                                share size size)))
-                     '("cb7" "cbr" "cbt" "cbz")))
-                  '(16 22 24 32 48))
-                 #t))))))
-      (home-page "https://sourceforge.net/p/mcomix/wiki/Home/")
-      (synopsis "Image viewer for comics")
-      (description "MComix is a customizable image viewer that specializes as
+  (package
+    (name "mcomix")
+    (version "2.0.1")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "mirror://sourceforge/mcomix/MComix-" version "/"
+                           "mcomix-" version ".tar.gz"))
+       (sha256
+        (base32
+         "187ca815vxb2in1ryvfiaf1zapi0bc9jxdac3c1bky0kr6x7xyap"))))
+    (build-system python-build-system)
+    (inputs
+     (list p7zip python python-pillow python-pygobject python-pycairo))
+    (arguments
+     (list
+      #:phases
+      #~(modify-phases %standard-phases
+          (add-after 'unpack 'patch-source
+            (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'")))
+
+                (substitute* "mcomix/image_tools.py"
+                  (("assert name not in supported_formats_gdk")
+                   "if name in supported_formats_gdk: continue")))))
+         (add-after 'install 'install-data
+           (lambda* (#:key outputs #:allow-other-keys)
+             (copy-recursively
+              "mcomix/images"
+              (string-append (assoc-ref outputs "out")
+                             "/lib/python"
+                             #$(version-major+minor
+                                (package-version (this-package-input "python")))
+                             "/site-packages/mcomix/images")))))))
+    (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.
 
 For PDF support, install the @emph{mupdf} package.")
-      (license license:gpl2+))))
+    (license license:gpl2+)))
 
 (define-public qview
   (package
-- 
2.34.0





Information forwarded to bug-guix <at> gnu.org:
bug#54558; Package guix. (Sat, 26 Mar 2022 02:58:02 GMT) Full text and rfc822 format available.

Message #14 received at 54558 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
Cc: 54558 <at> debbugs.gnu.org, batalie <at> riseup.net
Subject: Re: bug#54558: mcomix refuses to run, missing GTK libraries
Date: Fri, 25 Mar 2022 22:57:26 -0400
Hi Liliana,

Liliana Marie Prikler <liliana.prikler <at> gmail.com> writes:

> * gnu/packages/image-viewers.scm (mcomix): Update to 2.0.1.
> [source]: Use sourceforge once again.
> [inputs]: Add ‘python’.
> [arguments]: Convert to a list of G-Expressions.
> <#:tests?>: Drop.
> <#:phases>: Rename ‘configure’ to ‘patch-sources’.  Also fix an assertion
> error in "mcomix/image_tools.py".
> No longer replace ‘build’ or ‘install’.  Add ‘install-data’ phase after
> ‘install’.  Use (this-package-input) to refer to the python input.
> ---
>  gnu/packages/image-viewers.scm | 123 +++++++++++----------------------
>  1 file changed, 42 insertions(+), 81 deletions(-)
>
> diff --git a/gnu/packages/image-viewers.scm b/gnu/packages/image-viewers.scm
> index 6ab6cf3307..1c325cd8a7 100644
> --- a/gnu/packages/image-viewers.scm
> +++ b/gnu/packages/image-viewers.scm
> @@ -662,91 +662,52 @@ (define-public luminance-hdr
>  
>  ;; CBR and RAR are currently unsupported, due to non-free dependencies.
>  (define-public mcomix
> -  ;; Official mcomix hasn't been updated since 2016, it's broken with
> -  ;; python-pillow 6+ and only supports Python 2.  We use fork instead.
> -  (let ((commit "fea55a7a9369569eefed72209eed830409c4af98"))
> -    (package
> -      (name "mcomix")
> -      (version (git-version "1.2.1" "1" commit))
> -      (source
> -       (origin
> -         (method git-fetch)
> -         (uri (git-reference
> -               (url "https://github.com/multiSnow/mcomix3")
> -               (commit commit)))
> -         (file-name (git-file-name name version))
> -         (sha256
> -          (base32
> -           "05zl0dkjwbdcm2zlk4nz9w33amlqj8pbf32a8ymshc2356fqhhi5"))))
> -      (build-system python-build-system)
> -      (inputs
> -       (list p7zip python-pillow python-pygobject python-pycairo))
> -      (arguments
> -       `(#:tests? #f                    ; FIXME: How do we run tests?
> -         #:phases
> -         (modify-phases %standard-phases
> -           (add-after 'unpack 'configure
> -             (lambda* (#:key inputs #:allow-other-keys)
> -               (let ((p7zip (assoc-ref inputs "p7zip")))
> -                 ;; insert absolute path to 7z executable
> -                 (substitute* "mcomix/mcomix/archive/sevenzip_external.py"
> -                   (("_7z_executable = -1")
> -                    (string-append "_7z_executable = u'" p7zip "/bin/7z'"))))
> -               #t))
> -           (replace 'build
> -             (lambda* (#:key outputs #:allow-other-keys)
> -               (let* ((out (assoc-ref outputs "out"))
> -                      (pyver ,(version-major+minor (package-version python)))
> -                      (lib (string-append out "/lib/python" pyver)))
> -                 (invoke (which "python") "installer.py" "--srcdir=mcomix"
> -                         (string-append "--target=" lib))
> -                 (rename-file (string-append lib "/mcomix")
> -                              (string-append lib "/site-packages"))
> -                 #t)))
> -           (replace 'install
> -             (lambda* (#:key outputs #:allow-other-keys)
> -               (let* ((out (assoc-ref outputs "out"))
> -                      (share (string-append out "/share"))
> -                      (bin (string-append out "/bin"))
> -                      (pyver ,(version-major+minor (package-version python)))
> -                      (lib (string-append out "/lib/python" pyver "/site-packages")))
> -                 (mkdir-p bin)
> -                 (rename-file (string-append lib "/mcomixstarter.py")
> -                              (string-append bin "/mcomix"))
> -                 (rename-file (string-append lib "/comicthumb.py")
> -                              (string-append bin "/comicthumb"))
> -                 (install-file "mime/mcomix.desktop"
> -                               (string-append share "/applications"))
> -                 (install-file "mime/mcomix.appdata.xml"
> -                               (string-append share "/metainfo"))
> -                 (install-file "mime/mcomix.xml"
> -                               (string-append share "/mime/packages"))
> -                 (install-file "mime/comicthumb.thumbnailer"
> -                               (string-append share "/thumbnailers"))
> -                 (install-file "man/mcomix.1" (string-append share "/man/man1"))
> -                 (install-file "man/comicthumb.1" (string-append share "/man/man1"))
> -                 (for-each
> -                  (lambda (size)
> -                    (install-file
> -                     (format #f "mcomix/mcomix/images/~sx~s/mcomix.png" size size)
> -                     (format #f "~a/icons/hicolor/~sx~s/apps/" share size size))
> -                    (for-each
> -                     (lambda (ext)
> -                       (install-file
> -                        (format #f "mime/icons/~sx~s/application-x-~a.png" size size ext)
> -                        (format #f "~a/icons/hicolor/~sx~s/mimetypes/"
> -                                share size size)))
> -                     '("cb7" "cbr" "cbt" "cbz")))
> -                  '(16 22 24 32 48))
> -                 #t))))))
> -      (home-page "https://sourceforge.net/p/mcomix/wiki/Home/")
> -      (synopsis "Image viewer for comics")
> -      (description "MComix is a customizable image viewer that specializes as
> +  (package
> +    (name "mcomix")
> +    (version "2.0.1")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append "mirror://sourceforge/mcomix/MComix-" version "/"
> +                           "mcomix-" version ".tar.gz"))
> +       (sha256
> +        (base32
> +         "187ca815vxb2in1ryvfiaf1zapi0bc9jxdac3c1bky0kr6x7xyap"))))
> +    (build-system python-build-system)
> +    (inputs
> +     (list p7zip python python-pillow python-pygobject python-pycairo))
> +    (arguments
> +     (list
> +      #:phases
> +      #~(modify-phases %standard-phases
> +          (add-after 'unpack 'patch-source
> +            (lambda* (#:key inputs #:allow-other-keys)
> +              (let ((p7zip (assoc-ref inputs "p7zip")))
> +                ;; insert absolute path to 7z executable

I know it's in the original, but since while we're revamping the
package, let's make this comment a proper complete sentence.

> +                (substitute* "mcomix/archive/sevenzip_external.py"
> +                  (("_7z_executable = -1")
> +                   (string-append "_7z_executable = u'" p7zip "/bin/7z'")))

I'd use something like:

(format #f "_7z_executable = ~s"
 (search-input-file inputs "bin/7z"))

For the replacement.  That unicode string (u"something") is
obsolete/unnecessary (all strings are unicode in Python 3).


> +                (substitute* "mcomix/image_tools.py"
> +                  (("assert name not in supported_formats_gdk")
> +                   "if name in supported_formats_gdk: continue")))))
> +         (add-after 'install 'install-data
> +           (lambda* (#:key outputs #:allow-other-keys)
> +             (copy-recursively
> +              "mcomix/images"
> +              (string-append (assoc-ref outputs "out")
> +                             "/lib/python"
> +                             #$(version-major+minor
> +                                (package-version (this-package-input "python")))
> +                             "/site-packages/mcomix/images")))))))
> +    (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.
>  
>  For PDF support, install the @emph{mupdf} package.")
> -      (license license:gpl2+))))
> +    (license license:gpl2+)))

I haven't tried it, but LGTM with my above comments addressed.

Thank you!

Maxim




Reply sent to Liliana Marie Prikler <liliana.prikler <at> gmail.com>:
You have taken responsibility. (Sat, 26 Mar 2022 08:02:02 GMT) Full text and rfc822 format available.

Notification sent to Natalie <batalie <at> riseup.net>:
bug acknowledged by developer. (Sat, 26 Mar 2022 08:02:02 GMT) Full text and rfc822 format available.

Message #19 received at 54558-done <at> debbugs.gnu.org (full text, mbox):

From: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
To: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Cc: 54558-done <at> debbugs.gnu.org, batalie <at> riseup.net
Subject: Re: bug#54558: mcomix refuses to run, missing GTK libraries
Date: Sat, 26 Mar 2022 09:01:47 +0100
Hi Maxim,

Am Freitag, dem 25.03.2022 um 22:57 -0400 schrieb Maxim Cournoyer:
> Hi Liliana,
> 
> Liliana Marie Prikler <liliana.prikler <at> gmail.com> writes:
> 
> > [...]
> > +              (let ((p7zip (assoc-ref inputs "p7zip")))
> > +                ;; insert absolute path to 7z executable
> 
> I know it's in the original, but since while we're revamping the
> package, let's make this comment a proper complete sentence.
Thanks, did that.

> > +                (substitute* "mcomix/archive/sevenzip_external.py"
> > +                  (("_7z_executable = -1")
> > +                   (string-append "_7z_executable = u'" p7zip
> > "/bin/7z'")))
> 
> I'd use something like:
> 
> (format #f "_7z_executable = ~s"
>  (search-input-file inputs "bin/7z"))
> 
> For the replacement.  That unicode string (u"something") is
> obsolete/unnecessary (all strings are unicode in Python 3).
Thanks, did that.

> I haven't tried it, but LGTM with my above comments addressed.
I tried it in a pure shell and apart from the usual missing icons, it
launched fine.  Closing.

Cheers




Information forwarded to bug-guix <at> gnu.org:
bug#54558; Package guix. (Sun, 27 Mar 2022 02:27:02 GMT) Full text and rfc822 format available.

Message #22 received at 54558 <at> debbugs.gnu.org (full text, mbox):

From: "Feng Shu" <tumashu <at> 163.com>
To: 54558 <at> debbugs.gnu.org
Cc: Feng Shu <tumashu <at> 163.com>
Subject: Mcomix 2.0.1 do not install locale data.
Date: Sun, 27 Mar 2022 10:26:32 +0800
I think Mcomix 2.0.1 do not install locale date.

-- 





Information forwarded to bug-guix <at> gnu.org:
bug#54558; Package guix. (Sun, 27 Mar 2022 09:40:01 GMT) Full text and rfc822 format available.

Message #25 received at 54558 <at> debbugs.gnu.org (full text, mbox):

From: "Feng Shu" <tumashu <at> 163.com>
To: 54558 <at> debbugs.gnu.org
Cc: Feng Shu <tumashu <at> 163.com>
Subject: Re: Mcomix 2.0.1 do not install locale data.
Date: Sun, 27 Mar 2022 17:39:07 +0800
[Message part 1 (text/plain, inline)]
"Feng Shu" <tumashu <at> 163.com> writes:

> I think Mcomix 2.0.1 do not install locale date.

I try to code a patch:

[0001-gnu-mcomix-Install-messages-directory.patch (text/x-patch, inline)]
From bc1627e85cde9569b96057daa863184566863858 Mon Sep 17 00:00:00 2001
From: Feng Shu <tumashu <at> 163.com>
Date: Sun, 27 Mar 2022 17:35:13 +0800
Subject: [PATCH] gnu: mcomix: Install messages directory.

* gnu/packages/image-viewers.scm (mcomix)
[arguments]: Install messages directory.
---
 gnu/packages/image-viewers.scm | 51 +++++++++++++++++-----------------
 1 file changed, 26 insertions(+), 25 deletions(-)

diff --git a/gnu/packages/image-viewers.scm b/gnu/packages/image-viewers.scm
index 05a8151c29..7410605077 100644
--- a/gnu/packages/image-viewers.scm
+++ b/gnu/packages/image-viewers.scm
@@ -695,31 +695,32 @@ (define-public mcomix
               (substitute* "mcomix/image_tools.py"
                 (("assert name not in supported_formats_gdk")
                  "if name in supported_formats_gdk: continue"))))
-         (add-after 'install 'install-data
-           (lambda* (#:key outputs #:allow-other-keys)
-             (copy-recursively
-              "mcomix/images"
-              (string-append (assoc-ref outputs "out")
-                             "/lib/python"
-                             #$(version-major+minor
-                                (package-version (this-package-input "python")))
-                             "/site-packages/mcomix/images"))))
-         (add-after 'install 'install-messages
-           (lambda* (#:key outputs #:allow-other-keys)
-             (copy-recursively
-              "mcomix/messages"
-              (string-append (assoc-ref outputs "out")
-                             "/share/locale"))))
-         (add-after 'glib-or-gtk-compile-schemas 'glib-or-gtk-wrap
-           (assoc-ref glib-or-gtk:%standard-phases 'glib-or-gtk-wrap))
-         (add-after 'wrap 'gi-wrap
-           (lambda* (#:key outputs #:allow-other-keys)
-             (let ((bin (string-append (assoc-ref outputs "out") "/bin")))
-               (for-each
-                (lambda (prog)
-                  (wrap-program prog
-                    `("GI_TYPELIB_PATH" = (,(getenv "GI_TYPELIB_PATH")))))
-                (list (string-append bin "/mcomix")))))))))
+          (add-after 'install 'install-data
+            (lambda* (#:key outputs #:allow-other-keys)
+              (copy-recursively
+               "mcomix/images"
+               (string-append (assoc-ref outputs "out")
+                              "/lib/python"
+                              #$(version-major+minor
+                                 (package-version (this-package-input "python")))
+                              "/site-packages/mcomix/images"))
+              (copy-recursively
+               "mcomix/messages"
+               (string-append (assoc-ref outputs "out")
+                              "/lib/python"
+                              #$(version-major+minor
+                                 (package-version (this-package-input "python")))
+                              "/site-packages/mcomix/messages"))))
+          (add-after 'glib-or-gtk-compile-schemas 'glib-or-gtk-wrap
+            (assoc-ref glib-or-gtk:%standard-phases 'glib-or-gtk-wrap))
+          (add-after 'wrap 'gi-wrap
+            (lambda* (#:key outputs #:allow-other-keys)
+              (let ((bin (string-append (assoc-ref outputs "out") "/bin")))
+                (for-each
+                 (lambda (prog)
+                   (wrap-program prog
+                     `("GI_TYPELIB_PATH" = (,(getenv "GI_TYPELIB_PATH")))))
+                 (list (string-append bin "/mcomix")))))))))
     (home-page "https://sourceforge.net/p/mcomix/wiki/Home/")
     (synopsis "Image viewer for comics")
     (description "MComix is a customizable image viewer that specializes as
-- 
2.34.0

[Message part 3 (text/plain, inline)]

-- 

Information forwarded to bug-guix <at> gnu.org:
bug#54558; Package guix. (Sun, 27 Mar 2022 09:53:01 GMT) Full text and rfc822 format available.

Message #28 received at 54558 <at> debbugs.gnu.org (full text, mbox):

From: "Feng Shu" <tumashu <at> 163.com>
To: 54558 <at> debbugs.gnu.org
Cc: Feng Shu <tumashu <at> 163.com>
Subject: Re: Mcomix 2.0.1 do not install locale data.
Date: Sun, 27 Mar 2022 17:52:04 +0800
[Message part 1 (text/plain, inline)]
"Feng Shu" <tumashu <at> 163.com> writes:

> "Feng Shu" <tumashu <at> 163.com> writes:
>
>> I think Mcomix 2.0.1 do not install locale date.
>
> I try to code a patch:
>
> From bc1627e85cde9569b96057daa863184566863858 Mon Sep 17 00:00:00 2001
> From: Feng Shu <tumashu <at> 163.com>
> Date: Sun, 27 Mar 2022 17:35:13 +0800
> Subject: [PATCH] gnu: mcomix: Install messages directory.
>
> * gnu/packages/image-viewers.scm (mcomix)
> [arguments]: Install messages directory.
> ---
>  gnu/packages/image-viewers.scm | 51 +++++++++++++++++-----------------
>  1 file changed, 26 insertions(+), 25 deletions(-)
>
> diff --git a/gnu/packages/image-viewers.scm b/gnu/packages/image-viewers.scm
> index 05a8151c29..7410605077 100644
> --- a/gnu/packages/image-viewers.scm
> +++ b/gnu/packages/image-viewers.scm
> @@ -695,31 +695,32 @@ (define-public mcomix
>                (substitute* "mcomix/image_tools.py"
>                  (("assert name not in supported_formats_gdk")
>                   "if name in supported_formats_gdk: continue"))))
> -         (add-after 'install 'install-data
> -           (lambda* (#:key outputs #:allow-other-keys)
> -             (copy-recursively
> -              "mcomix/images"
> -              (string-append (assoc-ref outputs "out")
> -                             "/lib/python"
> -                             #$(version-major+minor
> -                                (package-version (this-package-input "python")))
> -                             "/site-packages/mcomix/images"))))
> -         (add-after 'install 'install-messages
> -           (lambda* (#:key outputs #:allow-other-keys)
> -             (copy-recursively
> -              "mcomix/messages"
> -              (string-append (assoc-ref outputs "out")
> -                             "/share/locale"))))
> -         (add-after 'glib-or-gtk-compile-schemas 'glib-or-gtk-wrap
> -           (assoc-ref glib-or-gtk:%standard-phases 'glib-or-gtk-wrap))
> -         (add-after 'wrap 'gi-wrap
> -           (lambda* (#:key outputs #:allow-other-keys)
> -             (let ((bin (string-append (assoc-ref outputs "out") "/bin")))
> -               (for-each
> -                (lambda (prog)
> -                  (wrap-program prog
> -                    `("GI_TYPELIB_PATH" = (,(getenv "GI_TYPELIB_PATH")))))
> -                (list (string-append bin "/mcomix")))))))))
> +          (add-after 'install 'install-data
> +            (lambda* (#:key outputs #:allow-other-keys)
> +              (copy-recursively
> +               "mcomix/images"
> +               (string-append (assoc-ref outputs "out")
> +                              "/lib/python"
> +                              #$(version-major+minor
> +                                 (package-version (this-package-input "python")))
> +                              "/site-packages/mcomix/images"))
> +              (copy-recursively
> +               "mcomix/messages"
> +               (string-append (assoc-ref outputs "out")
> +                              "/lib/python"
> +                              #$(version-major+minor
> +                                 (package-version (this-package-input "python")))
> +                              "/site-packages/mcomix/messages"))))
> +          (add-after 'glib-or-gtk-compile-schemas 'glib-or-gtk-wrap
> +            (assoc-ref glib-or-gtk:%standard-phases 'glib-or-gtk-wrap))
> +          (add-after 'wrap 'gi-wrap
> +            (lambda* (#:key outputs #:allow-other-keys)
> +              (let ((bin (string-append (assoc-ref outputs "out") "/bin")))
> +                (for-each
> +                 (lambda (prog)
> +                   (wrap-program prog
> +                     `("GI_TYPELIB_PATH" = (,(getenv "GI_TYPELIB_PATH")))))
> +                 (list (string-append bin "/mcomix")))))))))
>      (home-page "https://sourceforge.net/p/mcomix/wiki/Home/")
>      (synopsis "Image viewer for comics")
>      (description "MComix is a customizable image viewer that specializes as
> -- 
> 2.34.0


Sorry, the above is wrong patch, please see below:

[0001-gnu-mcomix-Install-messages-directory.patch (text/x-patch, inline)]
From a84f1c8d1ad923f2b90fdf40a0655f8015f122a8 Mon Sep 17 00:00:00 2001
From: Feng Shu <tumashu <at> 163.com>
Date: Sun, 27 Mar 2022 17:47:52 +0800
Subject: [PATCH] gnu: mcomix: Install messages directory.

* gnu/packages/image-viewers.scm (mcomix)
[arguments]: Install messages directory.
---
 gnu/packages/image-viewers.scm | 45 ++++++++++++++++++++--------------
 1 file changed, 26 insertions(+), 19 deletions(-)

diff --git a/gnu/packages/image-viewers.scm b/gnu/packages/image-viewers.scm
index 09c448e8f5..7410605077 100644
--- a/gnu/packages/image-viewers.scm
+++ b/gnu/packages/image-viewers.scm
@@ -695,25 +695,32 @@ (define-public mcomix
               (substitute* "mcomix/image_tools.py"
                 (("assert name not in supported_formats_gdk")
                  "if name in supported_formats_gdk: continue"))))
-         (add-after 'install 'install-data
-           (lambda* (#:key outputs #:allow-other-keys)
-             (copy-recursively
-              "mcomix/images"
-              (string-append (assoc-ref outputs "out")
-                             "/lib/python"
-                             #$(version-major+minor
-                                (package-version (this-package-input "python")))
-                             "/site-packages/mcomix/images"))))
-         (add-after 'glib-or-gtk-compile-schemas 'glib-or-gtk-wrap
-           (assoc-ref glib-or-gtk:%standard-phases 'glib-or-gtk-wrap))
-         (add-after 'wrap 'gi-wrap
-           (lambda* (#:key outputs #:allow-other-keys)
-             (let ((bin (string-append (assoc-ref outputs "out") "/bin")))
-               (for-each
-                (lambda (prog)
-                  (wrap-program prog
-                    `("GI_TYPELIB_PATH" = (,(getenv "GI_TYPELIB_PATH")))))
-                (list (string-append bin "/mcomix")))))))))
+          (add-after 'install 'install-data
+            (lambda* (#:key outputs #:allow-other-keys)
+              (copy-recursively
+               "mcomix/images"
+               (string-append (assoc-ref outputs "out")
+                              "/lib/python"
+                              #$(version-major+minor
+                                 (package-version (this-package-input "python")))
+                              "/site-packages/mcomix/images"))
+              (copy-recursively
+               "mcomix/messages"
+               (string-append (assoc-ref outputs "out")
+                              "/lib/python"
+                              #$(version-major+minor
+                                 (package-version (this-package-input "python")))
+                              "/site-packages/mcomix/messages"))))
+          (add-after 'glib-or-gtk-compile-schemas 'glib-or-gtk-wrap
+            (assoc-ref glib-or-gtk:%standard-phases 'glib-or-gtk-wrap))
+          (add-after 'wrap 'gi-wrap
+            (lambda* (#:key outputs #:allow-other-keys)
+              (let ((bin (string-append (assoc-ref outputs "out") "/bin")))
+                (for-each
+                 (lambda (prog)
+                   (wrap-program prog
+                     `("GI_TYPELIB_PATH" = (,(getenv "GI_TYPELIB_PATH")))))
+                 (list (string-append bin "/mcomix")))))))))
     (home-page "https://sourceforge.net/p/mcomix/wiki/Home/")
     (synopsis "Image viewer for comics")
     (description "MComix is a customizable image viewer that specializes as
-- 
2.34.0

[Message part 3 (text/plain, inline)]


-- 

bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Sun, 24 Apr 2022 11:24:04 GMT) Full text and rfc822 format available.

This bug report was last modified 3 years and 149 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.