[PATCH 0/5] Add emacs-mu4e-alert

[PATCH 0/5] Add emacs-mu4e-alert

[Mathieu Othacehe]

The point of this serie is to add mu4e-alert and it's dependencies.
I used the importer for every package and rewrote incorrect descriptions. 

Mathieu Othacehe (5):
  gnu: Add emacs-ht.
  gnu: Add emacs-log4e.
  gnu: Add emacs-gntp.
  gnu: Add emacs-alert.
  gnu: Add emacs-mu4e-alert.

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

-- 
2.11.0

[PATCH 1/5] gnu: Add emacs-ht.

[Mathieu Othacehe]

* gnu/packages/emacs.scm (emacs-ht): 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 d3e64c04f..699d41712 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -3645,3 +3645,28 @@ Streams are implemented as delayed evaluation of cons cells.")
       (description "This package provides expression based interactive search
 procedures for emacs-lisp-mode.")
       (license license:gpl3+))))
+
+(define-public emacs-ht
+  (package
+  (name "emacs-ht")
+  (version "20161015.1945")
+  (source
+    (origin
+      (method url-fetch)
+      (uri (string-append
+             "http://melpa.org/packages/ht-"
+             version
+             ".el"))
+      (sha256
+        (base32
+          "1ar6rw0vl18pws72nm167jfxn9vzl8ngnh1xw3npfk19x3y6n5g6"))))
+  (build-system emacs-build-system)
+  (propagated-inputs `(("emacs-dash" ,emacs-dash)))
+  (home-page "https://github.com/Wilfred/ht.el")
+  (synopsis
+    "Hash table library for Emacs")
+  (description
+    "This package simplifies the use of hash tables in elisp. It also
+provides functions to convert hash tables from and to alists and plists.")
+  (license license:gpl3+)))
+
-- 
2.11.0

Re: [PATCH 1/5] gnu: Add emacs-ht.

[Alex Kost]

Mathieu Othacehe (2017-01-26 13:31 +0100) wrote:

> * gnu/packages/emacs.scm (emacs-ht): 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 d3e64c04f..699d41712 100644
> --- a/gnu/packages/emacs.scm
> +++ b/gnu/packages/emacs.scm
> @@ -3645,3 +3645,28 @@ Streams are implemented as delayed evaluation of cons cells.")
>        (description "This package provides expression based interactive search
>  procedures for emacs-lisp-mode.")
>        (license license:gpl3+))))
> +
> +(define-public emacs-ht
> +  (package
> +  (name "emacs-ht")
> +  (version "20161015.1945")
> +  (source
> +    (origin
> +      (method url-fetch)
> +      (uri (string-append
> +             "http://melpa.org/packages/ht-"
> +             version
> +             ".el"))

We can't use files from melpa.org because once a new commit appears in
the upstream repo, melpa will rebuilt the package and remove the
previous version, so this package will lost its source and will not be
buildable anymore.  So please use the latest available tarballs instead.
In this case it is:

  https://github.com/Wilfred/ht.el/archive/2.1.tar.gz

> +      (sha256
> +        (base32
> +          "1ar6rw0vl18pws72nm167jfxn9vzl8ngnh1xw3npfk19x3y6n5g6"))))
> +  (build-system emacs-build-system)
> +  (propagated-inputs `(("emacs-dash" ,emacs-dash)))
> +  (home-page "https://github.com/Wilfred/ht.el")
> +  (synopsis
> +    "Hash table library for Emacs")
> +  (description
> +    "This package simplifies the use of hash tables in elisp. It also
> +provides functions to convert hash tables from and to alists and plists.")
> +  (license license:gpl3+)))
> +

-- 
Alex

Re: [PATCH 1/5] gnu: Add emacs-ht.

[Leo Famulari]

On Fri, Jan 27, 2017 at 12:20:46PM +0300, Alex Kost wrote:
> Mathieu Othacehe (2017-01-26 13:31 +0100) wrote:
> > +      (uri (string-append
> > +             "http://melpa.org/packages/ht-"
> > +             version
> > +             ".el"))
> 
> We can't use files from melpa.org because once a new commit appears in
> the upstream repo, melpa will rebuilt the package and remove the
> previous version, so this package will lost its source and will not be
> buildable anymore.  So please use the latest available tarballs instead.

I think the linter should warn about this. It seems like you have to
send this email often.

Re: [PATCH 1/5] gnu: Add emacs-ht.

[Mathieu OTHACEHE]

> On Fri, Jan 27, 2017 at 12:20:46PM +0300, Alex Kost wrote:
>> Mathieu Othacehe (2017-01-26 13:31 +0100) wrote:
>> > +      (uri (string-append
>> > +             "http://melpa.org/packages/ht-"
>> > +             version
>> > +             ".el"))
>> 
>> We can't use files from melpa.org because once a new commit appears in
>> the upstream repo, melpa will rebuilt the package and remove the
>> previous version, so this package will lost its source and will not be
>> buildable anymore.  So please use the latest available tarballs instead.

Ok, I'll send and update.

>
> I think the linter should warn about this. It seems like you have to
> send this email often.

Yes, or modify guix import for melpa ? It seems that I
trusted wrongly the output of the importer ...

Mathieu

Re: [PATCH 1/5] gnu: Add emacs-ht.

[Alex Kost]

Mathieu OTHACEHE (2017-01-27 11:16 +0100) wrote:

>> On Fri, Jan 27, 2017 at 12:20:46PM +0300, Alex Kost wrote:
>>> Mathieu Othacehe (2017-01-26 13:31 +0100) wrote:
>>> > +      (uri (string-append
>>> > +             "http://melpa.org/packages/ht-"
>>> > +             version
>>> > +             ".el"))
>>> 
>>> We can't use files from melpa.org because once a new commit appears in
>>> the upstream repo, melpa will rebuilt the package and remove the
>>> previous version, so this package will lost its source and will not be
>>> buildable anymore.  So please use the latest available tarballs instead.
>
> Ok, I'll send and update.
>
>>
>> I think the linter should warn about this. It seems like you have to
>> send this email often.
>
> Yes, or modify guix import for melpa ? It seems that I
> trusted wrongly the output of the importer ...

Yeah, I think adjusting guix import for that would be really great (but
I'm not a volunteer for this task :-))

-- 
Alex

Re: [PATCH 1/5] gnu: Add emacs-ht.

[Ludovic Courtès]

Alex Kost <alezost@gmail.com> skribis:

> Mathieu OTHACEHE (2017-01-27 11:16 +0100) wrote:
>
>>> On Fri, Jan 27, 2017 at 12:20:46PM +0300, Alex Kost wrote:
>>>> Mathieu Othacehe (2017-01-26 13:31 +0100) wrote:
>>>> > +      (uri (string-append
>>>> > +             "http://melpa.org/packages/ht-"
>>>> > +             version
>>>> > +             ".el"))
>>>> 
>>>> We can't use files from melpa.org because once a new commit appears in
>>>> the upstream repo, melpa will rebuilt the package and remove the
>>>> previous version, so this package will lost its source and will not be
>>>> buildable anymore.  So please use the latest available tarballs instead.
>>
>> Ok, I'll send and update.
>>
>>>
>>> I think the linter should warn about this. It seems like you have to
>>> send this email often.
>>
>> Yes, or modify guix import for melpa ? It seems that I
>> trusted wrongly the output of the importer ...
>
> Yeah, I think adjusting guix import for that would be really great (but
> I'm not a volunteer for this task :-))

What needs to be done?  We cannot guess the upstream URL just from the
MELPA one, can we?

Ludo’.

Re: [PATCH 1/5] gnu: Add emacs-ht.

[Alex Kost]

Ludovic Courtès (2017-01-28 00:16 +0100) wrote:

> Alex Kost <alezost@gmail.com> skribis:
>
>> Mathieu OTHACEHE (2017-01-27 11:16 +0100) wrote:
>>
>>>> On Fri, Jan 27, 2017 at 12:20:46PM +0300, Alex Kost wrote:
>>>>> Mathieu Othacehe (2017-01-26 13:31 +0100) wrote:
>>>>> > +      (uri (string-append
>>>>> > +             "http://melpa.org/packages/ht-"
>>>>> > +             version
>>>>> > +             ".el"))
>>>>> 
>>>>> We can't use files from melpa.org because once a new commit appears in
>>>>> the upstream repo, melpa will rebuilt the package and remove the
>>>>> previous version, so this package will lost its source and will not be
>>>>> buildable anymore.  So please use the latest available tarballs instead.
>>>
>>> Ok, I'll send and update.
>>>
>>>>
>>>> I think the linter should warn about this. It seems like you have to
>>>> send this email often.
>>>
>>> Yes, or modify guix import for melpa ? It seems that I
>>> trusted wrongly the output of the importer ...
>>
>> Yeah, I think adjusting guix import for that would be really great (but
>> I'm not a volunteer for this task :-))
>
> What needs to be done?  We cannot guess the upstream URL just from the
> MELPA one, can we?

Sorry, but I have never used and looked at importers, and I have no
idea how they work, including the (m)elpa one, so I don't know what
can be done on this subject.

-- 
Alex

Re: [PATCH 1/5] gnu: Add emacs-ht.

[Mathieu OTHACEHE]

> What needs to be done?  We cannot guess the upstream URL just from the
> MELPA one, can we?

No, but we can maybe use the other fields provided by melpa to find the
github archive url ?

There's a field ":url" pointing to github when appropriate :

--8<---------------cut here---------------start------------->8---
(log4e . [(20150105 505) nil "provide logging framework for elisp" single ((:url . "https://github.com/aki2o/log4e") (:keywords "log"))]) 
--8<---------------cut here---------------end--------------->8---

But it seems a bit hacky ...

Mathieu

ELPA importer and upstream URLs

[Ludovic Courtès]

Hello!

Mathieu OTHACEHE <m.othacehe@gmail.com> skribis:

>> What needs to be done?  We cannot guess the upstream URL just from the
>> MELPA one, can we?
>
> No, but we can maybe use the other fields provided by melpa to find the
> github archive url ?

Sounds like a good idea.

> There's a field ":url" pointing to github when appropriate :
>
> (log4e . [(20150105 505) nil "provide logging framework for elisp" single ((:url . "https://github.com/aki2o/log4e") (:keywords "log"))]) 
>
> But it seems a bit hacky ...

The difficulty is that we need both the Git URL and a commit ID.  Is the
commit ID available?

If we have all of that ‘guix import elpa’ could just extract that data
and generate the right thing.  For packages that don’t have a URL/commit
associated with them, it could just keep doing the same as now.

WDYT?  Would you or Carlo like to give it a try?  :-)

Thanks,
Ludo’.

Re: ELPA importer and upstream URLs

[Mathieu OTHACEHE]

> The difficulty is that we need both the Git URL and a commit ID.  Is the
> commit ID available?

I can't find it at first glance, maybe with github API ?

>
> If we have all of that ‘guix import elpa’ could just extract that data
> and generate the right thing.  For packages that don’t have a URL/commit
> associated with them, it could just keep doing the same as now.
>
> WDYT?  Would you or Carlo like to give it a try?  :-)

I'm ok to do it but I'll be afk next couple of weeks.

Mathieu

Re: [PATCH 1/5] gnu: Add emacs-ht.

[Carlo Zancanaro]

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


On Fri, Jan 27 2017, Ludovic Courtès wrote
> What needs to be done?  We cannot guess the upstream URL just from the
> MELPA one, can we?

No, we can't just get it from the URL, but MELPA's recipes should give
us enough information to be able to build them. (After all, they give
MELPA enough information to be able to build them.)

https://github.com/melpa/melpa/tree/master/recipes

I think this is where they're actually processed:

https://github.com/melpa/melpa/blob/8babd80660a50183d81a9269e83c9e473c450ffb/package-build/package-build.el

Carlo

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

[PATCH 2/5] gnu: Add emacs-log4e.

[Mathieu Othacehe]

* gnu/packages/emacs.scm (emacs-log4e): 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 699d41712..022ba4e3d 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -3670,3 +3670,25 @@ procedures for emacs-lisp-mode.")
 provides functions to convert hash tables from and to alists and plists.")
   (license license:gpl3+)))
 
