unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#51686] [PATCH 1/2] gnu: Add emacs-plz.
@ 2021-11-08 11:11 phodina via Guix-patches via
  2021-11-08 11:21 ` [bug#51686] [PATCH 2/2] gnu: Add emacs-ement phodina via Guix-patches via
                   ` (4 more replies)
  0 siblings, 5 replies; 10+ messages in thread
From: phodina via Guix-patches via @ 2021-11-08 11:11 UTC (permalink / raw)
  To: 51686

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




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

* [bug#51686] [PATCH 2/2] gnu: Add emacs-ement.
  2021-11-08 11:11 [bug#51686] [PATCH 1/2] gnu: Add emacs-plz phodina via Guix-patches via
@ 2021-11-08 11:21 ` phodina via Guix-patches via
  2021-11-08 11:30   ` Nicolas Goaziou
  2021-11-08 11:27 ` [bug#51686] [PATCH 1/2] gnu: Add emacs-plz Nicolas Goaziou
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 10+ messages in thread
From: phodina via Guix-patches via @ 2021-11-08 11:21 UTC (permalink / raw)
  To: 51686

* 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




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

* [bug#51686] [PATCH 1/2] gnu: Add emacs-plz.
  2021-11-08 11:11 [bug#51686] [PATCH 1/2] gnu: Add emacs-plz phodina via Guix-patches via
  2021-11-08 11:21 ` [bug#51686] [PATCH 2/2] gnu: Add emacs-ement phodina via Guix-patches via
@ 2021-11-08 11:27 ` Nicolas Goaziou
  2021-11-08 11:31 ` Nicolas Goaziou
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 10+ messages in thread
From: Nicolas Goaziou @ 2021-11-08 11:27 UTC (permalink / raw)
  To: 51686; +Cc: phodina

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




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

* [bug#51686] [PATCH 2/2] gnu: Add emacs-ement.
  2021-11-08 11:21 ` [bug#51686] [PATCH 2/2] gnu: Add emacs-ement phodina via Guix-patches via
@ 2021-11-08 11:30   ` Nicolas Goaziou
  0 siblings, 0 replies; 10+ messages in thread
From: Nicolas Goaziou @ 2021-11-08 11:30 UTC (permalink / raw)
  To: 51686; +Cc: phodina

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




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

* [bug#51686] [PATCH 1/2] gnu: Add emacs-plz.
  2021-11-08 11:11 [bug#51686] [PATCH 1/2] gnu: Add emacs-plz phodina via Guix-patches via
  2021-11-08 11:21 ` [bug#51686] [PATCH 2/2] gnu: Add emacs-ement phodina via Guix-patches via
  2021-11-08 11:27 ` [bug#51686] [PATCH 1/2] gnu: Add emacs-plz Nicolas Goaziou
@ 2021-11-08 11:31 ` Nicolas Goaziou
  2021-11-10 13:21 ` [bug#51686] [PATCH v2 2/2] gnu: Add emacs-ement phodina via Guix-patches via
  2021-11-10 13:23 ` [bug#51686] Feedback phodina via Guix-patches via
  4 siblings, 0 replies; 10+ messages in thread
From: Nicolas Goaziou @ 2021-11-08 11:31 UTC (permalink / raw)
  To: 51686; +Cc: phodina

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




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

* [bug#51686] [PATCH v2 2/2] gnu: Add emacs-ement.
  2021-11-08 11:11 [bug#51686] [PATCH 1/2] gnu: Add emacs-plz phodina via Guix-patches via
                   ` (2 preceding siblings ...)
  2021-11-08 11:31 ` Nicolas Goaziou
@ 2021-11-10 13:21 ` phodina via Guix-patches via
  2021-11-10 14:00   ` Nicolas Goaziou
                     ` (2 more replies)
  2021-11-10 13:23 ` [bug#51686] Feedback phodina via Guix-patches via
  4 siblings, 3 replies; 10+ messages in thread
From: phodina via Guix-patches via @ 2021-11-10 13:21 UTC (permalink / raw)
  To: Nicolas Goaziou; +Cc: 51686

* 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




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

* [bug#51686] Feedback
  2021-11-08 11:11 [bug#51686] [PATCH 1/2] gnu: Add emacs-plz phodina via Guix-patches via
                   ` (3 preceding siblings ...)
  2021-11-10 13:21 ` [bug#51686] [PATCH v2 2/2] gnu: Add emacs-ement phodina via Guix-patches via
@ 2021-11-10 13:23 ` phodina via Guix-patches via
  4 siblings, 0 replies; 10+ messages in thread
From: phodina via Guix-patches via @ 2021-11-10 13:23 UTC (permalink / raw)
  To: Nicolas Goaziou; +Cc: 51686

Hi Nicolas,

thanks for you're feedback :-)

--
Kind regards
Petr




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

* [bug#51686] [PATCH v2 2/2] gnu: Add emacs-ement.
  2021-11-10 13:21 ` [bug#51686] [PATCH v2 2/2] gnu: Add emacs-ement phodina via Guix-patches via
@ 2021-11-10 14:00   ` Nicolas Goaziou
  2021-12-10 14:03   ` bug#51751: " Ludovic Courtès
  2021-12-10 14:06   ` [bug#51751] " Ludovic Courtès
  2 siblings, 0 replies; 10+ messages in thread
From: Nicolas Goaziou @ 2021-11-10 14:00 UTC (permalink / raw)
  To: 51686; +Cc: phodina, 51686-done

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




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

* bug#51751: [PATCH v2 2/2] gnu: Add emacs-ement.
  2021-11-10 13:21 ` [bug#51686] [PATCH v2 2/2] gnu: Add emacs-ement phodina via Guix-patches via
  2021-11-10 14:00   ` Nicolas Goaziou
@ 2021-12-10 14:03   ` Ludovic Courtès
  2021-12-10 14:06   ` [bug#51751] " Ludovic Courtès
  2 siblings, 0 replies; 10+ messages in thread
From: Ludovic Courtès @ 2021-12-10 14:03 UTC (permalink / raw)
  To: phodina; +Cc: 51751-done, Nicolas Goaziou, 51686-done

Hi,

phodina <phodina@protonmail.com> skribis:

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

Applied, thanks!

Ludo’.




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

* [bug#51751] [PATCH v2 2/2] gnu: Add emacs-ement.
  2021-11-10 13:21 ` [bug#51686] [PATCH v2 2/2] gnu: Add emacs-ement phodina via Guix-patches via
  2021-11-10 14:00   ` Nicolas Goaziou
  2021-12-10 14:03   ` bug#51751: " Ludovic Courtès
@ 2021-12-10 14:06   ` Ludovic Courtès
  2 siblings, 0 replies; 10+ messages in thread
From: Ludovic Courtès @ 2021-12-10 14:06 UTC (permalink / raw)
  To: phodina; +Cc: 51751, 51686, Nicolas Goaziou

phodina <phodina@protonmail.com> skribis:

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

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




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

end of thread, other threads:[~2021-12-10 14:08 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-11-08 11:11 [bug#51686] [PATCH 1/2] gnu: Add emacs-plz phodina via Guix-patches via
2021-11-08 11:21 ` [bug#51686] [PATCH 2/2] gnu: Add emacs-ement phodina via Guix-patches via
2021-11-08 11:30   ` Nicolas Goaziou
2021-11-08 11:27 ` [bug#51686] [PATCH 1/2] gnu: Add emacs-plz Nicolas Goaziou
2021-11-08 11:31 ` Nicolas Goaziou
2021-11-10 13:21 ` [bug#51686] [PATCH v2 2/2] gnu: Add emacs-ement phodina via Guix-patches via
2021-11-10 14:00   ` Nicolas Goaziou
2021-12-10 14:03   ` bug#51751: " Ludovic Courtès
2021-12-10 14:06   ` [bug#51751] " Ludovic Courtès
2021-11-10 13:23 ` [bug#51686] Feedback phodina via Guix-patches via

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).