all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#31081] [PATCH] gnu: Add emacs-anzu-evil
@ 2018-04-06 19:20 Sohom Bhattacharjee
  2018-04-07 16:25 ` Oleg Pykhalov
  0 siblings, 1 reply; 4+ messages in thread
From: Sohom Bhattacharjee @ 2018-04-06 19:20 UTC (permalink / raw)
  To: 31081

* gnu/packages/emacs.scm (emacs-anzu-evil): New variable.
---
test

 gnu/packages/emacs.scm | 23 +++++++++++++++++++++++
 1 file changed, 23 insertions(+)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index fd0b8acd0..2463d0dab 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -7494,3 +7494,26 @@ used with SGML-like languages: XML, HTML, XHTML, XSL, etc.")
 standard Unix password manager\").")
     (license license:gpl2+)))
 
+(define-public emacs-evil-anzu
+ (package
+  (name "emacs-evil-anzu")
+  (version "0.03")
+  (source
+   (origin
+     (method url-fetch)
+     (uri (string-append "https://github.com/syohex/emacs-evil-anzu/archive/"
+                         version ".tar.gz"))
+     (file-name (string-append name "-" version ".tar.gz"))
+     (sha256
+      (base32 "032hh2946z529cizqsg8pm6cpn5qdj8lfk3qskmx6xv3g2ra56ns"))))
+  (build-system emacs-build-system)
+  (propagated-inputs
+    `(("emacs-evil" ,emacs-evil)
+      ("emacs-anzu" ,emacs-anzu)))
+  (home-page
+    "https://github.com/syohex/emacs-evil-anzu")
+  (synopsis "anzu for evil-mode")
+  (description "Anzu provides a minor mode that displays the current match and
+total match information in the mode-line in various search modes.")
+  (license license:gpl3+)))
+
-- 
2.16.2

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

