* [bug#28583] [PATCH 2/7] gnu: Add ruby-multi-json.
2017-09-24 17:17 ` [bug#28583] [PATCH 1/7] gnu: Add ruby-httpclient Christopher Baines
@ 2017-09-24 17:17 ` Christopher Baines
2017-09-27 12:03 ` Ben Woodcroft
2017-09-24 17:17 ` [bug#28583] [PATCH 3/7] gnu: Add ruby-options Christopher Baines
` (5 subsequent siblings)
6 siblings, 1 reply; 31+ messages in thread
From: Christopher Baines @ 2017-09-24 17:17 UTC (permalink / raw)
To: 28583
* gnu/packages/ruby.scm (ruby-multi-json): New variable.
---
gnu/packages/ruby.scm | 24 ++++++++++++++++++++++++
1 file changed, 24 insertions(+)
diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm
index ff6be875f..116db8c62 100644
--- a/gnu/packages/ruby.scm
+++ b/gnu/packages/ruby.scm
@@ -1865,6 +1865,30 @@ net/http library.")
(home-page "https://github.com/nicksieger/multipart-post")
(license license:expat)))
+(define-public ruby-multi-json
+ (package
+ (name "ruby-multi-json")
+ (version "1.12.2")
+ (source
+ (origin
+ (method url-fetch)
+ (uri (rubygems-uri "multi_json" version))
+ (sha256
+ (base32
+ "1raim9ddjh672m32psaa9niw67ywzjbxbdb8iijx3wv9k5b0pk2x"))))
+ (build-system ruby-build-system)
+ (arguments
+ '(#:tests? #f)) ;; No testsuite.
+ (synopsis
+ "Common interface to multiple JSON libraries for Ruby")
+ (description
+ "This package provides a common interface to multiple JSON libraries,
+including Oj, Yajl, the JSON gem (with C-extensions), the pure-Ruby JSON gem,
+NSJSONSerialization, gson.rb, JrJackson, and OkJson.")
+ (home-page
+ "http://github.com/intridea/multi_json")
+ (license license:expat)))
+
(define-public ruby-arel
(package
(name "ruby-arel")
--
2.14.1
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [bug#28583] [PATCH 2/7] gnu: Add ruby-multi-json.
2017-09-24 17:17 ` [bug#28583] [PATCH 2/7] gnu: Add ruby-multi-json Christopher Baines
@ 2017-09-27 12:03 ` Ben Woodcroft
2017-09-30 8:21 ` Christopher Baines
0 siblings, 1 reply; 31+ messages in thread
From: Ben Woodcroft @ 2017-09-27 12:03 UTC (permalink / raw)
To: Christopher Baines, 28583
On 25/09/17 03:17, Christopher Baines wrote:
> * gnu/packages/ruby.scm (ruby-multi-json): New variable.
[..]
> + (arguments
> + '(#:tests? #f)) ;; No testsuite.
No testsuite included in distributed gem, rather.
> + (synopsis
> + "Common interface to multiple JSON libraries for Ruby")
> + (description
> + "This package provides a common interface to multiple JSON libraries,
> +including Oj, Yajl, the JSON gem (with C-extensions), the pure-Ruby JSON gem,
> +NSJSONSerialization, gson.rb, JrJackson, and OkJson.")
> + (home-page
> + "http://github.com/intridea/multi_json")
Both home-page and synopsis could be moved to the line before.
Otherwise LGTM.
ben
^ permalink raw reply [flat|nested] 31+ messages in thread
* [bug#28583] [PATCH 2/7] gnu: Add ruby-multi-json.
2017-09-27 12:03 ` Ben Woodcroft
@ 2017-09-30 8:21 ` Christopher Baines
0 siblings, 0 replies; 31+ messages in thread
From: Christopher Baines @ 2017-09-30 8:21 UTC (permalink / raw)
To: Ben Woodcroft; +Cc: 28583
[-- Attachment #1: Type: text/plain, Size: 903 bytes --]
On Wed, 27 Sep 2017 22:03:32 +1000
Ben Woodcroft <b.woodcroft@uq.edu.au> wrote:
> On 25/09/17 03:17, Christopher Baines wrote:
> > * gnu/packages/ruby.scm (ruby-multi-json): New variable.
> [..]
> > + (arguments
> > + '(#:tests? #f)) ;; No testsuite.
> No testsuite included in distributed gem, rather.
I've updated the comment.
> > + (synopsis
> > + "Common interface to multiple JSON libraries for Ruby")
> > + (description
> > + "This package provides a common interface to multiple JSON
> > libraries, +including Oj, Yajl, the JSON gem (with C-extensions),
> > the pure-Ruby JSON gem, +NSJSONSerialization, gson.rb, JrJackson,
> > and OkJson.")
> > + (home-page
> > + "http://github.com/intridea/multi_json")
> Both home-page and synopsis could be moved to the line before.
And also updated the formatting.
> Otherwise LGTM.
> ben
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 963 bytes --]
^ permalink raw reply [flat|nested] 31+ messages in thread
* [bug#28583] [PATCH 3/7] gnu: Add ruby-options.
2017-09-24 17:17 ` [bug#28583] [PATCH 1/7] gnu: Add ruby-httpclient Christopher Baines
2017-09-24 17:17 ` [bug#28583] [PATCH 2/7] gnu: Add ruby-multi-json Christopher Baines
@ 2017-09-24 17:17 ` Christopher Baines
2017-09-27 12:10 ` Ben Woodcroft
2017-09-24 17:17 ` [bug#28583] [PATCH 4/7] gnu: Add ruby-highline Christopher Baines
` (4 subsequent siblings)
6 siblings, 1 reply; 31+ messages in thread
From: Christopher Baines @ 2017-09-24 17:17 UTC (permalink / raw)
To: 28583
* gnu/packages/ruby.scm (ruby-options): New variable.
---
gnu/packages/ruby.scm | 28 ++++++++++++++++++++++++++++
1 file changed, 28 insertions(+)
diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm
index 116db8c62..1ee03e066 100644
--- a/gnu/packages/ruby.scm
+++ b/gnu/packages/ruby.scm
@@ -660,6 +660,34 @@ complexity.")
(home-page "https://github.com/ThoughtWorksStudios/saikuro_treemap")
(license license:expat)))
+(define-public ruby-options
+ (package
+ (name "ruby-options")
+ (version "2.3.2")
+ (source
+ (origin
+ (method url-fetch)
+ (uri (rubygems-uri "options" version))
+ (sha256
+ (base32
+ "1s650nwnabx66w584m1cyw82icyym6hv5kzfsbp38cinkr5klh9j"))))
+ (build-system ruby-build-system)
+ (arguments
+ '(#:tests? #f ;; TODO: NameError: uninitialized constant Config
+ #:phases
+ (modify-phases %standard-phases
+ (add-before 'check 'set-LIB
+ (lambda _
+ ;; This is used in the Rakefile, and setting it avoids an issue
+ ;; with running the tests.
+ (setenv "LIB" "options"))))))
+ (synopsis "Ruby library to parse options from *args cleanly")
+ (description
+ "The @code{options} library helps with parsing keyword options in Ruby
+functions.")
+ (home-page "https://github.com/ahoward/options")
+ (license license:ruby)))
+
(define-public ruby-orderedhash
(package
(name "ruby-orderedhash")
--
2.14.1
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [bug#28583] [PATCH 4/7] gnu: Add ruby-highline.
2017-09-24 17:17 ` [bug#28583] [PATCH 1/7] gnu: Add ruby-httpclient Christopher Baines
2017-09-24 17:17 ` [bug#28583] [PATCH 2/7] gnu: Add ruby-multi-json Christopher Baines
2017-09-24 17:17 ` [bug#28583] [PATCH 3/7] gnu: Add ruby-options Christopher Baines
@ 2017-09-24 17:17 ` Christopher Baines
2017-09-27 12:37 ` Ben Woodcroft
2017-09-24 17:17 ` [bug#28583] [PATCH 5/7] gnu: Add ruby-progress_bar Christopher Baines
` (3 subsequent siblings)
6 siblings, 1 reply; 31+ messages in thread
From: Christopher Baines @ 2017-09-24 17:17 UTC (permalink / raw)
To: 28583
* gnu/packages/ruby.scm (ruby-highline): New variable.
---
gnu/packages/ruby.scm | 27 +++++++++++++++++++++++++++
1 file changed, 27 insertions(+)
diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm
index 1ee03e066..9863d4b37 100644
--- a/gnu/packages/ruby.scm
+++ b/gnu/packages/ruby.scm
@@ -216,6 +216,33 @@ a focus on simplicity and productivity.")
(("/bin/sh") (which "sh"))))
%standard-phases)))))
+(define-public ruby-highline
+ (package
+ (name "ruby-highline")
+ (version "1.7.8")
+ (source
+ (origin
+ (method url-fetch)
+ (uri (rubygems-uri "highline" version))
+ (sha256
+ (base32
+ "1nf5lgdn6ni2lpfdn4gk3gi47fmnca2bdirabbjbz1fk9w4p8lkr"))))
+ (build-system ruby-build-system)
+ (arguments
+ `(#:tests? #f)) ;; TODO: Requires code_statistics
+ (native-inputs
+ `(("bundler" ,bundler)))
+ (synopsis
+ "HighLine helps you build command-line interfaces")
+ (description
+ "HighLine provides a high-level IO library that provides validation,
+type conversion, and more for command-line interfaces.
+
+HighLine also includes a menu system for providing multiple options to the
+user.")
+ (home-page "https://github.com/JEG2/highline")
+ (license (list license:gpl2 license:ruby))))
+
(define-public ruby-hoe
(package
(name "ruby-hoe")
--
2.14.1
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [bug#28583] [PATCH 4/7] gnu: Add ruby-highline.
2017-09-24 17:17 ` [bug#28583] [PATCH 4/7] gnu: Add ruby-highline Christopher Baines
@ 2017-09-27 12:37 ` Ben Woodcroft
2017-09-30 8:23 ` Christopher Baines
0 siblings, 1 reply; 31+ messages in thread
From: Ben Woodcroft @ 2017-09-27 12:37 UTC (permalink / raw)
To: Christopher Baines, 28583
[-- Attachment #1: Type: text/plain, Size: 1640 bytes --]
On 25/09/17 03:17, Christopher Baines wrote:
> * gnu/packages/ruby.scm (ruby-highline): New variable.
> ---
> gnu/packages/ruby.scm | 27 +++++++++++++++++++++++++++
> 1 file changed, 27 insertions(+)
>
> diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm
> index 1ee03e066..9863d4b37 100644
> --- a/gnu/packages/ruby.scm
> +++ b/gnu/packages/ruby.scm
> @@ -216,6 +216,33 @@ a focus on simplicity and productivity.")
> (("/bin/sh") (which "sh"))))
> %standard-phases)))))
>
> +(define-public ruby-highline
> + (package
> + (name "ruby-highline")
> + (version "1.7.8")
> + (source
> + (origin
> + (method url-fetch)
> + (uri (rubygems-uri "highline" version))
> + (sha256
> + (base32
> + "1nf5lgdn6ni2lpfdn4gk3gi47fmnca2bdirabbjbz1fk9w4p8lkr"))))
> + (build-system ruby-build-system)
> + (arguments
> + `(#:tests? #f)) ;; TODO: Requires code_statistics
I've attached a package for code_statistics, but adding it does not help
because there appears to be something amiss in the Rakefile. This has
been fixed in the not quite released 2.0.0 version, so the patch may be
useful in future.
> + (native-inputs
> + `(("bundler" ,bundler)))
> + (synopsis
> + "HighLine helps you build command-line interfaces")
> + (description
> + "HighLine provides a high-level IO library that provides validation,
> +type conversion, and more for command-line interfaces.
> +
> +HighLine also includes a menu system for providing multiple options to the
> +user.")
I would combine the two sentences into one paragraph.
Otherwise LGTM.
[-- Attachment #2: 0001-gnu-Add-ruby-code-statistics.patch --]
[-- Type: text/x-patch, Size: 1493 bytes --]
From e2fb88306d527db0bbc53e980a071474d3b67100 Mon Sep 17 00:00:00 2001
From: Ben Woodcroft <donttrustben@gmail.com>
Date: Wed, 27 Sep 2017 22:18:33 +1000
Subject: [PATCH] gnu: Add ruby-code-statistics.
* gnu/packages/ruby.scm (ruby-code-statistics): New variable.
---
gnu/packages/ruby.scm | 21 +++++++++++++++++++++
1 file changed, 21 insertions(+)
diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm
index 1ee03e066..c41c80569 100644
--- a/gnu/packages/ruby.scm
+++ b/gnu/packages/ruby.scm
@@ -4378,3 +4378,24 @@ really opens up the email messages you are parsing, if you know what you
are doing, you can fiddle with every last bit of your email directly.")
(home-page "https://github.com/mikel/mail")
(license license:expat)))
+
+(define-public ruby-code-statistics
+ (package
+ (name "ruby-code-statistics")
+ (version "0.2.13")
+ (source
+ (origin
+ (method url-fetch)
+ (uri (rubygems-uri "code_statistics" version))
+ (sha256
+ (base32
+ "07rdpsbwbmh4vp8nxyh308cj7am2pbrfhv9v5xr2d5gq8hnnsm93"))))
+ (build-system ruby-build-system)
+ (arguments
+ `(#:tests? #f)) ; Not all test code is included in gem.
+ (synopsis "Port of the rails 'rake stats' method")
+ (description
+ "This gem is a port of the rails 'rake stats' method so it can be made
+more robust and work for non rails projects.")
+ (home-page "http://github.com/danmayer/code_statistics")
+ (license license:expat)))
--
2.14.1
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [bug#28583] [PATCH 4/7] gnu: Add ruby-highline.
2017-09-27 12:37 ` Ben Woodcroft
@ 2017-09-30 8:23 ` Christopher Baines
0 siblings, 0 replies; 31+ messages in thread
From: Christopher Baines @ 2017-09-30 8:23 UTC (permalink / raw)
To: Ben Woodcroft; +Cc: 28583
[-- Attachment #1: Type: text/plain, Size: 2018 bytes --]
On Wed, 27 Sep 2017 22:37:39 +1000
Ben Woodcroft <b.woodcroft@uq.edu.au> wrote:
> On 25/09/17 03:17, Christopher Baines wrote:
> > * gnu/packages/ruby.scm (ruby-highline): New variable.
> > ---
> > gnu/packages/ruby.scm | 27 +++++++++++++++++++++++++++
> > 1 file changed, 27 insertions(+)
> >
> > diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm
> > index 1ee03e066..9863d4b37 100644
> > --- a/gnu/packages/ruby.scm
> > +++ b/gnu/packages/ruby.scm
> > @@ -216,6 +216,33 @@ a focus on simplicity and productivity.")
> > (("/bin/sh") (which "sh"))))
> > %standard-phases)))))
> >
> > +(define-public ruby-highline
> > + (package
> > + (name "ruby-highline")
> > + (version "1.7.8")
> > + (source
> > + (origin
> > + (method url-fetch)
> > + (uri (rubygems-uri "highline" version))
> > + (sha256
> > + (base32
> > + "1nf5lgdn6ni2lpfdn4gk3gi47fmnca2bdirabbjbz1fk9w4p8lkr"))))
> > + (build-system ruby-build-system)
> > + (arguments
> > + `(#:tests? #f)) ;; TODO: Requires code_statistics
> I've attached a package for code_statistics, but adding it does not
> help because there appears to be something amiss in the Rakefile.
> This has been fixed in the not quite released 2.0.0 version, so the
> patch may be useful in future.
Awesome, thanks Ben. I've send an updated set of patches, including this
one now. It looks good to me.
> > + (native-inputs
> > + `(("bundler" ,bundler)))
> > + (synopsis
> > + "HighLine helps you build command-line interfaces")
> > + (description
> > + "HighLine provides a high-level IO library that provides
> > validation, +type conversion, and more for command-line interfaces.
> > +
> > +HighLine also includes a menu system for providing multiple
> > options to the +user.")
> I would combine the two sentences into one paragraph.
I've made this change in the updated patches.
> Otherwise LGTM.
>
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 963 bytes --]
^ permalink raw reply [flat|nested] 31+ messages in thread
* [bug#28583] [PATCH 5/7] gnu: Add ruby-progress_bar.
2017-09-24 17:17 ` [bug#28583] [PATCH 1/7] gnu: Add ruby-httpclient Christopher Baines
` (2 preceding siblings ...)
2017-09-24 17:17 ` [bug#28583] [PATCH 4/7] gnu: Add ruby-highline Christopher Baines
@ 2017-09-24 17:17 ` Christopher Baines
2017-09-27 12:43 ` Ben Woodcroft
2017-09-24 17:17 ` [bug#28583] [PATCH 6/7] gnu: Add ruby-rubyzip Christopher Baines
` (2 subsequent siblings)
6 siblings, 1 reply; 31+ messages in thread
From: Christopher Baines @ 2017-09-24 17:17 UTC (permalink / raw)
To: 28583
* gnu/packages/ruby.scm (ruby-progress_bar): New variable.
---
gnu/packages/ruby.scm | 31 +++++++++++++++++++++++++++++++
1 file changed, 31 insertions(+)
diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm
index 9863d4b37..5dde022c6 100644
--- a/gnu/packages/ruby.scm
+++ b/gnu/packages/ruby.scm
@@ -2114,6 +2114,37 @@ for select languages.")
(home-page "http://coderay.rubychan.de")
(license license:expat)))
+(define-public ruby-progress_bar
+ (package
+ (name "ruby-progress_bar")
+ (version "1.1.0")
+ (source
+ (origin
+ (method url-fetch)
+ (uri (rubygems-uri "progress_bar" version))
+ (sha256
+ (base32
+ "1qc40mr6p1z9a3vlpnsg1zfgk1qswviql2a31y63wpv3vr6b5f48"))))
+ (build-system ruby-build-system)
+ (arguments
+ '(#:test-target "spec"))
+ (propagated-inputs
+ `(("ruby-highline" ,ruby-highline)
+ ("ruby-options" ,ruby-options)))
+ (native-inputs
+ `(("bundler" ,bundler)
+ ("ruby-rspec" ,ruby-rspec)
+ ("ruby-timecop" ,ruby-timecop)))
+ (synopsis
+ "Ruby library for displaying progress bars")
+ (description
+ "ProgressBar is a simple library for displaying progress bars. The
+maximum value is configurable, and additional information can be displayed
+like the percentage completion, estimated time remaining, elapsed time and
+rate.")
+ (home-page "https://github.com/paul/progress_bar")
+ (license license:wtfpl2)))
+
(define-public ruby-pry
(package
(name "ruby-pry")
--
2.14.1
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [bug#28583] [PATCH 5/7] gnu: Add ruby-progress_bar.
2017-09-24 17:17 ` [bug#28583] [PATCH 5/7] gnu: Add ruby-progress_bar Christopher Baines
@ 2017-09-27 12:43 ` Ben Woodcroft
2017-09-30 8:23 ` Christopher Baines
0 siblings, 1 reply; 31+ messages in thread
From: Ben Woodcroft @ 2017-09-27 12:43 UTC (permalink / raw)
To: Christopher Baines, 28583
On 25/09/17 03:17, Christopher Baines wrote:
> * gnu/packages/ruby.scm (ruby-progress_bar): New variable.
> ---
> gnu/packages/ruby.scm | 31 +++++++++++++++++++++++++++++++
> 1 file changed, 31 insertions(+)
>
> diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm
> index 9863d4b37..5dde022c6 100644
> --- a/gnu/packages/ruby.scm
> +++ b/gnu/packages/ruby.scm
> @@ -2114,6 +2114,37 @@ for select languages.")
> (home-page "http://coderay.rubychan.de")
> (license license:expat)))
>
> +(define-public ruby-progress_bar
> + (package
> + (name "ruby-progress_bar")
OK. Unfortunate that there is also a progress-bar gem.
LGTM.
^ permalink raw reply [flat|nested] 31+ messages in thread
* [bug#28583] [PATCH 5/7] gnu: Add ruby-progress_bar.
2017-09-27 12:43 ` Ben Woodcroft
@ 2017-09-30 8:23 ` Christopher Baines
0 siblings, 0 replies; 31+ messages in thread
From: Christopher Baines @ 2017-09-30 8:23 UTC (permalink / raw)
To: Ben Woodcroft; +Cc: 28583
[-- Attachment #1: Type: text/plain, Size: 896 bytes --]
On Wed, 27 Sep 2017 22:43:31 +1000
Ben Woodcroft <b.woodcroft@uq.edu.au> wrote:
> On 25/09/17 03:17, Christopher Baines wrote:
> > * gnu/packages/ruby.scm (ruby-progress_bar): New variable.
> > ---
> > gnu/packages/ruby.scm | 31 +++++++++++++++++++++++++++++++
> > 1 file changed, 31 insertions(+)
> >
> > diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm
> > index 9863d4b37..5dde022c6 100644
> > --- a/gnu/packages/ruby.scm
> > +++ b/gnu/packages/ruby.scm
> > @@ -2114,6 +2114,37 @@ for select languages.")
> > (home-page "http://coderay.rubychan.de")
> > (license license:expat)))
> >
> > +(define-public ruby-progress_bar
> > + (package
> > + (name "ruby-progress_bar")
> OK. Unfortunate that there is also a progress-bar gem.
Indeed, I packaged the wrong one at first, and was wondering why things
were not working...
> LGTM.
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 963 bytes --]
^ permalink raw reply [flat|nested] 31+ messages in thread
* [bug#28583] [PATCH 6/7] gnu: Add ruby-rubyzip.
2017-09-24 17:17 ` [bug#28583] [PATCH 1/7] gnu: Add ruby-httpclient Christopher Baines
` (3 preceding siblings ...)
2017-09-24 17:17 ` [bug#28583] [PATCH 5/7] gnu: Add ruby-progress_bar Christopher Baines
@ 2017-09-24 17:17 ` Christopher Baines
2017-09-28 8:34 ` Ben Woodcroft
2017-09-24 17:17 ` [bug#28583] [PATCH 7/7] gnu: Add es-dump-restore Christopher Baines
2017-09-27 11:56 ` [bug#28583] [PATCH 1/7] gnu: Add ruby-httpclient Ben Woodcroft
6 siblings, 1 reply; 31+ messages in thread
From: Christopher Baines @ 2017-09-24 17:17 UTC (permalink / raw)
To: 28583
* gnu/packages/ruby.scm (ruby-rubyzip): New variable.
---
gnu/packages/ruby.scm | 34 ++++++++++++++++++++++++++++++++++
1 file changed, 34 insertions(+)
diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm
index 5dde022c6..6f9589489 100644
--- a/gnu/packages/ruby.scm
+++ b/gnu/packages/ruby.scm
@@ -1002,6 +1002,40 @@ Ruby Gems.")
(home-page "https://github.com/postmodern/rubygems-tasks")
(license license:expat)))
+(define-public ruby-rubyzip
+ (package
+ (name "ruby-rubyzip")
+ (version "1.2.1")
+ (source
+ (origin
+ (method url-fetch)
+ (uri (rubygems-uri "rubyzip" version))
+ (sha256
+ (base32
+ "06js4gznzgh8ac2ldvmjcmg9v1vg9llm357yckkpylaj6z456zqz"))))
+ (build-system ruby-build-system)
+ (arguments
+ '(#:tests? #f ;; TODO: 265 runs, 2636 assertions, 5 failures, 0 errors, 0
+ ;; skips
+ #:phases
+ (modify-phases %standard-phases
+ (add-before 'check 'patch-test/gentestfiles.rb
+ (lambda* (#:key inputs #:allow-other-keys)
+ (substitute* "test/gentestfiles.rb"
+ (("/usr/bin/zip")
+ (string-append
+ (assoc-ref inputs "zip") "/bin/zip"))))))))
+ (native-inputs
+ `(("bundler" ,bundler)
+ ("ruby-simplecov" ,ruby-simplecov)
+ ("zip" ,zip)))
+ (synopsis
+ "The rubyzip module is for reading and writing zip files")
+ (description
+ "The rubyzip module provides ways to read from and create zip files.")
+ (home-page "http://github.com/rubyzip/rubyzip")
+ (license license:bsd-2)))
+
(define-public ruby-ffi
(package
(name "ruby-ffi")
--
2.14.1
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [bug#28583] [PATCH 6/7] gnu: Add ruby-rubyzip.
2017-09-24 17:17 ` [bug#28583] [PATCH 6/7] gnu: Add ruby-rubyzip Christopher Baines
@ 2017-09-28 8:34 ` Ben Woodcroft
2017-09-30 8:24 ` Christopher Baines
0 siblings, 1 reply; 31+ messages in thread
From: Ben Woodcroft @ 2017-09-28 8:34 UTC (permalink / raw)
To: Christopher Baines, 28583
[-- Attachment #1: Type: text/plain, Size: 183 bytes --]
Hi,
On 25/09/17 03:17, Christopher Baines wrote:
> * gnu/packages/ruby.scm (ruby-rubyzip): New variable.
I made a few changes, and got the tests to pass - patch attached. WDYT?
ben
[-- Attachment #2: 0001-gnu-Add-ruby-rubyzip.patch --]
[-- Type: text/x-patch, Size: 1903 bytes --]
From 8d7bcd94b011a326c3c42802589c7e326c5d09c1 Mon Sep 17 00:00:00 2001
From: Christopher Baines <mail@cbaines.net>
Date: Thu, 28 Sep 2017 18:31:03 +1000
Subject: [PATCH] gnu: Add ruby-rubyzip.
* gnu/packages/ruby.scm (ruby-rubyzip): New variable.
Co-authored by Ben Woodcroft <donttrustben@gmail.com>.
---
gnu/packages/ruby.scm | 35 +++++++++++++++++++++++++++++++++++
1 file changed, 35 insertions(+)
diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm
index 5dde022c6..4f2ba6ea7 100644
--- a/gnu/packages/ruby.scm
+++ b/gnu/packages/ruby.scm
@@ -1002,6 +1002,41 @@ Ruby Gems.")
(home-page "https://github.com/postmodern/rubygems-tasks")
(license license:expat)))
+(define-public ruby-rubyzip
+ (package
+ (name "ruby-rubyzip")
+ (version "1.2.1")
+ (source
+ (origin
+ (method url-fetch)
+ (uri (rubygems-uri "rubyzip" version))
+ (sha256
+ (base32
+ "06js4gznzgh8ac2ldvmjcmg9v1vg9llm357yckkpylaj6z456zqz"))))
+ (build-system ruby-build-system)
+ (arguments
+ '(#:phases
+ (modify-phases %standard-phases
+ (add-before 'check 'patch-tests
+ (lambda* (#:key inputs #:allow-other-keys)
+ (substitute* "test/gentestfiles.rb"
+ (("/usr/bin/zip")
+ (string-append
+ (assoc-ref inputs "zip") "/bin/zip")))
+ (substitute* "test/input_stream_test.rb"
+ (("/usr/bin/env ruby") (which "ruby")))
+ #t)))))
+ (native-inputs
+ `(("bundler" ,bundler)
+ ("ruby-simplecov" ,ruby-simplecov)
+ ("zip" ,zip)
+ ("unzip" ,unzip)))
+ (synopsis "Ruby module is for reading and writing zip files")
+ (description
+ "The rubyzip module provides ways to read from and create zip files.")
+ (home-page "http://github.com/rubyzip/rubyzip")
+ (license license:bsd-2)))
+
(define-public ruby-ffi
(package
(name "ruby-ffi")
--
2.14.1
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [bug#28583] [PATCH 7/7] gnu: Add es-dump-restore.
2017-09-24 17:17 ` [bug#28583] [PATCH 1/7] gnu: Add ruby-httpclient Christopher Baines
` (4 preceding siblings ...)
2017-09-24 17:17 ` [bug#28583] [PATCH 6/7] gnu: Add ruby-rubyzip Christopher Baines
@ 2017-09-24 17:17 ` Christopher Baines
2017-09-28 8:38 ` Ben Woodcroft
2017-09-27 11:56 ` [bug#28583] [PATCH 1/7] gnu: Add ruby-httpclient Ben Woodcroft
6 siblings, 1 reply; 31+ messages in thread
From: Christopher Baines @ 2017-09-24 17:17 UTC (permalink / raw)
To: 28583
* gnu/packages/databases.scm (es-dump-restore): New variable.
---
gnu/packages/databases.scm | 43 +++++++++++++++++++++++++++++++++++++++++++
1 file changed, 43 insertions(+)
diff --git a/gnu/packages/databases.scm b/gnu/packages/databases.scm
index 6ce58985e..2f64e61ab 100644
--- a/gnu/packages/databases.scm
+++ b/gnu/packages/databases.scm
@@ -68,6 +68,7 @@
#:use-module (gnu packages python)
#:use-module (gnu packages rdf)
#:use-module (gnu packages readline)
+ #:use-module (gnu packages ruby)
#:use-module (gnu packages tcl)
#:use-module (gnu packages tls)
#:use-module (gnu packages xml)
@@ -77,6 +78,7 @@
#:use-module (guix build-system gnu)
#:use-module (guix build-system perl)
#:use-module (guix build-system python)
+ #:use-module (guix build-system ruby)
#:use-module (guix build-system cmake)
#:use-module (guix utils)
#:use-module (srfi srfi-26)
@@ -256,6 +258,47 @@ SQL, Key/Value, XML/XQuery or Java Object storage for their data model.")
"--enable-cxx"))))
%standard-phases)))))
+(define-public es-dump-restore
+ (package
+ (name "es-dump-restore")
+ (version "2.1.0")
+ (source
+ (origin
+ (method url-fetch)
+ (uri (rubygems-uri "es_dump_restore" version))
+ (sha256
+ (base32
+ "020yk7f1hw48clmf5501z3xv9shsdchyymcv0y2cci2c1xvr1mim"))))
+ (build-system ruby-build-system)
+ (arguments
+ '(#:tests? #f
+ #:phases
+ (modify-phases %standard-phases
+ (add-after 'install 'wrap-bin-es_dump_restore
+ (lambda* (#:key outputs #:allow-other-keys)
+ (wrap-program (string-append (assoc-ref outputs "out")
+ "/bin/es_dump_restore")
+ `("GEM_PATH" ":" prefix (,(string-append
+ (getenv "GEM_PATH")
+ ":"
+ (getenv "GEM_HOME")))))
+ #t)))))
+ (propagated-inputs
+ `(("ruby-httpclient" ,ruby-httpclient)
+ ("ruby-multi-json" ,ruby-multi-json)
+ ("ruby-progress_bar" ,ruby-progress_bar)
+ ("ruby-rubyzip" ,ruby-rubyzip)
+ ("ruby-thor" ,ruby-thor)))
+ (synopsis
+ "Utility for dumping and restoring ElasticSearch indexes")
+ (description
+ "This package provides a utility for dumping the contents of an
+ElasticSearch index to a compressed file and restoring the dumpfile back to an
+ElasticSearch server")
+ (home-page
+ "https://github.com/patientslikeme/es_dump_restore")
+ (license license:expat)))
+
(define-public leveldb
(package
(name "leveldb")
--
2.14.1
^ permalink raw reply related [flat|nested] 31+ messages in thread
* [bug#28583] [PATCH 7/7] gnu: Add es-dump-restore.
2017-09-24 17:17 ` [bug#28583] [PATCH 7/7] gnu: Add es-dump-restore Christopher Baines
@ 2017-09-28 8:38 ` Ben Woodcroft
2017-09-30 8:25 ` Christopher Baines
0 siblings, 1 reply; 31+ messages in thread
From: Ben Woodcroft @ 2017-09-28 8:38 UTC (permalink / raw)
To: Christopher Baines, 28583
On 25/09/17 03:17, Christopher Baines wrote:
> * gnu/packages/databases.scm (es-dump-restore): New variable.
[..]
> + (arguments
> + '(#:tests? #f
Including a reason here would be good.
> + #:phases
> + (modify-phases %standard-phases
> + (add-after 'install 'wrap-bin-es_dump_restore
> + (lambda* (#:key outputs #:allow-other-keys)
> + (wrap-program (string-append (assoc-ref outputs "out")
> + "/bin/es_dump_restore")
> + `("GEM_PATH" ":" prefix (,(string-append
> + (getenv "GEM_PATH")
> + ":"
> + (getenv "GEM_HOME")))))
> + #t)))))
Again, it would be preferable to have this done as part of the build
system, but OK.
> + (propagated-inputs
> + `(("ruby-httpclient" ,ruby-httpclient)
> + ("ruby-multi-json" ,ruby-multi-json)
> + ("ruby-progress_bar" ,ruby-progress_bar)
> + ("ruby-rubyzip" ,ruby-rubyzip)
> + ("ruby-thor" ,ruby-thor)))
> + (synopsis
> + "Utility for dumping and restoring ElasticSearch indexes")
This can be moved onto the previous line.
> + (description
> + "This package provides a utility for dumping the contents of an
> +ElasticSearch index to a compressed file and restoring the dumpfile back to an
> +ElasticSearch server")
> + (home-page
> + "https://github.com/patientslikeme/es_dump_restore")
As can this. Otherwise LGTM - thanks for this series.
ben
^ permalink raw reply [flat|nested] 31+ messages in thread
* [bug#28583] [PATCH 7/7] gnu: Add es-dump-restore.
2017-09-28 8:38 ` Ben Woodcroft
@ 2017-09-30 8:25 ` Christopher Baines
2017-09-30 10:32 ` Ben Woodcroft
0 siblings, 1 reply; 31+ messages in thread
From: Christopher Baines @ 2017-09-30 8:25 UTC (permalink / raw)
To: Ben Woodcroft; +Cc: 28583
[-- Attachment #1: Type: text/plain, Size: 1971 bytes --]
On Thu, 28 Sep 2017 18:38:19 +1000
Ben Woodcroft <b.woodcroft@uq.edu.au> wrote:
> On 25/09/17 03:17, Christopher Baines wrote:
> > * gnu/packages/databases.scm (es-dump-restore): New variable.
> [..]
> > + (arguments
> > + '(#:tests? #f
> Including a reason here would be good.
Done, turns out the reason is that there is no testsuite.
> > + #:phases
> > + (modify-phases %standard-phases
> > + (add-after 'install 'wrap-bin-es_dump_restore
> > + (lambda* (#:key outputs #:allow-other-keys)
> > + (wrap-program (string-append (assoc-ref outputs "out")
> > + "/bin/es_dump_restore")
> > + `("GEM_PATH" ":" prefix (,(string-append
> > + (getenv "GEM_PATH")
> > + ":"
> > + (getenv "GEM_HOME")))))
> > + #t)))))
> Again, it would be preferable to have this done as part of the build
> system, but OK.
>
> > + (propagated-inputs
> > + `(("ruby-httpclient" ,ruby-httpclient)
> > + ("ruby-multi-json" ,ruby-multi-json)
> > + ("ruby-progress_bar" ,ruby-progress_bar)
> > + ("ruby-rubyzip" ,ruby-rubyzip)
> > + ("ruby-thor" ,ruby-thor)))
> > + (synopsis
> > + "Utility for dumping and restoring ElasticSearch indexes")
> This can be moved onto the previous line.
>
> > + (description
> > + "This package provides a utility for dumping the contents of
> > an +ElasticSearch index to a compressed file and restoring the
> > dumpfile back to an +ElasticSearch server")
> > + (home-page
> > + "https://github.com/patientslikeme/es_dump_restore")
> As can this. Otherwise LGTM - thanks for this series.
These two formatting issues should be fixed now. Thanks for the review
and patches Ben. I've sent an updated set of patches.
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 963 bytes --]
^ permalink raw reply [flat|nested] 31+ messages in thread
* [bug#28583] [PATCH 1/7] gnu: Add ruby-httpclient.
2017-09-24 17:17 ` [bug#28583] [PATCH 1/7] gnu: Add ruby-httpclient Christopher Baines
` (5 preceding siblings ...)
2017-09-24 17:17 ` [bug#28583] [PATCH 7/7] gnu: Add es-dump-restore Christopher Baines
@ 2017-09-27 11:56 ` Ben Woodcroft
2017-09-30 8:21 ` Christopher Baines
6 siblings, 1 reply; 31+ messages in thread
From: Ben Woodcroft @ 2017-09-27 11:56 UTC (permalink / raw)
To: Christopher Baines, 28583
Hi,
Thanks for the patches.
On 25/09/17 03:17, Christopher Baines wrote:
> * gnu/packages/ruby.scm (ruby-httpclient): New variable.
> ---
> gnu/packages/ruby.scm | 51 +++++++++++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 51 insertions(+)
>
> diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm
> index 80d3bffd8..ff6be875f 100644
> --- a/gnu/packages/ruby.scm
> +++ b/gnu/packages/ruby.scm
> @@ -3579,6 +3579,57 @@ It has built-in support for the legacy @code{cookies.txt} and
> (home-page "https://github.com/sparklemotion/http-cookie")
> (license license:expat)))
>
> +(define-public ruby-httpclient
> + (package
> + (name "ruby-httpclient")
> + (version "2.8.3")
> + (source
> + (origin
> + (method url-fetch)
> + (uri (rubygems-uri "httpclient" version))
> + (sha256
> + (base32
> + "19mxmvghp7ki3klsxwrlwr431li7hm1lczhhj8z4qihl2acy8l99"))))
> + (build-system ruby-build-system)
> + (arguments
> + '(;; TODO: Some tests currently fail
> + ;; ------
> + ;; 211 tests, 729 assertions, 13 failures, 4 errors, 0 pendings,
> + ;; 2 omissions, 0 notifications
> + ;; 91.866% passed
> + ;; ------
> + ;; 6.49 tests/s, 22.41 assertions/s
> + #:tests? #f
> + #:phases
> + (modify-phases %standard-phases
> + (replace 'check
> + (lambda* (#:key tests? #:allow-other-keys)
> + (if tests?
> + (zero?
> + (system* "ruby"
> + "-Ilib"
> + "test/runner.rb"))
> + #t)))
Since tests? is #f, this is effectively dead code. But it could be
useful to someone continuing the tests, I'm not fussed.
> + (add-after 'install 'wrap-bin-httpclient
> + (lambda* (#:key outputs #:allow-other-keys)
> + (wrap-program (string-append (assoc-ref outputs "out")
> + "/bin/httpclient")
> + `("GEM_HOME" ":" prefix (,(getenv "GEM_HOME"))))
> + #t)))))
It would be preferable to move this to be part of the ruby-build-system
as is done in e.g. python-build-system, but OK.
Otherwise LGTM.
ben
^ permalink raw reply [flat|nested] 31+ messages in thread
* [bug#28583] [PATCH 1/7] gnu: Add ruby-httpclient.
2017-09-27 11:56 ` [bug#28583] [PATCH 1/7] gnu: Add ruby-httpclient Ben Woodcroft
@ 2017-09-30 8:21 ` Christopher Baines
0 siblings, 0 replies; 31+ messages in thread
From: Christopher Baines @ 2017-09-30 8:21 UTC (permalink / raw)
To: Ben Woodcroft; +Cc: 28583
[-- Attachment #1: Type: text/plain, Size: 2660 bytes --]
On Wed, 27 Sep 2017 21:56:04 +1000
Ben Woodcroft <b.woodcroft@uq.edu.au> wrote:
> Hi,
>
> Thanks for the patches.
>
>
> On 25/09/17 03:17, Christopher Baines wrote:
> > * gnu/packages/ruby.scm (ruby-httpclient): New variable.
> > ---
> > gnu/packages/ruby.scm | 51
> > +++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed,
> > 51 insertions(+)
> >
> > diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm
> > index 80d3bffd8..ff6be875f 100644
> > --- a/gnu/packages/ruby.scm
> > +++ b/gnu/packages/ruby.scm
> > @@ -3579,6 +3579,57 @@ It has built-in support for the legacy
> > @code{cookies.txt} and (home-page
> > "https://github.com/sparklemotion/http-cookie") (license
> > license:expat)))
> > +(define-public ruby-httpclient
> > + (package
> > + (name "ruby-httpclient")
> > + (version "2.8.3")
> > + (source
> > + (origin
> > + (method url-fetch)
> > + (uri (rubygems-uri "httpclient" version))
> > + (sha256
> > + (base32
> > + "19mxmvghp7ki3klsxwrlwr431li7hm1lczhhj8z4qihl2acy8l99"))))
> > + (build-system ruby-build-system)
> > + (arguments
> > + '(;; TODO: Some tests currently fail
> > + ;; ------
> > + ;; 211 tests, 729 assertions, 13 failures, 4 errors, 0
> > pendings,
> > + ;; 2 omissions, 0 notifications
> > + ;; 91.866% passed
> > + ;; ------
> > + ;; 6.49 tests/s, 22.41 assertions/s
> > + #:tests? #f
> > + #:phases
> > + (modify-phases %standard-phases
> > + (replace 'check
> > + (lambda* (#:key tests? #:allow-other-keys)
> > + (if tests?
> > + (zero?
> > + (system* "ruby"
> > + "-Ilib"
> > + "test/runner.rb"))
> > + #t)))
> Since tests? is #f, this is effectively dead code. But it could be
> useful to someone continuing the tests, I'm not fussed.
I'm keen to keep it, as the #:tests? #f has a TODO comment, maybe the
tests can be made to pass in the future.
> > + (add-after 'install 'wrap-bin-httpclient
> > + (lambda* (#:key outputs #:allow-other-keys)
> > + (wrap-program (string-append (assoc-ref outputs "out")
> > + "/bin/httpclient")
> > + `("GEM_HOME" ":" prefix (,(getenv "GEM_HOME"))))
> > + #t)))))
> It would be preferable to move this to be part of the
> ruby-build-system as is done in e.g. python-build-system, but OK.
Agreed, I'll try and make some time to look at this.
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 963 bytes --]
^ permalink raw reply [flat|nested] 31+ messages in thread