Re: [PATCH] gnu: Add ruby-rack.

[Ludovic Courtès]

David Thompson <address@hidden> skribis:

> From a740a5ca98c02bd0e9c792677dfc8ff8464c8365 Mon Sep 17 00:00:00 2001
> From: David Thompson <address@hidden>
> Date: Fri, 4 Sep 2015 16:47:52 -0400
> Subject: [PATCH] gnu: Add ruby-rack.
>
> * gnu/packages/ruby.scm (ruby-rack): New variable.

[...]

> +         (add-before 'check 'fix-tests
> +           (lambda _
> +             ;; A few of the tests use the length of a file on disk for
> +             ;; Content-Length and Content-Range headers.  However, this file
> +             ;; has a shebang in it which an earlier phase patches, growing
> +             ;; the file size from 193 to 239 bytes.
> +             (substitute* '("test/spec_file.rb")
> +               (("193") "239")

I think this phase should use the actual length of the shebang, in case
the store is not at /gnu/store, with something like:

  (- (string-length (which "ruby")) (string-length "/usr/bin/ruby"))

Could you look into it?

TIA,
Ludo’.