* [bug#31081] [PATCH] gnu: Add emacs-anzu-evil
  2018-04-06 19:20 [bug#31081] [PATCH] gnu: Add emacs-anzu-evil Sohom Bhattacharjee
@ 2018-04-07 16:25 ` Oleg Pykhalov
  2018-04-07 18:17   ` Sohom Bhattacharjee
  0 siblings, 1 reply; 4+ messages in thread
From: Oleg Pykhalov @ 2018-04-07 16:25 UTC (permalink / raw)
  To: Sohom Bhattacharjee; +Cc: 31081

[-- Attachment #1: Type: text/plain, Size: 2299 bytes --]

Hello Soham,

Sohom Bhattacharjee <soham.bhattacharjee15@gmail.com> writes:

> * gnu/packages/emacs.scm (emacs-anzu-evil): New variable.
> ---
> test
>
>  gnu/packages/emacs.scm | 23 +++++++++++++++++++++++
>  1 file changed, 23 insertions(+)
>
> diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
> index fd0b8acd0..2463d0dab 100644
> --- a/gnu/packages/emacs.scm
> +++ b/gnu/packages/emacs.scm
> @@ -7494,3 +7494,26 @@ used with SGML-like languages: XML, HTML, XHTML, XSL, etc.")
>  standard Unix password manager\").")
>      (license license:gpl2+)))
>  
> +(define-public emacs-evil-anzu
> + (package
> +  (name "emacs-evil-anzu")
> +  (version "0.03")
> +  (source
> +   (origin
> +     (method url-fetch)
> +     (uri (string-append "https://github.com/syohex/emacs-evil-anzu/archive/"
> +                         version ".tar.gz"))
> +     (file-name (string-append name "-" version ".tar.gz"))
> +     (sha256
> +      (base32 "032hh2946z529cizqsg8pm6cpn5qdj8lfk3qskmx6xv3g2ra56ns"))))
> +  (build-system emacs-build-system)
> +  (propagated-inputs
> +    `(("emacs-evil" ,emacs-evil)
> +      ("emacs-anzu" ,emacs-anzu)))
> +  (home-page
> +    "https://github.com/syohex/emacs-evil-anzu")
       ^
       This string could be on the previous line (‘M-^’ will do it).

> +  (synopsis "anzu for evil-mode")

--8<---------------cut here---------------start------------->8---
$ ./pre-inst-env guix lint emacs-evil-anzu

gnu/packages/emacs.scm:7515:12: emacs-evil-anzu@0.03: synopsis should start with an upper-case letter or digit
--8<---------------cut here---------------end--------------->8---

I think ‘@code{anzu}’ will be appropriate.

> +  (description "Anzu provides a minor mode that displays the current match and
> +total match information in the mode-line in various search modes.")
> +  (license license:gpl3+)))
> +

The whole ‘(define-public emacs-evil-anzu …)’ should be indented.  I'm
not an ‘evil-mode’ user, but in pure Emacs you could do it with ‘M-h’
and ‘TAB’.  Also it will probably require to fill some strings with
‘M-q’.  Alternatively you use ‘etc/indent-code.el’.  See
https://www.gnu.org/software/guix/manual/html_node/Formatting-Code.html

Thanks,
Oleg.

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

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

* [bug#31081] [PATCH] gnu: Add emacs-anzu-evil
  2018-04-07 16:25 ` Oleg Pykhalov
@ 2018-04-07 18:17   ` Sohom Bhattacharjee
  2018-04-07 18:38     ` Oleg Pykhalov
  0 siblings, 1 reply; 4+ messages in thread
From: Sohom Bhattacharjee @ 2018-04-07 18:17 UTC (permalink / raw)
  To: Oleg Pykhalov; +Cc: 31081

[-- Attachment #1: Type: text/plain, Size: 2507 bytes --]

I have made the suggested changes and attached the new patch. 

Thank You :-)


On Sat, Apr 07, 2018 at 07:25:22PM +0300, Oleg Pykhalov wrote:
> Hello Soham,
> 
> Sohom Bhattacharjee <soham.bhattacharjee15@gmail.com> writes:
> 
> > * gnu/packages/emacs.scm (emacs-anzu-evil): New variable.
> > ---
> > test
> >
> >  gnu/packages/emacs.scm | 23 +++++++++++++++++++++++
> >  1 file changed, 23 insertions(+)
> >
> > diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
> > index fd0b8acd0..2463d0dab 100644
> > --- a/gnu/packages/emacs.scm
> > +++ b/gnu/packages/emacs.scm
> > @@ -7494,3 +7494,26 @@ used with SGML-like languages: XML, HTML, XHTML, XSL, etc.")
> >  standard Unix password manager\").")
> >      (license license:gpl2+)))
> >  
> > +(define-public emacs-evil-anzu
> > + (package
> > +  (name "emacs-evil-anzu")
> > +  (version "0.03")
> > +  (source
> > +   (origin
> > +     (method url-fetch)
> > +     (uri (string-append "https://github.com/syohex/emacs-evil-anzu/archive/"
> > +                         version ".tar.gz"))
> > +     (file-name (string-append name "-" version ".tar.gz"))
> > +     (sha256
> > +      (base32 "032hh2946z529cizqsg8pm6cpn5qdj8lfk3qskmx6xv3g2ra56ns"))))
> > +  (build-system emacs-build-system)
> > +  (propagated-inputs
> > +    `(("emacs-evil" ,emacs-evil)
> > +      ("emacs-anzu" ,emacs-anzu)))
> > +  (home-page
> > +    "https://github.com/syohex/emacs-evil-anzu")
>        ^
>        This string could be on the previous line (‘M-^’ will do it).
> 
> > +  (synopsis "anzu for evil-mode")
> 
> --8<---------------cut here---------------start------------->8---
> $ ./pre-inst-env guix lint emacs-evil-anzu
> 
> gnu/packages/emacs.scm:7515:12: emacs-evil-anzu@0.03: synopsis should start with an upper-case letter or digit
> --8<---------------cut here---------------end--------------->8---
> 
> I think ‘@code{anzu}’ will be appropriate.
> 
> > +  (description "Anzu provides a minor mode that displays the current match and
> > +total match information in the mode-line in various search modes.")
> > +  (license license:gpl3+)))
> > +
> 
> The whole ‘(define-public emacs-evil-anzu …)’ should be indented.  I'm
> not an ‘evil-mode’ user, but in pure Emacs you could do it with ‘M-h’
> and ‘TAB’.  Also it will probably require to fill some strings with
> ‘M-q’.  Alternatively you use ‘etc/indent-code.el’.  See
> https://www.gnu.org/software/guix/manual/html_node/Formatting-Code.html
> 
> Thanks,
> Oleg.



[-- Attachment #2: 0001-gnu-Add-emacs-evil-anzu.patch --]
[-- Type: text/plain, Size: 1539 bytes --]

From baa056f96e1bb5a2fba0276a729237762d697cd2 Mon Sep 17 00:00:00 2001
From: Sohom Bhattacharjee <soham.bhattacharjee15@gmail.com>
Date: Sat, 7 Apr 2018 23:36:31 +0530
Subject: [PATCH] gnu: Add emacs-evil-anzu.

* gnu/packages/emacs.scm (emacs-evil-anzu): New variable.
---
 gnu/packages/emacs.scm | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index fd0b8acd0..0ff712117 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -7494,3 +7494,25 @@ used with SGML-like languages: XML, HTML, XHTML, XSL, etc.")
 standard Unix password manager\").")
     (license license:gpl2+)))
 
+
+(define-public emacs-evil-anzu
+  (package
+    (name "emacs-evil-anzu")
+    (version "0.03")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append "https://github.com/syohex/emacs-evil-anzu/archive/"
+                           version ".tar.gz"))
+       (file-name (string-append name "-" version ".tar.gz"))
+       (sha256
+        (base32 "032hh2946z529cizqsg8pm6cpn5qdj8lfk3qskmx6xv3g2ra56ns"))))
+    (build-system emacs-build-system)
+    (propagated-inputs
+     `(("emacs-evil" ,emacs-evil)
+       ("emacs-anzu" ,emacs-anzu)))
+    (home-page "https://github.com/syohex/emacs-evil-anzu")
+    (synopsis "Anzu for evil-mode")
+    (description "@code{anzu} provides a minor mode that displays the current match
+and total match information in the mode-line in various search modes.")
+    (license license:gpl3+)))
-- 
2.16.2


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

* [bug#31081] [PATCH] gnu: Add emacs-anzu-evil
  2018-04-07 18:17   ` Sohom Bhattacharjee
@ 2018-04-07 18:38     ` Oleg Pykhalov
  0 siblings, 0 replies; 4+ messages in thread
From: Oleg Pykhalov @ 2018-04-07 18:38 UTC (permalink / raw)
  To: Sohom Bhattacharjee; +Cc: 31081, 31081-done

[-- Attachment #1: Type: text/plain, Size: 2022 bytes --]

Sohom Bhattacharjee <soham.bhattacharjee15@gmail.com> writes:

> I have made the suggested changes and attached the new patch. 
>
> Thank You :-)

[…]

> From baa056f96e1bb5a2fba0276a729237762d697cd2 Mon Sep 17 00:00:00 2001
> From: Sohom Bhattacharjee <soham.bhattacharjee15@gmail.com>
> Date: Sat, 7 Apr 2018 23:36:31 +0530
> Subject: [PATCH] gnu: Add emacs-evil-anzu.
>
> * gnu/packages/emacs.scm (emacs-evil-anzu): New variable.
> ---
>  gnu/packages/emacs.scm | 22 ++++++++++++++++++++++
>  1 file changed, 22 insertions(+)
>
> diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
> index fd0b8acd0..0ff712117 100644
> --- a/gnu/packages/emacs.scm
> +++ b/gnu/packages/emacs.scm
> @@ -7494,3 +7494,25 @@ used with SGML-like languages: XML, HTML, XHTML, XSL, etc.")
>  standard Unix password manager\").")
>      (license license:gpl2+)))
>  
> +
  ^
  Only 1 empty line between definitions.