+(define-public emacs-log4e
+  (package
+    (name "emacs-log4e")
+    (version "20150105.505")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append
+             "http://melpa.org/packages/log4e-"
+             version
+             ".el"))
+       (sha256
+        (base32
+         "18bvqjlxmwvfycvk9rg38bp2r6ydxvz0djv63kx3qnn8qp5hbvvw"))))
+    (build-system emacs-build-system)
+    (home-page "https://github.com/aki2o/log4e")
+    (synopsis "Logging framework for elisp.")
+    (description
+     "This package provides a logging framework for elisp. It allows to deal with
+multiple log levels.")
+    (license license:gpl3+)))
+
-- 
2.11.0

[PATCH 3/5] gnu: Add emacs-gntp.

[Mathieu Othacehe]

* gnu/packages/emacs.scm (emacs-gntp): 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 022ba4e3d..3859125dc 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -3692,3 +3692,28 @@ provides functions to convert hash tables from and to alists and plists.")
 multiple log levels.")
     (license license:gpl3+)))
 
+(define-public emacs-gntp
+  (package
+    (name "emacs-gntp")
+    (version "20141024.1950")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append
+             "http://melpa.org/packages/gntp-"
+             version
+             ".el"))
+       (sha256
+        (base32
+         "1m6yj64n1d5fbzhpjx22lph5zs502zlkzhfxlzx9bh2f3cyd2wx3"))))
+    (build-system emacs-build-system)
+    (home-page "https://github.com/tekai/gntp.el")
+    (synopsis
+     "Growl Notification Protocol for Emacs")
+    (description
+     "This package implements the Growl Notification Protocol GNTP
+described at http://www.growlforwindows.com/gfw/help/gntp.aspx
+It is incomplete as it only lets you send but not receive
+notifications.")
+    (license license:gpl3+)))
+
-- 
2.11.0

