[bug#51686] [PATCH 1/2] gnu: Add emacs-plz.

[bug#51686] [PATCH 1/2] gnu: Add emacs-plz.

[phodina via Guix-patches via]

Hi,

here's a set of 2 patches to add Matrix client support to Emacs.

However, there is one warning about images:

Warning (ement): This Emacs was not built with ImageMagick support, nor does it support Cairo/XRender scaling, so images can't be displayed in Ement

Emacs inputs don't list imagemagick. However, it's deleted from a list in emacs-no-x.

So should we patch Emacs as the magickimage is missing by mistake or is there a reason?

Petr

* gnu/packages/emacs-xyz.scm (emacs-plz): Add variable.

---8<--------cut here------------------<8---
diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 543ce6303a..c7e5c05030 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -15033,6 +15033,31 @@ (define-public emacs-emamux
 multiplexer.")
     (license license:gpl3+)))

+(define-public emacs-plz
+  (let ((commit "7e456638a651bab3a814e3ea81742dd917509cbb")
+        (revision "1"))
+    (package
+      (name "emacs-plz")
+      (version (git-version "0.0" revision commit))
+      (source
+       (origin
+         (method git-fetch)
+         (uri (git-reference
+               (url "https://github.com/alphapapa/plz.el")
+               (commit commit)))
+         (file-name (git-file-name name version))
+         (sha256
+          (base32
+           "05kgxrps1s20im5hhq799nrs3615bvssm4r0ysgmwm203mmzsjgj"))))
+      (build-system emacs-build-system)
+      (inputs `(("curl" ,curl)))
+      (home-page "https://github.com/alphapapa/plz.el")
+      (synopsis "HTTP library for Emacs")
+      (description "This package provides HTTP library for Emacs.  It uses curl
+as a backend, which avoids some of the issues with using Emacs’s built-in
+url.")
+      (license license:gpl3+))))
+
 (define-public emacs-rpm-spec-mode
   (package
     (name "emacs-rpm-spec-mode")
--
2.33.1

[bug#51686] [PATCH 2/2] gnu: Add emacs-ement.

[phodina via Guix-patches via]

* gnu/packages/emacs-xyz.scm (emacs-ement): Add variable.

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index c7e5c05030..c772e86748 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -15058,6 +15058,31 @@ (define-public emacs-plz
 url.")
       (license license:gpl3+))))

+(define-public emacs-ement
+  (let ((commit "c951737dc855604aba389166bb0e7366afadc533")
+        (revision "1"))
+    (package
+      (name "emacs-ement")
+      (version (git-version "0.0" revision commit))
+      (source
+       (origin
+         (method git-fetch)
+         (uri (git-reference
+               (url "https://github.com/alphapapa/ement.el")
+               (commit commit)))
+         (file-name (git-file-name name version))
+         (sha256
+          (base32
+           "00iwwz4hzg4g59wrb5df6snqz3ppvrsadhfp61w1pa8gvg2z9bvy"))))
+      (build-system emacs-build-system)
+      (inputs `(("emacs-plz" ,emacs-plz)
+                ("emacs-ts" ,emacs-ts)))
+      (home-page "https://github.com/alphapapa/ement.el")
+      (synopsis "Matrix client for Emacs")
+      (description "This package provides simple, fast, featureful and
+reliable Matrix client for Emacs.")
+      (license license:gpl3+))))
+
 (define-public emacs-rpm-spec-mode
   (package
     (name "emacs-rpm-spec-mode")
--
2.33.1

[bug#51686] [PATCH 2/2] gnu: Add emacs-ement.

[Nicolas Goaziou]

Hello,

phodina via Guix-patches via <guix-patches@gnu.org> writes:

> +    (package
> +      (name "emacs-ement")
> +      (version (git-version "0.0" revision commit))

Version keyword from ement.el mentions "0.1-pre" as the version. It
should be a better fit than "0.0".

> +      (source
> +       (origin
> +         (method git-fetch)
> +         (uri (git-reference
> +               (url "https://github.com/alphapapa/ement.el")
> +               (commit commit)))
> +         (file-name (git-file-name name version))
> +         (sha256
> +          (base32
> +           "00iwwz4hzg4g59wrb5df6snqz3ppvrsadhfp61w1pa8gvg2z9bvy"))))
> +      (build-system emacs-build-system)
> +      (inputs `(("emacs-plz" ,emacs-plz)
> +                ("emacs-ts" ,emacs-ts)))
> +      (home-page "https://github.com/alphapapa/ement.el")
> +      (synopsis "Matrix client for Emacs")
> +      (description "This package provides simple, fast, featureful and
> +reliable Matrix client for Emacs.")

simple + fast + featureful + reliable sounds like buzzwords to me.
I suggest to use:

  Ement.el is a Matrix client for Emacs.

Regards,
-- 
Nicolas Goaziou

[bug#51686] [PATCH 1/2] gnu: Add emacs-plz.

[Nicolas Goaziou]

Hello,

phodina via Guix-patches via <guix-patches@gnu.org> writes:

> here's a set of 2 patches to add Matrix client support to Emacs.

Thank you.

> However, there is one warning about images:
>
> Warning (ement): This Emacs was not built with ImageMagick support, nor does it support Cairo/XRender scaling, so images can't be displayed in Ement
>
> Emacs inputs don't list imagemagick. However, it's deleted from a list in emacs-no-x.
>
> So should we patch Emacs as the magickimage is missing by mistake or
> is there a reason?

You need to provide a different emacs package to build it, using keyword
#:emacs.  See, e.g., emacs-auctex definiton.

Regards,
-- 
Nicolas Goaziou

[bug#51686] [PATCH 1/2] gnu: Add emacs-plz.

[Nicolas Goaziou]

Hello,

phodina via Guix-patches via <guix-patches@gnu.org> writes:

> +    (package
> +      (name "emacs-plz")
> +      (version (git-version "0.0" revision commit))

This should be "0.1-pre", not "0.0"

> +      (source
> +       (origin
> +         (method git-fetch)
> +         (uri (git-reference
> +               (url "https://github.com/alphapapa/plz.el")
> +               (commit commit)))
> +         (file-name (git-file-name name version))
> +         (sha256
> +          (base32
> +           "05kgxrps1s20im5hhq799nrs3615bvssm4r0ysgmwm203mmzsjgj"))))
> +      (build-system emacs-build-system)
> +      (inputs `(("curl" ,curl)))
> +      (home-page "https://github.com/alphapapa/plz.el")
> +      (synopsis "HTTP library for Emacs")
> +      (description "This package provides HTTP library for Emacs.  It uses curl
> +as a backend, which avoids some of the issues with using Emacs’s built-in
> +url.")

Nitpick: I would add "... built-in url library."

Regards,
-- 
Nicolas Goaziou

[bug#51686] [PATCH v2 2/2] gnu: Add emacs-ement.

[phodina via Guix-patches via]

* gnu/packages/emacs-xyz.scm (emacs-ement): New variable.

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 88f0dd7f2f..97c7ce8c1d 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -15097,6 +15097,32 @@ (define-public emacs-plz
 url library.")
       (license license:gpl3+))))

+(define-public emacs-ement
+  (let ((commit "c951737dc855604aba389166bb0e7366afadc533")
+        (revision "1"))
+    (package
+      (name "emacs-ement")
+      (version (git-version "0.1-pre" revision commit))
+      (source
+       (origin
+         (method git-fetch)
+         (uri (git-reference
+               (url "https://github.com/alphapapa/ement.el")
+               (commit commit)))
+         (file-name (git-file-name name version))
+         (sha256
+          (base32
+           "00iwwz4hzg4g59wrb5df6snqz3ppvrsadhfp61w1pa8gvg2z9bvy"))))
+      (build-system emacs-build-system)
+      (arguments
+       `(#:emacs ,emacs))
+      (inputs `(("emacs-plz" ,emacs-plz)
+                ("emacs-ts" ,emacs-ts)))
+      (home-page "https://github.com/alphapapa/ement.el")
+      (synopsis "Matrix client for Emacs")
+      (description "Ement.el is a Matrix client for Emacs.")
+      (license license:gpl3+))))
+
 (define-public emacs-rpm-spec-mode
   (package
     (name "emacs-rpm-spec-mode")
--
2.33.1

[bug#51686] [PATCH v2 2/2] gnu: Add emacs-ement.

[Nicolas Goaziou]

Hello,

phodina via Guix-patches via <guix-patches@gnu.org> writes:

> * gnu/packages/emacs-xyz.scm (emacs-ement): New variable.

Applied. Thank you.

Regards,
-- 
Nicolas Goaziou

bug#51751: [PATCH v2 2/2] gnu: Add emacs-ement.

[Ludovic Courtès]

Hi,

phodina <phodina@protonmail.com> skribis:

> * gnu/packages/emacs-xyz.scm (emacs-ement): New variable.

Applied, thanks!

Ludo’.

[bug#51751] [PATCH v2 2/2] gnu: Add emacs-ement.

[Ludovic Courtès]

phodina <phodina@protonmail.com> skribis:

> * gnu/packages/emacs-xyz.scm (emacs-ement): New variable.

Actually it had already been applied as
140b486437670ce95cb24a935b58cba52a9dac31, nevermind!

[bug#51686] Feedback

[phodina via Guix-patches via]

Hi Nicolas,

thanks for you're feedback :-)

--
Kind regards
Petr