all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [PATCH] gnu: Add guile-dsv.
@ 2017-11-24 11:22 Alex Sassmannshausen
  2017-11-24 13:53 ` Ludovic Courtès
  0 siblings, 1 reply; 3+ messages in thread
From: Alex Sassmannshausen @ 2017-11-24 11:22 UTC (permalink / raw)
  To: guix-devel; +Cc: Alex Sassmannshausen

* gnu/packages/guile.scm (guile-dsv): New variable.
---
 gnu/packages/guile.scm | 51 ++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 51 insertions(+)

diff --git a/gnu/packages/guile.scm b/gnu/packages/guile.scm
index abcefd32e..b94ffa754 100644
--- a/gnu/packages/guile.scm
+++ b/gnu/packages/guile.scm
@@ -1492,6 +1492,57 @@ It currently supports MySQL, Postgres and SQLite3.")
 SQL databases.  This package implements the interface for SQLite.")
     (license license:gpl2+)))
 
+(define-public guile-dsv
+  (package
+    (name "guile-dsv")
+    (version "0.2.0")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/artyom-poptsov/guile-dsv")
+                    (commit "7d2e06a15e1d8478cd0e8fb4c79aec519dc4cfd0")))
+              (file-name (string-append name "-" version "-checkout"))
+              (sha256
+               (base32
+                "0ywb0hdbs4lcjag8b3id43fpyn5s6gscg7dk0n9ryigyvch80wxj"))))
+    (build-system gnu-build-system)
+    (native-inputs
+     `(("autoconf" ,autoconf)
+       ("automake" ,automake)
+       ("pkg-config" ,pkg-config)
+       ("texinfo" ,texinfo)))
+    (inputs `(("guile" ,guile-2.2)))
+    (propagated-inputs `(("guile-lib" ,guile2.2-lib)))
+    (arguments
+     '(#:phases (modify-phases %standard-phases
+                  (add-before 'configure 'set-guilesitedir
+                    (lambda _
+                      (substitute* "Makefile.in"
+                        (("^guilesitedir =.*$")
+                         "guilesitedir = \
+$(datadir)/guile/site/$(GUILE_EFFECTIVE_VERSION)\n"))
+                      (substitute* "modules/Makefile.in"
+                        (("^guilesitedir =.*$")
+                         "guilesitedir = \
+$(datadir)/guile/site/$(GUILE_EFFECTIVE_VERSION)\n"))
+                      (substitute* "modules/dsv/Makefile.in"
+                        (("^guilesitedir =.*$")
+                         "guilesitedir = \
+$(datadir)/guile/site/$(GUILE_EFFECTIVE_VERSION)\n"))
+                      #t))
+                  (add-after 'unpack 'autoreconf
+                    (lambda _
+                      (zero? (system* "autoreconf" "-vfi")))))))
+    (home-page "https://github.com/artyom-poptsov/guile-dsv")
+    (synopsis "DSV module for Guile")
+    (description
+     "Guile-DSV is a GNU Guile module for working with the
+delimiter-separated values (DSV) data format.
+
+Guile-DSV supports the Unix-style DSV format and RFC 4180 format.
+")
+    (license license:gpl3)))
+
 (define-public guile-xosd
   (package
     (name "guile-xosd")
-- 
2.15.0

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

* Re: [PATCH] gnu: Add guile-dsv.
  2017-11-24 11:22 [PATCH] gnu: Add guile-dsv Alex Sassmannshausen
@ 2017-11-24 13:53 ` Ludovic Courtès
  2017-11-24 23:28   ` Alex Sassmannshausen
  0 siblings, 1 reply; 3+ messages in thread
From: Ludovic Courtès @ 2017-11-24 13:53 UTC (permalink / raw)
  To: Alex Sassmannshausen; +Cc: guix-devel, Alex Sassmannshausen

Hi Alex,

Alex Sassmannshausen <alex.sassmannshausen@gmail.com> skribis:

> * gnu/packages/guile.scm (guile-dsv): New variable.

[...]

> +    (inputs `(("guile" ,guile-2.2)))
> +    (propagated-inputs `(("guile-lib" ,guile2.2-lib)))

Should be ‘guile-lib’ instead of ‘guile2.2-lib’ (the latter is a
deprecated alias.)

> +    (synopsis "DSV module for Guile")
> +    (description
> +     "Guile-DSV is a GNU Guile module for working with the
> +delimiter-separated values (DSV) data format.
> +
> +Guile-DSV supports the Unix-style DSV format and RFC 4180 format.
> +")

I think you can remove the extra newlines.

> +    (license license:gpl3)))

‘gpl3+’ I guess?

OK with these changes, thank you!

Ludo’.

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

* Re: [PATCH] gnu: Add guile-dsv.
  2017-11-24 13:53 ` Ludovic Courtès
@ 2017-11-24 23:28   ` Alex Sassmannshausen
  0 siblings, 0 replies; 3+ messages in thread
From: Alex Sassmannshausen @ 2017-11-24 23:28 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

Hi Ludo,

Thanks for the review, I've pushed it with the changes you proposed.

Cheers,

Alex

Ludovic Courtès writes:

> Hi Alex,
>
> Alex Sassmannshausen <alex.sassmannshausen@gmail.com> skribis:
>
>> * gnu/packages/guile.scm (guile-dsv): New variable.
>
> [...]
>
>> +    (inputs `(("guile" ,guile-2.2)))
>> +    (propagated-inputs `(("guile-lib" ,guile2.2-lib)))
>
> Should be ‘guile-lib’ instead of ‘guile2.2-lib’ (the latter is a
> deprecated alias.)
>
>> +    (synopsis "DSV module for Guile")
>> +    (description
>> +     "Guile-DSV is a GNU Guile module for working with the
>> +delimiter-separated values (DSV) data format.
>> +
>> +Guile-DSV supports the Unix-style DSV format and RFC 4180 format.
>> +")
>
> I think you can remove the extra newlines.
>
>> +    (license license:gpl3)))
>
> ‘gpl3+’ I guess?
>
> OK with these changes, thank you!
>
> Ludo’.

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

end of thread, other threads:[~2017-11-24 23:28 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-11-24 11:22 [PATCH] gnu: Add guile-dsv Alex Sassmannshausen
2017-11-24 13:53 ` Ludovic Courtès
2017-11-24 23:28   ` Alex Sassmannshausen

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.