[bug#39320] [PATCH] gnu: Add emacs-mwim.

[bug#39320] [PATCH] gnu: Add emacs-mwim.

[Amin Bandali]

* gnu/packages/emacs-xyz.scm (emacs-mwim): New variable.
---
 gnu/packages/emacs-xyz.scm | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index c5214405d0..18b976796f 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -21125,3 +21125,27 @@ pattern guessed from thing under current cursor position.
 mercury-mode provided by Emacs as a wrapper around prolog-mode.")
       (home-page "https://github.com/ahungry/metal-mercury-mode")
       (license license:gpl3+))))
+
+(define-public emacs-mwim
+  (let ((commit "b4f3edb4c0fb8f8b71cecbf8095c2c25a8ffbf85")
+        (revision "0"))
+    (package
+      (name "emacs-mwim")
+      (version (git-version "0.4" revision commit))
+      (source
+       (origin
+         (method git-fetch)
+         (uri (git-reference
+               (url "https://github.com/alezost/mwim.el.git")
+               (commit commit)))
+         (file-name (git-file-name name version))
+         (sha256
+          (base32
+           "0l3k611gp9g2x2vfmh92wnhnda81dslpwwpb8mxmzk308man77ya"))))
+      (build-system emacs-build-system)
+      (home-page "https://github.com/alezost/mwim.el")
+      (synopsis "Move to the beginning/end of line, code or comment")
+      (description "@code{mwim} provides several commands to switch between
+various line positions, like moving to the beginning/end of code, line, or
+comment.")
+      (license license:gpl3+))))
-- 
2.25.0

[bug#39320] [PATCH] gnu: Add emacs-mwim.

[Nicolas Goaziou]

Hello,

Amin Bandali <mab@gnu.org> writes:

> +(define-public emacs-mwim
> +  (let ((commit "b4f3edb4c0fb8f8b71cecbf8095c2c25a8ffbf85")
> +        (revision "0"))
> +    (package
> +      (name "emacs-mwim")
> +      (version (git-version "0.4" revision commit))

Thank you for the patch.

Is there any particular reason to prefer this particular commit over
stable 0.4 release?

If so, it may be worth mentioning as a comment.


Regards,

-- 
Nicolas Goaziou

[bug#39320] [PATCH] gnu: Add emacs-mwim.

[Amin Bandali]

[-- Attachment #1.1: Type: text/plain, Size: 864 bytes --]

Hello,

Nicolas Goaziou <mail@nicolasgoaziou.fr> writes:

[...]
>
> Thank you for the patch.
>

Cheers, and thank you for the feedback (addressed below).

>
> Is there any particular reason to prefer this particular commit over
> stable 0.4 release?
>
> If so, it may be worth mentioning as a comment.
>

The main reason is that commit is a fix for a reported issue [0], but
sadly isn’t part of any tagged release.  Further, the repository hasn’t
seen any activity in over a year, so I figured the likelihood of the
author tagging a new release any time soon is pretty slim.

[0]: https://github.com/alezost/mwim.el/issues/12

For what it’s worth, I’ve had this package definition in my personal
Guix channel for about six months now, and no activity in the upstream
repo through this time.

How does the following v2 look?


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: 0001-gnu-Add-emacs-mwim.patch --]
[-- Type: text/x-patch, Size: 1858 bytes --]

From 3832e92daebd5fd1659f00179b5ad88bfd1aeef7 Mon Sep 17 00:00:00 2001
From: Amin Bandali <mab@gnu.org>
Date: Tue, 28 Jan 2020 03:18:08 -0500
Subject: [PATCH v2] gnu: Add emacs-mwim.

* gnu/packages/emacs-xyz.scm (emacs-mwim): New variable.
---
 gnu/packages/emacs-xyz.scm | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index ed03c2bddd..5788f57418 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -21144,3 +21144,29 @@ mercury-mode provided by Emacs as a wrapper around prolog-mode.")
 EBDB.  It is copied more or less intact from @code{company-bbdb}, originally
 by Jan Tatarik.")
     (license license:gpl3+)))
+
+(define-public emacs-mwim
+  ;; Use the latest commit not in a release version as of yet, since it
+  ;; contains a bug fix for the cases where `comment-start-skip' is nil.
+  (let ((commit "b4f3edb4c0fb8f8b71cecbf8095c2c25a8ffbf85")
+        (revision "0"))
+    (package
+      (name "emacs-mwim")
+      (version (git-version "0.4" revision commit))
+      (source
+       (origin
+         (method git-fetch)
+         (uri (git-reference
+               (url "https://github.com/alezost/mwim.el.git")
+               (commit commit)))
+         (file-name (git-file-name name version))
+         (sha256
+          (base32
+           "0l3k611gp9g2x2vfmh92wnhnda81dslpwwpb8mxmzk308man77ya"))))
+      (build-system emacs-build-system)
+      (home-page "https://github.com/alezost/mwim.el")
+      (synopsis "Move to the beginning/end of line, code or comment")
+      (description "@code{mwim} provides several commands to switch between
+various line positions, like moving to the beginning/end of code, line, or
+comment.")
+      (license license:gpl3+))))
-- 
2.25.0


[-- Attachment #1.3: Type: text/plain, Size: 93 bytes --]


>
>
> Regards,

Best,
amin

P.S. thanks for bumping my copyright year in emacs-xyz.scm. :-)

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

bug#39320: [PATCH] gnu: Add emacs-mwim.

[Nicolas Goaziou]

Amin Bandali <mab@gnu.org> writes:

> The main reason is that commit is a fix for a reported issue [0], but
> sadly isn’t part of any tagged release.  Further, the repository hasn’t
> seen any activity in over a year, so I figured the likelihood of the
> author tagging a new release any time soon is pretty slim.
>
> [0]: https://github.com/alezost/mwim.el/issues/12
>
> For what it’s worth, I’ve had this package definition in my personal
> Guix channel for about six months now, and no activity in the upstream
> repo through this time.
>
> How does the following v2 look?

Fine. I applied it as 19568f9e79ed74b7100d25d3f2ea245f857ca742. Thank
you.

Regards,