all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH] gnu: Add emacs-aggresive-indent-mode.
@ 2017-03-23  2:11 Vasile Dumitrascu
  2017-03-23 18:13 ` Marius Bakke
  0 siblings, 1 reply; 5+ messages in thread
From: Vasile Dumitrascu @ 2017-03-23  2:11 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/emacs.scm (emacs-aggresive-indent-mode): New variable.
---
 gnu/packages/emacs.scm | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 547c44033..301552839 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -1066,6 +1066,31 @@ like.  It can be linked with various Emacs mail
clients (Message and Mail
 mode, Rmail, Gnus, MH-E, and VM).  BBDB is fully customizable.")
     (license license:gpl3+)))

+(package
+  (name "emacs-aggresive-indent-mode")
+  (version "1.8.3")
+  (source
+   (origin
+     (method url-fetch)
+     (uri (string-append "https://elpa.gnu.org/packages/"
+                         "aggressive-indent-" version ".el"))
+     (sha256
+      (base32
+       "0jnzccl50x0wapprgwxinp99pwwa6j43q6msn4gv437j7swy8wnj"))))
+  (build-system emacs-build-system)
+  (home-page "https://elpa.gnu.org/packages/aggressive-indent.html")
+  (synopsis "Minor mode to aggressively keep your code always indented")
+  (description
+   "`electric-indent-mode' is enough to keep your code nicely aligned when
+all you do is type.  However, once you start shifting blocks around,
+transposing lines, or slurping and barfing sexps, indentation is bound
+to go wrong.
+
+`aggressive-indent-mode' is a minor mode that keeps your code always
+indented.  It reindents after every change, making it more reliable
+than `electric-indent-mode'.")
+  (license gpl2+))
+
 (define-public emacs-async
   (package
     (name "emacs-async")
-- 
2.11.0

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

* Re: [PATCH] gnu: Add emacs-aggresive-indent-mode.
  2017-03-23  2:11 [PATCH] gnu: Add emacs-aggresive-indent-mode Vasile Dumitrascu
@ 2017-03-23 18:13 ` Marius Bakke
  2017-03-24  2:56   ` Vasile Dumitrascu
  0 siblings, 1 reply; 5+ messages in thread
From: Marius Bakke @ 2017-03-23 18:13 UTC (permalink / raw)
  To: Vasile Dumitrascu, guix-devel

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

Vasile Dumitrascu <va511e@yahoo.com> writes:

> * gnu/packages/emacs.scm (emacs-aggresive-indent-mode): New variable.

Thanks!

> ---
>  gnu/packages/emacs.scm | 25 +++++++++++++++++++++++++
>  1 file changed, 25 insertions(+)
>
> diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
> index 547c44033..301552839 100644
> --- a/gnu/packages/emacs.scm
> +++ b/gnu/packages/emacs.scm
> @@ -1066,6 +1066,31 @@ like.  It can be linked with various Emacs mail
> clients (Message and Mail
>  mode, Rmail, Gnus, MH-E, and VM).  BBDB is fully customizable.")
>      (license license:gpl3+)))
>
> +(package

Hmm. This should be enclosed in a 'define-public' block.

> +  (name "emacs-aggresive-indent-mode")
                      ^^^

There should be an extra 's' here. Also '-mode' can be dropped.


> +  (version "1.8.3")
> +  (source
> +   (origin
> +     (method url-fetch)
> +     (uri (string-append "https://elpa.gnu.org/packages/"
> +                         "aggressive-indent-" version ".el"))
> +     (sha256
> +      (base32
> +       "0jnzccl50x0wapprgwxinp99pwwa6j43q6msn4gv437j7swy8wnj"))))
> +  (build-system emacs-build-system)
> +  (home-page "https://elpa.gnu.org/packages/aggressive-indent.html")
> +  (synopsis "Minor mode to aggressively keep your code always indented")
> +  (description
> +   "`electric-indent-mode' is enough to keep your code nicely aligned when
> +all you do is type.  However, once you start shifting blocks around,
> +transposing lines, or slurping and barfing sexps, indentation is bound
> +to go wrong.
> +
> +`aggressive-indent-mode' is a minor mode that keeps your code always
> +indented.  It reindents after every change, making it more reliable
> +than `electric-indent-mode'.")

The quotes in the description should be replaced with @code{} blocks.
Running `./pre-inst-env guix lint emacs-aggressive-indent` should warn
about this. I also think that the second paragraph is sufficient.

Can you send an updated patch? Please also add a copyright line for
yourself at the top of the file.

TIA!


> +  (license gpl2+))
> +
>  (define-public emacs-async
>    (package
>      (name "emacs-async")
> -- 
> 2.11.0

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

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

* Re: [PATCH] gnu: Add emacs-aggresive-indent-mode.
  2017-03-23 18:13 ` Marius Bakke
@ 2017-03-24  2:56   ` Vasile Dumitrascu
  2017-03-24 13:22     ` Ricardo Wurmus
  0 siblings, 1 reply; 5+ messages in thread
From: Vasile Dumitrascu @ 2017-03-24  2:56 UTC (permalink / raw)
  To: Marius Bakke, guix-devel

    Hi

  You are correct. I have recreated the patch as I have trashed the old
repo clone, please find below the new version. Thank you,

  Vasile


* gnu/packages/emacs.scm (emacs-aggressive-indent): New variable.
---
 gnu/packages/emacs.scm | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index b84faaff0..33a919458 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -20,6 +20,7 @@
 ;;; Copyright © 2017 Christopher Baines <mail@cbaines.net>
 ;;; Copyright © 2017 Mathieu Othacehe <m.othacehe@gmail.com>
 ;;; Copyright © 2017 Clément Lassieur <clement@lassieur.org>
+;;; Copyright © 2017 Vasile Dumitrascu <va511e@yahoo.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -1080,6 +1081,26 @@ like.  It can be linked with various Emacs mail
clients (Message and Mail
 mode, Rmail, Gnus, MH-E, and VM).  BBDB is fully customizable.")
     (license license:gpl3+)))

+(define-public emacs-aggressive-indent
+  (package
+    (name "emacs-aggressive-indent")
+    (version "1.8.3")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "https://elpa.gnu.org/packages/"
+                                  "aggressive-indent-" version ".el"))
+              (sha256
+               (base32
+                "0jnzccl50x0wapprgwxinp99pwwa6j43q6msn4gv437j7swy8wnj"))))
+    (build-system emacs-build-system)
+    (home-page "https://elpa.gnu.org/packages/aggressive-indent.html")
+    (synopsis "Minor mode to aggressively keep your code always indented")
+    (description
+     "@code{aggressive-indent-mode} is a minor mode that keeps your code
+always indented.  It reindents after every change, making it more reliable
+than @code{electric-indent-mode}.")
+    (license license:gpl2+)))
+
 (define-public emacs-ag
   (package
     (name "emacs-ag")
--
2.11.0



Marius Bakke:
> Vasile Dumitrascu <va511e@yahoo.com> writes:
> 
>> * gnu/packages/emacs.scm (emacs-aggresive-indent-mode): New variable.
> 
> Thanks!
> 
>> ---
>>  gnu/packages/emacs.scm | 25 +++++++++++++++++++++++++
>>  1 file changed, 25 insertions(+)
>>
>> diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
>> index 547c44033..301552839 100644
>> --- a/gnu/packages/emacs.scm
>> +++ b/gnu/packages/emacs.scm
>> @@ -1066,6 +1066,31 @@ like.  It can be linked with various Emacs mail
>> clients (Message and Mail
>>  mode, Rmail, Gnus, MH-E, and VM).  BBDB is fully customizable.")
>>      (license license:gpl3+)))
>>
>> +(package
> 
> Hmm. This should be enclosed in a 'define-public' block.
> 
>> +  (name "emacs-aggresive-indent-mode")
>                       ^^^
> 
> There should be an extra 's' here. Also '-mode' can be dropped.
> 
> 
>> +  (version "1.8.3")
>> +  (source
>> +   (origin
>> +     (method url-fetch)
>> +     (uri (string-append "https://elpa.gnu.org/packages/"
>> +                         "aggressive-indent-" version ".el"))
>> +     (sha256
>> +      (base32
>> +       "0jnzccl50x0wapprgwxinp99pwwa6j43q6msn4gv437j7swy8wnj"))))
>> +  (build-system emacs-build-system)
>> +  (home-page "https://elpa.gnu.org/packages/aggressive-indent.html")
>> +  (synopsis "Minor mode to aggressively keep your code always indented")
>> +  (description
>> +   "`electric-indent-mode' is enough to keep your code nicely aligned when
>> +all you do is type.  However, once you start shifting blocks around,
>> +transposing lines, or slurping and barfing sexps, indentation is bound
>> +to go wrong.
>> +
>> +`aggressive-indent-mode' is a minor mode that keeps your code always
>> +indented.  It reindents after every change, making it more reliable
>> +than `electric-indent-mode'.")
> 
> The quotes in the description should be replaced with @code{} blocks.
> Running `./pre-inst-env guix lint emacs-aggressive-indent` should warn
> about this. I also think that the second paragraph is sufficient.
> 
> Can you send an updated patch? Please also add a copyright line for
> yourself at the top of the file.
> 
> TIA!
> 
> 
>> +  (license gpl2+))
>> +
>>  (define-public emacs-async
>>    (package
>>      (name "emacs-async")
>> -- 
>> 2.11.0

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

* Re: [PATCH] gnu: Add emacs-aggresive-indent-mode.
  2017-03-24  2:56   ` Vasile Dumitrascu
@ 2017-03-24 13:22     ` Ricardo Wurmus
  2017-03-24 14:43       ` Vasile Dumitrascu
  0 siblings, 1 reply; 5+ messages in thread
From: Ricardo Wurmus @ 2017-03-24 13:22 UTC (permalink / raw)
  To: Vasile Dumitrascu; +Cc: guix-devel


Hi Vasile,

>   You are correct. I have recreated the patch as I have trashed the old
> repo clone, please find below the new version. Thank you,

Thanks for the updated commit.  I couldn’t apply it automatically for
some reason, so I applied it manually.  It’s now in master with commit
4d3d3bd21.

We have recently switched to using guix-patches@gnu.org for patch
submissions.  In the future, please send patches there instead of
guix-devel@gnu.org, so that we don’t accidentally drop your
contributions.

Thanks again for your contribution and welcome!

--
Ricardo

GPG: BCA6 89B6 3655 3801 C3C6  2150 197A 5888 235F ACAC
https://elephly.net

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

* Re: [PATCH] gnu: Add emacs-aggresive-indent-mode.
  2017-03-24 13:22     ` Ricardo Wurmus
@ 2017-03-24 14:43       ` Vasile Dumitrascu
  0 siblings, 0 replies; 5+ messages in thread
From: Vasile Dumitrascu @ 2017-03-24 14:43 UTC (permalink / raw)
  To: Ricardo Wurmus; +Cc: guix-devel

  Hi Ricardo,

Perfect. Thank you,

  Vasile


Ricardo Wurmus:
> 
> Hi Vasile,
> 
>>   You are correct. I have recreated the patch as I have trashed the old
>> repo clone, please find below the new version. Thank you,
> 
> Thanks for the updated commit.  I couldn’t apply it automatically for
> some reason, so I applied it manually.  It’s now in master with commit
> 4d3d3bd21.
> 
> We have recently switched to using guix-patches@gnu.org for patch
> submissions.  In the future, please send patches there instead of
> guix-devel@gnu.org, so that we don’t accidentally drop your
> contributions.
> 
> Thanks again for your contribution and welcome!
> 
> --
> Ricardo
> 
> GPG: BCA6 89B6 3655 3801 C3C6  2150 197A 5888 235F ACAC
> https://elephly.net
> 

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

end of thread, other threads:[~2017-03-24 14:43 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-03-23  2:11 [PATCH] gnu: Add emacs-aggresive-indent-mode Vasile Dumitrascu
2017-03-23 18:13 ` Marius Bakke
2017-03-24  2:56   ` Vasile Dumitrascu
2017-03-24 13:22     ` Ricardo Wurmus
2017-03-24 14:43       ` Vasile Dumitrascu

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.