unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#44880] [PATCH] gnu: ruby-rubocop: Update to 1.4.2
@ 2020-11-26  9:21 Tomás Ortín Fernández via Guix-patches via
  2020-11-27 14:56 ` [bug#44880] Mistake Tomás Ortín Fernández via Guix-patches via
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Tomás Ortín Fernández via Guix-patches via @ 2020-11-26  9:21 UTC (permalink / raw)
  To: 44880

* gnu/packages/ruby.scm (ruby-rubocop): Update to 1.4.2
---
gnu/packages/ruby.scm | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm
index 2753403834..f2a3e3b3f9 100644
--- a/gnu/packages/ruby.scm
+++ b/gnu/packages/ruby.scm
@@ -7052,7 +7052,7 @@ run.")
 (define-public ruby-rubocop
   (package
     (name "ruby-rubocop")
-    (version "0.88.0")
+    (version "1.4.2")
     (source
      (origin
        (method git-fetch)               ;no tests in distributed gem
@@ -7062,7 +7062,7 @@ run.")
        (file-name (git-file-name name version))
        (sha256
         (base32
-         "1d06893jp8pd85fvgp5d16vqcf31bafi430v4f4y746ihyvhzz5r"))
+         "1kvzhzhzcdd5bqwjilb0fpp51sqjniww2b0g713n0cvhnlgchn2y"))
        (patches (search-patches "ruby-rubocop-break-dependency-cycle.patch"))))
     (build-system ruby-build-system)
     (arguments




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

* [bug#44880] Mistake
  2020-11-26  9:21 [bug#44880] [PATCH] gnu: ruby-rubocop: Update to 1.4.2 Tomás Ortín Fernández via Guix-patches via
@ 2020-11-27 14:56 ` Tomás Ortín Fernández via Guix-patches via
  2020-11-27 19:05 ` [bug#44880] [PATCH] gnu: ruby-rubocop: Update to 1.4.2 Tomás Ortín Fernández via Guix-patches via
  2021-02-25 22:50 ` Björn Höfling
  2 siblings, 0 replies; 5+ messages in thread
From: Tomás Ortín Fernández via Guix-patches via @ 2020-11-27 14:56 UTC (permalink / raw)
  To: 44880

It seems I made a mistake with the patch. I used the gem hash, but this uses the github source. I didn't check it well enough, sorry: I will package it properly and send the updated patch.

Tomás




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

