unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH] gnu: Add geiser-next.
@ 2016-04-29 23:34 Christopher Allan Webber
  2016-04-29 23:35 ` Christopher Allan Webber
  0 siblings, 1 reply; 5+ messages in thread
From: Christopher Allan Webber @ 2016-04-29 23:34 UTC (permalink / raw)
  To: guix-devel

If you want to hack trying out guile-next, you can't use the geiser
package.  Fortunately, the git version of geiser supports guile-next /
2.2 / (2.1?) so, here's geiser-next!

Not sure if other people care enough to have a geiser-next package, but
since we're Guile based here, I figured other people might want to be
hacking trying things out with the latest and greatest.

 - Chris

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

* Re: [PATCH] gnu: Add geiser-next.
  2016-04-29 23:34 [PATCH] gnu: Add geiser-next Christopher Allan Webber
@ 2016-04-29 23:35 ` Christopher Allan Webber
  2016-05-01 20:36   ` Leo Famulari
  0 siblings, 1 reply; 5+ messages in thread
From: Christopher Allan Webber @ 2016-04-29 23:35 UTC (permalink / raw)
  To: guix-devel

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

Christopher Allan Webber writes:

> If you want to hack trying out guile-next, you can't use the geiser
> package.  Fortunately, the git version of geiser supports guile-next /
> 2.2 / (2.1?) so, here's geiser-next!
>
> Not sure if other people care enough to have a geiser-next package, but
> since we're Guile based here, I figured other people might want to be
> hacking trying things out with the latest and greatest.
>
>  - Chris

Annnnd guess who forgot the patch!


[-- Attachment #2: 0001-gnu-Add-geiser-next.patch --]
[-- Type: text/x-patch, Size: 2243 bytes --]

From bbc1f68204318d79ece3fa46ed71d7c21d6ec8a2 Mon Sep 17 00:00:00 2001
From: Christopher Allan Webber <cwebber@dustycloud.org>
Date: Fri, 29 Apr 2016 17:53:50 -0500
Subject: [PATCH] gnu: Add geiser-next.

Also, I previously committed to this repository in 2015; add self to copyright
headers for that year too.

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

diff --git a/gnu/packages/emacs.scm b/gnu/packages/emacs.scm
index 11010b2..7f904c0 100644
--- a/gnu/packages/emacs.scm
+++ b/gnu/packages/emacs.scm
@@ -7,6 +7,7 @@
 ;;; Copyright © 2015, 2016 Ricardo Wurmus <rekado@elephly.net>
 ;;; Copyright © 2016 Nils Gillmann <niasterisk@grrlz.net>
 ;;; Copyright © 2016 Chris Marusich <cmmarusich@gmail.com>
+;;; Copyright © 2015, 2016 Christopher Allan Webber <cwebber@dustycloud.org>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -260,6 +261,33 @@ implementation, Emacs and, ultimately, the schemer, giving them access to live
 metadata.")
     (license license:bsd-3)))
 
+(define-public geiser-next
+  ;; Geiser's upcoming version supports guile-next, and 0.8.1 does not.
+  ;; When the next Geiser release comes out, we can remove this.
+  (package (inherit geiser)
+    (name "geiser-next")
+    (version "20160428.4c8b3de")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/jaor/geiser.git")
+                    (commit "4c8b3dee30ce7258602bef356337def95fcccd7e")))
+              (sha256
+               (base32
+                "1nr183438z52v62amq38ngd83nwgkxzgssya19f7v6f47im0b95j"))))
+    (native-inputs
+     `(("autoconf" ,autoconf)
+       ("automake" ,automake)
+       ("texinfo" ,texinfo)
+       ,@(package-native-inputs geiser)))
+    (arguments
+     (substitute-keyword-arguments (package-arguments geiser)
+       ((#:phases phases)
+        `(modify-phases ,phases
+           (add-after 'unpack 'autogen
+             (lambda _
+               (zero? (system* "sh" "autogen.sh"))))))))))
+
 (define-public paredit
   (package
     (name "paredit")
-- 
2.7.4


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

* Re: [PATCH] gnu: Add geiser-next.
  2016-04-29 23:35 ` Christopher Allan Webber
@ 2016-05-01 20:36   ` Leo Famulari
  2016-05-02  7:53     ` Ludovic Courtès
  0 siblings, 1 reply; 5+ messages in thread
From: Leo Famulari @ 2016-05-01 20:36 UTC (permalink / raw)
  To: Christopher Allan Webber; +Cc: guix-devel

On Fri, Apr 29, 2016 at 06:35:43PM -0500, Christopher Allan Webber wrote:
> * gnu/packages/emacs.scm (geiser-next): New variable.

> +    (version "20160428.4c8b3de")

Looks good, but can you make the version string in the style described
in 7.6.3 Version Numbers?

https://www.gnu.org/software/guix/manual/guix.html#Version-Numbers

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

* Re: [PATCH] gnu: Add geiser-next.
  2016-05-01 20:36   ` Leo Famulari
@ 2016-05-02  7:53     ` Ludovic Courtès
  2016-05-16 20:05       ` Christopher Allan Webber
  0 siblings, 1 reply; 5+ messages in thread
From: Ludovic Courtès @ 2016-05-02  7:53 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

Leo Famulari <leo@famulari.name> skribis:

> On Fri, Apr 29, 2016 at 06:35:43PM -0500, Christopher Allan Webber wrote:
>> * gnu/packages/emacs.scm (geiser-next): New variable.
>
>> +    (version "20160428.4c8b3de")
>
> Looks good, but can you make the version string in the style described
> in 7.6.3 Version Numbers?
>
> https://www.gnu.org/software/guix/manual/guix.html#Version-Numbers

With this change, LGTM!

We should make sure to remove it when the next Geiser comes out.

Thanks,
Ludo’.

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

* Re: [PATCH] gnu: Add geiser-next.
  2016-05-02  7:53     ` Ludovic Courtès
@ 2016-05-16 20:05       ` Christopher Allan Webber
  0 siblings, 0 replies; 5+ messages in thread
From: Christopher Allan Webber @ 2016-05-16 20:05 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

Ludovic Courtès writes:

> Leo Famulari <leo@famulari.name> skribis:
>
>> On Fri, Apr 29, 2016 at 06:35:43PM -0500, Christopher Allan Webber wrote:
>>> * gnu/packages/emacs.scm (geiser-next): New variable.
>>
>>> +    (version "20160428.4c8b3de")
>>
>> Looks good, but can you make the version string in the style described
>> in 7.6.3 Version Numbers?
>>
>> https://www.gnu.org/software/guix/manual/guix.html#Version-Numbers
>
> With this change, LGTM!
>
> We should make sure to remove it when the next Geiser comes out.
>
> Thanks,
> Ludo’.

Done.  Sorry it took so long!  Life got in the way! :)
 - Chris

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

end of thread, other threads:[~2016-05-16 20:05 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-04-29 23:34 [PATCH] gnu: Add geiser-next Christopher Allan Webber
2016-04-29 23:35 ` Christopher Allan Webber
2016-05-01 20:36   ` Leo Famulari
2016-05-02  7:53     ` Ludovic Courtès
2016-05-16 20:05       ` Christopher Allan Webber

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).