Subject: bug#26515: closed (Re: bug#26515: Acknowledgement ([PATCH] gnu: Add emacs-calfw.))

From: [email-masked] (GNU bug Tracking System)
Date: Sun, 16 Apr 2017 09:21:02 +0000

Your bug report

#26515: [PATCH] gnu: Add emacs-calfw.

which was filed against the guix-patches package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
26515: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=26515
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#26515: Acknowledgement ([PATCH] gnu: Add emacs-calfw.)
From: Alex Kost <[email-masked]>
To: Vasile Dumitrascu <[email-masked]>
Cc: [email-masked]
Date: Sun, 16 Apr 2017 12:20:52 +0300
List-Id: <debbugs-submit.debbugs.gnu.org>

Vasile Dumitrascu (2017-04-15 14:37 +0000) wrote:

>    Hi
>
> I have added a small correction on the homepage for calfw, as I forgot
> to update the Url from the code stub I have used.

Applied¹, thanks!  Next time, if you have corrections to your original
patch, please send an updated patch instead of the new patch build on
top of the original one.

Note that "guix lint emacs-calfw" reports:

  gnu/packages/emacs.scm:1200:5: [email-masked]: the source file name should contain the package name

This means the source origin should contain:

  (file-name (string-append name "-" version ".tar.gz"))

Otherwise, the downloaded tarball will have the following name in the
store (this is often the case with github tagged releases):

   /gnu/store/...-v1.5.tar.gz

while we prefer

   /gnu/store/...-emacs-calfw-1.5.tar.gz

Finally, I see that for some additional features, this package require
'howm' and 'google-maps' emacs packages, so these features won't work;
but you probably know about it.

¹ http://git.savannah.gnu.org/cgit/guix.git/commit/?id=36890436710032d6c731e31becf6a5bd7aa4afaf

-- 
Alex

Subject: [PATCH] gnu: Add emacs-calfw.
From: Vasile Dumitrascu <[email-masked]>
To: [email-masked]
Cc: Vasile Dumitrascu <[email-masked]>
Date: Sat, 15 Apr 2017 13:24:52 +0200
List-Id: <debbugs-submit.debbugs.gnu.org>

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

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 976ea31e0..aa7d21dbc 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -1207,6 +1207,26 @@ writing input files for TeX, LaTeX, ConTeXt, Texinfo, and docTeX using Emacs
 or XEmacs.")
     (license license:gpl3+)))
 
+(define-public emacs-calfw
+  (package
+    (name "emacs-calfw")
+    (version "1.5")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append
+             "https://github.com/kiwanami/emacs-calfw/archive/v"
+             version ".tar.gz"))
+       (sha256
+        (base32
+         "17ssg8gx66yp63nhygjq2r6kgl4h45cacmrxsxs9f0lrfcx37k0l"))))
+    (build-system emacs-build-system)
+    (home-page "https://elpa.gnu.org/packages/aggressive-indent.html")
+    (synopsis "Calendar framework for Emacs")
+    (description
+     "This program displays a calendar view in the Emacs buffer.")
+    (license license:gpl3+)))
+
 (define-public emacs-mmm-mode
   (package
     (name "emacs-mmm-mode")
-- 
2.11.0