all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH] New package: aspell-dict-pt-br
@ 2017-04-22 20:38 Daniel Pimentel
  2017-04-22 21:46 ` Danny Milosavljevic
  0 siblings, 1 reply; 6+ messages in thread
From: Daniel Pimentel @ 2017-04-22 20:38 UTC (permalink / raw)
  To: guix-devel

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

Guixs,

in attach the patch for Aspell dictionary for Brazilian Portuguese.

Thanks,

-- 
Daniel Pimentel (d4n1)

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-Add-aspell-dict-pt-br.patch --]
[-- Type: text/x-diff; name=0001-gnu-Add-aspell-dict-pt-br.patch, Size: 1212 bytes --]

From 92481c27ab969e4a3e621560ab6aefadceaae84a Mon Sep 17 00:00:00 2001
From: Daniel Pimentel <d4n1@d4n1.org>
Date: Sat, 22 Apr 2017 17:32:47 -0300
Subject: [PATCH] gnu: Add aspell-dict-pt-br * gnu/packages/aspell.scm: Add new
 package.

---
 gnu/packages/aspell.scm | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/aspell.scm b/gnu/packages/aspell.scm
index 188e91d1e..61cfcb345 100644
--- a/gnu/packages/aspell.scm
+++ b/gnu/packages/aspell.scm
@@ -62,7 +62,6 @@ documents written in the UTF-8 encoding and its ability to use multiple
 dictionaries, including personal ones.")
     (license lgpl2.1+)))
 
-\f
 ;;;
 ;;; Dictionaries.
 ;;;
@@ -190,3 +189,11 @@ dictionaries, including personal ones.")
                      #:sha256
                      (base32
                       "1zxr8958v37v260fkqd4pg37ns5h5kyqm54hn1hg70wq5cz8h512")))
+
+(define-public aspell-dict-pt-br
+  (aspell-dictionary "pt_BR" "Brazilian Portuguese"
+                     #:version "20090702-0"
+                     #:prefix "aspell6-"
+                     #:sha256
+                     (base32
+                      "1y09lx9zf2rnp55r16b2vgj953l3538z1vaqgflg9mdvm555bz3p")))
-- 
2.12.2


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

* Re: [PATCH] New package: aspell-dict-pt-br
  2017-04-22 20:38 [PATCH] New package: aspell-dict-pt-br Daniel Pimentel
@ 2017-04-22 21:46 ` Danny Milosavljevic
  2017-04-22 23:10   ` Ludovic Courtès
  0 siblings, 1 reply; 6+ messages in thread
From: Danny Milosavljevic @ 2017-04-22 21:46 UTC (permalink / raw)
  To: Daniel Pimentel; +Cc: guix-devel

Hi,

for simplicity, I'd prefer if the variable was called aspell-dict-pt_BR like the package.  I don't like pointless aliases much.

What do you think?

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

* Re: [PATCH] New package: aspell-dict-pt-br
  2017-04-22 21:46 ` Danny Milosavljevic
@ 2017-04-22 23:10   ` Ludovic Courtès
  0 siblings, 0 replies; 6+ messages in thread
From: Ludovic Courtès @ 2017-04-22 23:10 UTC (permalink / raw)
  To: Danny Milosavljevic; +Cc: guix-devel

Hello,

Danny Milosavljevic <dannym@scratchpost.org> skribis:

> for simplicity, I'd prefer if the variable was called aspell-dict-pt_BR like the package.  I don't like pointless aliases much.

Following our conventions (info "(guix) Package Naming") I changed the
package name to “aspell-dict-pt-br” and applied the patch.

Thanks Daniel!

Ludo’.

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

* Re: [PATCH] New package: aspell-dict-pt-br
@ 2017-04-23  0:42 D4n1
  0 siblings, 0 replies; 6+ messages in thread
From: D4n1 @ 2017-04-23  0:42 UTC (permalink / raw)
  To: Danny Milosavljevic; +Cc: guix-devel

I agree. I'll update it and I'll send again.

Thanks,On Apr 22, 2017 6:46 PM, Danny Milosavljevic <dannym@scratchpost.org> wrote:
>
> Hi, 
>
> for simplicity, I'd prefer if the variable was called aspell-dict-pt_BR like the package.  I don't like pointless aliases much. 
>
> What do you think? 

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

* Re: [PATCH] New package: aspell-dict-pt-br
       [not found] <20170423011224.13B581CA1FE9@dd1012.kasserver.com>
@ 2017-04-23  1:39 ` Danny Milosavljevic
  0 siblings, 0 replies; 6+ messages in thread
From: Danny Milosavljevic @ 2017-04-23  1:39 UTC (permalink / raw)
  To: D4n1; +Cc: guix-devel

No, Ludo has applied it already (with changes), it's in master now.

He's fixed it the other way around: now the package is also called aspell-dict-pt-br.

Thanks for the patch!

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

* Re: [PATCH] New package: aspell-dict-pt-br
@ 2017-04-23  2:17 D4n1
  0 siblings, 0 replies; 6+ messages in thread
From: D4n1 @ 2017-04-23  2:17 UTC (permalink / raw)
  To: Danny Milosavljevic; +Cc: guix-devel

Thanks :)

Happy hacking,On Apr 22, 2017 10:39 PM, Danny Milosavljevic <dannym@scratchpost.org> wrote:
>
> No, Ludo has applied it already (with changes), it's in master now. 
>
> He's fixed it the other way around: now the package is also called aspell-dict-pt-br. 
>
> Thanks for the patch! 

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

end of thread, other threads:[~2017-04-23  2:17 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-04-22 20:38 [PATCH] New package: aspell-dict-pt-br Daniel Pimentel
2017-04-22 21:46 ` Danny Milosavljevic
2017-04-22 23:10   ` Ludovic Courtès
  -- strict thread matches above, loose matches on Subject: below --
2017-04-23  0:42 D4n1
     [not found] <20170423011224.13B581CA1FE9@dd1012.kasserver.com>
2017-04-23  1:39 ` Danny Milosavljevic
2017-04-23  2:17 D4n1

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.