[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Help Ruby packages be reproducible
From: |
Pjotr Prins |
Subject: |
Re: [PATCH] Help Ruby packages be reproducible |
Date: |
Wed, 30 Dec 2015 03:47:37 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Excellent.
On Wed, Dec 30, 2015 at 10:56:06AM +1000, Ben Woodcroft wrote:
> On 30/12/15 10:51, Ben Woodcroft wrote:
> >Too ham-fisted?
> Maybe, but badly committed at least. Try this.
> From 3d23171d88b9f38c90efa469f6519b52b15a1d01 Mon Sep 17 00:00:00 2001
> From: Ben Woodcroft <address@hidden>
> Date: Wed, 30 Dec 2015 10:27:33 +1000
> Subject: [PATCH] build: ruby: Remove cached gem after install.
>
> The .gem file stored in GEM_HOME after install is both redundant and an
> archive that stores timestamped files which makes builds non-deterministic. So
> delete it after 'gem install'.
>
> * guix/build/ruby-build-system.scm (install): Remove cached gem after install.
> ---
> guix/build/ruby-build-system.scm | 19 ++++++++++++-------
> 1 file changed, 12 insertions(+), 7 deletions(-)
>
> diff --git a/guix/build/ruby-build-system.scm
> b/guix/build/ruby-build-system.scm
> index 2685da1..b020272 100644
> --- a/guix/build/ruby-build-system.scm
> +++ b/guix/build/ruby-build-system.scm
> @@ -1,6 +1,7 @@
> ;;; GNU Guix --- Functional package management for GNU
> ;;; Copyright ?? 2015 David Thompson <address@hidden>
> ;;; Copyright ?? 2015 Pjotr Prins <address@hidden>
> +;;; Copyright ?? 2015 Ben Woodcroft <address@hidden>
> ;;;
> ;;; This file is part of GNU Guix.
> ;;;
> @@ -115,15 +116,19 @@ GEM-FLAGS are passed to the 'gem' invokation, if
> present."
> (assoc-ref inputs "ruby"))
> 1))
> (out (assoc-ref outputs "out"))
> - (gem-home (string-append out "/lib/ruby/gems/" ruby-version ".0")))
> -
> + (gem-home (string-append out "/lib/ruby/gems/" ruby-version ".0"))
> + (gem-name (first-matching-file "\\.gem$")))
> (setenv "GEM_HOME" gem-home)
> (mkdir-p gem-home)
> - (zero? (apply system* "gem" "install" (first-matching-file "\\.gem$")
> - "--local" "--ignore-dependencies"
> - ;; Executables should go into /bin, not /lib/ruby/gems.
> - "--bindir" (string-append out "/bin")
> - gem-flags))))
> + (apply system* "gem" "install" gem-name
> + "--local" "--ignore-dependencies"
> + ;; Executables should go into /bin, not /lib/ruby/gems.
> + "--bindir" (string-append out "/bin")
> + gem-flags)
> + ;; Remove the cached gem file as this is unnecessary and contains
> + ;; timestamped files rendering builds not reproducible.
> + (delete-file (string-append gem-home "/cache/" gem-name))
> + #t))
>
> (define %standard-phases
> (modify-phases gnu:%standard-phases
> --
> 2.6.3
>
--
- [PATCH] Add 12 rubygems., Ben Woodcroft, 2015/12/29
- Re: [PATCH] Add 12 rubygems., Ben Woodcroft, 2015/12/29
- Re: [PATCH] Add 12 rubygems., Ricardo Wurmus, 2015/12/29
- [PATCH] Help Ruby packages be reproducible (was: [PATCH] Add 12 rubygems.), Ben Woodcroft, 2015/12/29
- Re: [PATCH] Help Ruby packages be reproducible, Ben Woodcroft, 2015/12/29
- Re: [PATCH] Help Ruby packages be reproducible,
Pjotr Prins <=
- Re: [PATCH] Help Ruby packages be reproducible, Ricardo Wurmus, 2015/12/30
- Re: [PATCH] Help Ruby packages be reproducible, Ben Woodcroft, 2015/12/30
- Re: [PATCH] Help Ruby packages be reproducible, Ricardo Wurmus, 2015/12/31
- Re: [PATCH] Help Ruby packages be reproducible, Ben Woodcroft, 2015/12/31
Re: [PATCH] Add 12 rubygems., Ricardo Wurmus, 2015/12/30