* [bug#44880] [PATCH] gnu: ruby-rubocop: Update to 1.4.2.
  2020-11-26  9:21 [bug#44880] [PATCH] gnu: ruby-rubocop: Update to 1.4.2 Tomás Ortín Fernández via Guix-patches via
  2020-11-27 14:56 ` [bug#44880] Mistake Tomás Ortín Fernández via Guix-patches via
@ 2020-11-27 19:05 ` Tomás Ortín Fernández via Guix-patches via
  2020-12-04 19:58   ` Leo Famulari
  2021-02-25 22:50 ` Björn Höfling
  2 siblings, 1 reply; 5+ messages in thread
From: Tomás Ortín Fernández via Guix-patches via @ 2020-11-27 19:05 UTC (permalink / raw)
  To: 44880

I'm finding it very difficult to package the updated rubocop from git, so here is a patch with rubocop 1.4.2 packaged from the gem. It was packaged from git because the gem doesn't include tests, but I can't figure out how to make the tests work. I have already asked for help at help-guix, and I'm sending this patch in the meanwhile because the current rubocop version is too outdated to be useful at all. As soon as I have the tests figured out, I will send a patch enabling the tests.

Tomás


* gnu/packages/ruby.scm (ruby-rubocop): Update to 1.4.2, fetch from gem, disable tests
---
gnu/packages/ruby.scm | 51 +++++----------------------------------------------
1 file changed, 5 insertions(+), 46 deletions(-)

diff --git a/gnu/packages/ruby.scm b/gnu/packages/ruby.scm
index 2753403834..eeabedf3d1 100644
--- a/gnu/packages/ruby.scm
+++ b/gnu/packages/ruby.scm
@@ -7052,58 +7052,17 @@ run.")
 (define-public ruby-rubocop
   (package
     (name "ruby-rubocop")
-    (version "0.88.0")
+    (version "1.4.2")
     (source
      (origin
-       (method git-fetch)               ;no tests in distributed gem
-       (uri (git-reference
-             (url "https://github.com/rubocop-hq/rubocop")
-             (commit (string-append "v" version))))
-       (file-name (git-file-name name version))
+       (method url-fetch)
+       (uri (rubygems-uri "rubocop" version))
        (sha256
         (base32
-         "1d06893jp8pd85fvgp5d16vqcf31bafi430v4f4y746ihyvhzz5r"))
-       (patches (search-patches "ruby-rubocop-break-dependency-cycle.patch"))))
+         "1kvzhzhzcdd5bqwjilb0fpp51sqjniww2b0g713n0cvhnlgchn2y"))))
     (build-system ruby-build-system)
     (arguments
-     '(#:test-target "default"
-       #:phases
-       (modify-phases %standard-phases
-         (add-after 'unpack 'remove-problematic-tests
-           ;; These tests depend on Rubocop extensions, which cannot be
-           ;; included as they cause a dependency cycle with Rubocop itself.
-           (lambda _
-             (delete-file "spec/rubocop/config_loader_spec.rb")
-             (substitute* "Gemfile"
-               ((".*'rubocop-performance'.*") "")
-               ((".*'rubocop-rspec'.*") ""))
-             ;; Prevent "Unnecessary disabling of RSpec/* (unknown cop)"
-             ;; errors.
-             (substitute* (find-files "spec/rubocop/cop/" "_spec\\.rb$")
-               (("# (rubocop:(enable|disable) RSpec.*)" _ what)
-                (string-append "# Disabled: " what)))
-             #t))
-         (add-after 'unpack 'disable-bundler
-           (lambda _
-             (substitute* "Rakefile"
-               (("Bundler\\.setup.*") "nil\n"))
-             #t))
-         (replace 'replace-git-ls-files
-           (lambda _
-             (substitute* "rubocop.gemspec"
-               (("`git ls-files(.*)`" _ files)
-                (format #f "`find ~a -type f| sort`" files)))
-             #t))
-         (add-before 'check 'set-home
-           (lambda _
-             (setenv "HOME" (getcwd))
-             #t))
-         (add-before 'check 'make-adoc-files-writable
-           (lambda _
-             (let ((adoc-files (find-files "docs/modules/ROOT/pages"
-                                           "\\.adoc$")))
-               (for-each make-file-writable adoc-files))
-             #t)))))
+     '(#:tests? #f))  ;no tests in distributed gem
     (native-inputs
      `(("ruby-bump" ,ruby-bump)
        ("ruby-pry" ,ruby-pry)





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

* [bug#44880] [PATCH] gnu: ruby-rubocop: Update to 1.4.2.
  2020-11-27 19:05 ` [bug#44880] [PATCH] gnu: ruby-rubocop: Update to 1.4.2 Tomás Ortín Fernández via Guix-patches via
@ 2020-12-04 19:58   ` Leo Famulari
  0 siblings, 0 replies; 5+ messages in thread
From: Leo Famulari @ 2020-12-04 19:58 UTC (permalink / raw)
  To: 44880

On Fri, Nov 27, 2020 at 08:05:06PM +0100, Tomás Ortín Fernández via Guix-patches via wrote:
> I'm finding it very difficult to package the updated rubocop from git, so here is a patch with rubocop 1.4.2 packaged from the gem. It was packaged from git because the gem doesn't include tests, but I can't figure out how to make the tests work. I have already asked for help at help-guix, and I'm sending this patch in the meanwhile because the current rubocop version is too outdated to be useful at all. As soon as I have the tests figured out, I will send a patch enabling the tests.
> 
> * gnu/packages/ruby.scm (ruby-rubocop): Update to 1.4.2, fetch from gem, disable tests

Thanks! Did you check if the packages that depend on ruby-rubocop still
work after this upgrade? You can get the list of such packages with
`guix refresh --list-dependent ruby-rubocop` and then try building each
one.




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

* [bug#44880] [PATCH] gnu: ruby-rubocop: Update to 1.4.2
  2020-11-26  9:21 [bug#44880] [PATCH] gnu: ruby-rubocop: Update to 1.4.2 Tomás Ortín Fernández via Guix-patches via
  2020-11-27 14:56 ` [bug#44880] Mistake Tomás Ortín Fernández via Guix-patches via
  2020-11-27 19:05 ` [bug#44880] [PATCH] gnu: ruby-rubocop: Update to 1.4.2 Tomás Ortín Fernández via Guix-patches via
@ 2021-02-25 22:50 ` Björn Höfling
  2 siblings, 0 replies; 5+ messages in thread
From: Björn Höfling @ 2021-02-25 22:50 UTC (permalink / raw)
  To: 44880; +Cc: 44880-done, Tomás Ortín Fernández, Leo Famulari

[-- Attachment #1: Type: text/plain, Size: 330 bytes --]

Hi Tomás,

On Thu, 26 Nov 2020 10:21:18 +0100
Tomás Ortín Fernández via Guix-patches via <guix-patches@gnu.org> wrote:

> * gnu/packages/ruby.scm (ruby-rubocop): Update to 1.4.2

In the meantime, ruby-rubocop was updated to 1.10.0 in commit

301d1bacc1f33ed5f2f546488c3a1dbc66cc3f4f

Closing this patch.

Björn

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

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

end of thread, other threads:[~2021-02-25 22:51 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-11-26  9:21 [bug#44880] [PATCH] gnu: ruby-rubocop: Update to 1.4.2 Tomás Ortín Fernández via Guix-patches via
2020-11-27 14:56 ` [bug#44880] Mistake Tomás Ortín Fernández via Guix-patches via
2020-11-27 19:05 ` [bug#44880] [PATCH] gnu: ruby-rubocop: Update to 1.4.2 Tomás Ortín Fernández via Guix-patches via
2020-12-04 19:58   ` Leo Famulari
2021-02-25 22:50 ` Björn Höfling

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).