On 13/01/16 23:50, Ricardo Wurmus wrote:
Ben Woodcroft <b.woodcr...@uq.edu.au> writes:


[..]
+    (arguments
+     `(#:phases
+       (modify-phases %standard-phases
+         (add-before 'check 'patch-and-set-version
+           (lambda _
+             (setenv "VERSION" ,version)
+             #t)))))
That’s an odd phase name; why “patch”?  Is setting the environment
variable the only way to set the version or would a make-flag work here,
too?  (I haven’t checked if the ruby-build-system respects make-flags.)

The rest looks good to me.  Thanks!
Pushed, thanks. The phase name above was a bit crufty so I changed it, but had to go with setenv since the ruby-build-system doesn't support make-flags (or rake-flags).

Thanks,
ben

Reply via email to