From: Ben Woodcroft <b.woodcroft@uq.edu.au>
To: Efraim Flashner <efraim@flashner.co.il>,
Matthew Jordan <matthewjordandevops@yandex.com>
Cc: guix-devel <guix-devel@gnu.org>
Subject: Re: [Patch] ruby-debug-inspector
Date: Wed, 1 Jun 2016 17:35:14 +1000 [thread overview]
Message-ID: <574E9032.9040801@uq.edu.au> (raw)
In-Reply-To: <20160601070207.GA30401@debian-netbook>
Hi Matthew,
Thanks for the patch, all patches toward rails are much appreciated. I
do have a few comments though.
>> * gnu/packages/rails.scm: New file.
I'm of the opinion that we should keep most ruby packages in
gnu/packages/ruby.scm because to me "is this ruby package a rails
package?" is not one with a straightforward answer. We already have
activesupport in ruby.scm for instance, and that is more of a rails
package than this I would argue.
>> + #:use-module (guix build-system ruby))
>> +
>> +(define-public ruby-debug-inspector
>> + (package
>> + (name "ruby-debug-inspector")
>> + (version "0.0.2")
>> + (source
>> + (origin
>> + (method url-fetch)
>> + (uri (rubygems-uri "debug_inspector" version))
>> + (sha256
>> + (base32
>> + "109761g00dbrw5q0dfnbqg8blfm699z4jj70l4zrgf9mzn7ii50m"))))
>> + (build-system ruby-build-system)
>> + (arguments
>> + `(#:tests? #f)) ;; tests fail, when attepting to run the bacon command.
Looking a bit closer at this it appears the test directory is missing
from the gem but even the repository too. Maybe instead we add a cheap
and cheerful test like this:
(arguments
`(#:phases
(modify-phases %standard-phases
(replace 'check
(lambda _
(zero? (system* "ruby" "-Ilib" "-r" "debug_inspector")))))))
WDYT?
>> + (inputs
>> + `(("ruby-bacon" ,ruby-bacon)))
bacon is not a dependency other than for testing, is that right? If so I
don't think it is necessary.
>> + (synopsis "Ruby wrapper for the MRI 2.0 debug_inspector API")
>> + (description
>> + "This package provides a Ruby wrapper for the MRI 2.0 debug_inspector
>> +API.")
>> + (home-page
>> + "https://github.com/banister/debug_inspector")
>> + (license expat)))
Can you send an updated patch please? I'll look at your other patches
soon too.
Thanks,
ben
next prev parent reply other threads:[~2016-06-01 7:35 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-31 15:32 [Patch] ruby-debug-inspector Matthew Jordan
2016-05-31 17:59 ` Efraim Flashner
2016-05-31 20:44 ` Matthew Jordan
2016-06-01 7:02 ` Efraim Flashner
2016-06-01 7:35 ` Ben Woodcroft [this message]
2016-06-01 18:24 ` Matthew Jordan
2016-06-03 5:33 ` Matthew Jordan
2016-06-03 23:42 ` Ben Woodcroft
2016-06-04 0:04 ` Matthew Jordan
2016-06-04 1:43 ` Ben Woodcroft
2016-06-04 13:35 ` Matthew Jordan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=574E9032.9040801@uq.edu.au \
--to=b.woodcroft@uq.edu.au \
--cc=efraim@flashner.co.il \
--cc=guix-devel@gnu.org \
--cc=matthewjordandevops@yandex.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.