[PATCH] New package: aspell-dict-pt-br

[PATCH] New package: aspell-dict-pt-br

[Daniel Pimentel]

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

Re: [PATCH] New package: aspell-dict-pt-br

[Danny Milosavljevic]

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?

Re: [PATCH] New package: aspell-dict-pt-br

[Ludovic Courtès]

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’.

Re: [PATCH] New package: aspell-dict-pt-br

[D4n1]

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? 

Re: [PATCH] New package: aspell-dict-pt-br

[Danny Milosavljevic]

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!

Re: [PATCH] New package: aspell-dict-pt-br

[D4n1]

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!