[PATCH 4/5] gnu: Add emacs-alert.

[Mathieu Othacehe]

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

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 3859125dc..498e17f6b 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -3717,3 +3717,30 @@ It is incomplete as it only lets you send but not receive
 notifications.")
     (license license:gpl3+)))
 
+(define-public emacs-alert
+  (package
+    (name "emacs-alert")
+    (version "20170106.1020")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append
+             "http://melpa.org/packages/alert-"
+             version
+             ".el"))
+       (sha256
+        (base32
+         "0qix6a29z8mdf89sf6sp7cncdc45pai4h5hxzgk63ydl7yqals1v"))))
+    (build-system emacs-build-system)
+    (propagated-inputs
+     `(("emacs-gntp" ,emacs-gntp)
+       ("emacs-log4e" ,emacs-log4e)))
+    (home-page "https://github.com/jwiegley/alert")
+    (synopsis
+     "Growl-style notification system for Emacs")
+    (description
+     "Alert is a Growl-workalike for Emacs which uses a common notification
+interface and multiple, selectable \"styles\", whose use is fully
+customizable by the user.")
+    (license license:gpl3+)))
+
-- 
2.11.0

[PATCH 5/5] gnu: Add emacs-mu4e-alert.

[Mathieu Othacehe]

* gnu/packages/emacs.scm (emacs-mu4e-alert): New variable.
---
 gnu/packages/emacs.scm | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 498e17f6b..0e14d34b4 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -3744,3 +3744,29 @@ interface and multiple, selectable \"styles\", whose use is fully
 customizable by the user.")
     (license license:gpl3+)))
 
+(define-public emacs-mu4e-alert
+  (package
+    (name "emacs-mu4e-alert")
+    (version "20160909.918")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append
+             "http://melpa.org/packages/mu4e-alert-"
+             version
+             ".el"))
+       (sha256
+        (base32
+         "0w2s8690dkmgyf8864c1baa2gb13mxmn0x40l1fni5qgaj5iv49x"))))
+    (build-system emacs-build-system)
+    (propagated-inputs
+     `(("emacs-alert" ,emacs-alert)
+       ("emacs-s" ,emacs-s)
+       ("emacs-ht" ,emacs-ht)))
+    (home-page
+     "https://github.com/iqbalansari/mu4e-alert")
+    (synopsis "Desktop notification for mu4e")
+    (description
+     "This package provides desktop notifications for mu4e, additionally it can
+display the number of unread emails in the modeline.")
+    (license license:gpl3+)))
-- 
2.11.0