> +(define-public emacs-evil-anzu
> +  (package
> +    (name "emacs-evil-anzu")
> +    (version "0.03")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append "https://github.com/syohex/emacs-evil-anzu/archive/"

This line is longer than 78.

> +                           version ".tar.gz"))
> +       (file-name (string-append name "-" version ".tar.gz"))
> +       (sha256
> +        (base32 "032hh2946z529cizqsg8pm6cpn5qdj8lfk3qskmx6xv3g2ra56ns"))))
> +    (build-system emacs-build-system)
> +    (propagated-inputs
> +     `(("emacs-evil" ,emacs-evil)
> +       ("emacs-anzu" ,emacs-anzu)))
> +    (home-page "https://github.com/syohex/emacs-evil-anzu")
> +    (synopsis "Anzu for evil-mode")
> +    (description "@code{anzu} provides a minor mode that displays the current match

The description is longer than 78.

> +and total match information in the mode-line in various search modes.")
> +    (license license:gpl3+)))

Pushed as d4bb4551356c6601a952e89ad8e1b56a3bf2244d with changes above.

Thanks,
Oleg.

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

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

end of thread, other threads:[~2018-04-07 18:39 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-04-06 19:20 [bug#31081] [PATCH] gnu: Add emacs-anzu-evil Sohom Bhattacharjee
2018-04-07 16:25 ` Oleg Pykhalov
2018-04-07 18:17   ` Sohom Bhattacharjee
2018-04-07 18:38     ` Oleg Pykhalov

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.