diff mbox

ruby-debug-inspector

Message ID 87inxqhm49.fsf@mailerver.i-did-not-set--mail-host-address--so-tickle-me
State New
Headers show

Commit Message

Matthew Jordan June 3, 2016, 5:33 a.m. UTC
Good Day,

This patch includes the suggested changes, and an update to the
license prefix.
Regards,

Comments

Ben Woodcroft June 3, 2016, 11:42 p.m. UTC | #1
On 06/03/2016 03:33 PM, Matthew Jordan wrote:
> Good Day,
>
> This patch includes the suggested changes, and an update to the
> license prefix.

Thanks. However, I realised that the testing phase was ineffective 
because the code catches the LoadError and returns without error. So I 
modified it to actually use the debug_inspector code, and pushed.

I hope this is the first of the many patches in rails.scm.

Thanks.
ben
Matthew Jordan June 4, 2016, 12:04 a.m. UTC | #2
Okay and thanks.

Also yes that patch is the first in rails patch series.


Ben Woodcroft writes:

> On 06/03/2016 03:33 PM, Matthew Jordan wrote:
>> Good Day,
>>
>> This patch includes the suggested changes, and an update to the
>> license prefix.
>
> Thanks. However, I realised that the testing phase was ineffective 
> because the code catches the LoadError and returns without error. So I 
> modified it to actually use the debug_inspector code, and pushed.
>
> I hope this is the first of the many patches in rails.scm.
>
> Thanks.
> ben
Ben Woodcroft June 4, 2016, 1:43 a.m. UTC | #3
On 06/04/2016 10:04 AM, Matthew Jordan wrote:
> Okay and thanks.
>
> Also yes that patch is the first in rails patch series.

Excellent. In case it matters, I have patches for ruby-tzinfo-data, 
ruby-timecop and ruby-concurrent near complete - I will submit these to 
this list for review soon.

ben
Matthew Jordan June 4, 2016, 1:35 p.m. UTC | #4
Good Day,

> Excellent. In case it matters, I have patches for ruby-tzinfo-data, 
> ruby-timecop and ruby-concurrent near complete - I will submit these to 
> this list for review soon.

Okay I'll take look at the depdency tree when I get the chance. And see
what may benefit.

Sincerely,

--
Matthew Jordan
Sent with my mu4e
diff mbox

Patch

From cfb727017d298c71b925c835f8f298c8db45fd7c Mon Sep 17 00:00:00 2001
From: Matthew Jordan <matthewjordandevops@yandex.com>
Date: Tue, 31 May 2016 07:26:24 -0400
Subject: [PATCH 1/2] gnu: Add ruby-debug-inspector.

* gnu/local.mk (GNU_SYSTEM_MODULES): Add it.
* gnu/packages/rails.scm: New file.
---
 gnu/local.mk           |  1 +
 gnu/packages/rails.scm | 50 ++++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 51 insertions(+)
 create mode 100644 gnu/packages/rails.scm

diff --git a/gnu/local.mk b/gnu/local.mk
index 59aaccc..5665aae 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -285,6 +285,7 @@  GNU_SYSTEM_MODULES =				\
   %D%/packages/qemu.scm				\
   %D%/packages/qt.scm				\
   %D%/packages/ragel.scm			\
+  %D%/packages/rails.scm			\
   %D%/packages/ratpoison.scm			\
   %D%/packages/rc.scm				\
   %D%/packages/rdesktop.scm			\
diff --git a/gnu/packages/rails.scm b/gnu/packages/rails.scm
new file mode 100644
index 0000000..80e73d7
--- /dev/null
+++ b/gnu/packages/rails.scm
@@ -0,0 +1,50 @@ 
+;;; GNU Guix --- Functional package management for GNU
+;;; Copyright © 2016 Matthew Jordan <matthewjordandevops@yandex.com>
+;;;
+;;; This file is part of GNU Guix.
+;;;
+;;; GNU Guix is free software; you can redistribute it and/or modify it
+;;; under the terms of the GNU General Public License as published by
+;;; the Free Software Foundation; either version 3 of the License, or (at
+;;; your option) any later version.
+;;;
+;;; GNU Guix is distributed in the hope that it will be useful, but
+;;; WITHOUT ANY WARRANTY; without even the implied warranty of
+;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+;;; GNU General Public License for more details.
+;;;
+;;; You should have received a copy of the GNU General Public License
+;;; along with GNU Guix.  If not, see <http://www.gnu.org/licenses/>.
+
+(define-module (gnu packages rails)
+  #:use-module ((guix licenses) #:prefix license:)
+  #:use-module (guix download)
+  #:use-module (guix packages)
+  #:use-module (gnu packages ruby)
+  #: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
+     `(#:phases
+       (modify-phases %standard-phases
+         (replace 'check
+           (lambda _
+             (zero? (system* "ruby" "-Ilib" "-r" "debug_inspector")))))))
+    (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 license:expat)))
-- 
2.8.3