unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
From: Amin Bandali <mab@gnu.org>
To: Nicolas Goaziou <mail@nicolasgoaziou.fr>
Cc: 39320@debbugs.gnu.org
Subject: [bug#39320] [PATCH] gnu: Add emacs-mwim.
Date: Tue, 28 Jan 2020 03:20:30 -0500	[thread overview]
Message-ID: <87eevkhu0h.fsf@gnu.org> (raw)
In-Reply-To: <87pnf4m35y.fsf@nicolasgoaziou.fr>


[-- 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 --]

  reply	other threads:[~2020-01-28  8:21 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-01-27 23:50 [bug#39320] [PATCH] gnu: Add emacs-mwim Amin Bandali
2020-01-28  7:49 ` Nicolas Goaziou
2020-01-28  8:20   ` Amin Bandali [this message]
2020-01-28  9:17     ` bug#39320: " Nicolas Goaziou

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87eevkhu0h.fsf@gnu.org \
    --to=mab@gnu.org \
    --cc=39320@debbugs.gnu.org \
    --cc=mail@nicolasgoaziou.fr \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).