[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#44882] [PATCH] gnu: ruby-solargraph: Update to 0.39.17 and add two
From: |
Leo Famulari |
Subject: |
[bug#44882] [PATCH] gnu: ruby-solargraph: Update to 0.39.17 and add two dependencies |
Date: |
Fri, 4 Dec 2020 14:44:08 -0500 |
On Thu, Nov 26, 2020 at 11:02:11AM +0100, Tomás Ortín Fernández via
Guix-patches via wrote:
> * gnu/packages/ruby.scm (ruby-solargraph): Update to 0.39.17.
> * gnu/packages/ruby.scm (ruby-e2mmap): New variable.
> * gnu/packages/ruby.scm (ruby-benchmark): New variable.
Thanks! I read your followup email but I'm also replying with some
advice about this patch.
First, this patch should be split into 3 patches. Each new package
should be added in its own patch, and then the solargraph update should
be the third patch.
> +(define-public ruby-e2mmap
> + (package
> + (name "ruby-e2mmap")
> + (version "0.1.0")
> + (source
> + (origin
> + (method url-fetch)
> + (uri (rubygems-uri "e2mmap" version))
> + (sha256
> + (base32
> + "0n8gxjb63dck3vrmsdcqqll7xs7f3wk78mw8w0gdk9wp5nx6pvj5"))))
> + (build-system ruby-build-system)
> + (propagated-inputs
> + `(("bundler" ,bundler)))
Is this 'bundler' really used? If so, is it used only at build time? In
that case, it would be a native input.
I checked that the built ruby-e2mmap doesn't keep a reference to
bundler, and thus wouldn't be able to find it at run-time:
------
$ guix gc --references $(./pre-inst-env guix build --no-grafts ruby-e2mmap)
/gnu/store/8ql9jjzl8q291ghsxlkm1wn5bpdvgcqw-ruby-2.6.5
/gnu/store/pwcp239kjf7lnj5i4lkdzcfcxwcfyk72-bash-minimal-5.0.16
------
> + (description
> + "Module for defining custom exceptions with specific messages.")
Descriptions should be "complete sentences". For example, this could be
written as "This package provides a Ruby module for defining custom
exceptions with specific messages."
I've attached a revised patchset to show what I mean. I was going to
push these revised patches until I read your followup message.
solargraph-patches
Description: Text document
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [bug#44882] [PATCH] gnu: ruby-solargraph: Update to 0.39.17 and add two dependencies,
Leo Famulari <=