[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#30572] [PATCH 0/7] Add "guix system docker-image" command

From: Chris Marusich
Subject: [bug#30572] [PATCH 0/7] Add "guix system docker-image" command
Date: Thu, 22 Feb 2018 11:29:33 +0100


This patch series adds a new command: "guix system docker-image".  I
have the following questions for reviewers:

* In raw-initrd and base-initrd (gnu/system/linux-initrd.scm), why is
  %guile-static-stripped the right Guile to use by default?

* Instead of passing a Guile package to raw-initrd and base-initrd,
  would it be better to use a parameter (e.g., %guile-for-initrd),
  like we do for %guile-for-build?

* In system-docker-image (gnu/system/vm.scm), why is it necessary to
  define a custom (guix config) module?  I copied some of the logic
  from docker-image (guix/scripts/pack.scm), and it works, but I do
  not understand this part.

* Similarly, in gnu/system/vm.scm, why do we autoload libgcrypt?  I do
  not understand why we cannot simply #:use-module (gnu packages
  gnupg) like we do with all the other modules here.

* Similarly, in gnu/system/vm.scm, why do we use (ungexp-native json)?
  In the code, this is #+json and it occurs close to the comment
  "Guile-JSON is required by (guix docker)."

* Similarly, in gnu/system/vm.scm, there is a vestigial comment which
  says "This variable is unused but allows us to add INPUTS-TO-COPY as
  inputs" - this comment actually occurs in multiple places, not just
  the code I've added.  I think it refers to a variable that no longer
  exists, and I think that my variable to-register serves the intended
  purpose now, which is: add os-drv's inputs to the inputs of the gexp
  I'm building, so that they will be available on the build side.  So
  I'm thinking maybe we should just change this comment to say
  something more like that.  WDYT?

I have verified that this change builds and its tests pass.  I am
using it to run my very own Docker image today.  I hope you find it
useful, too!

Chris Marusich (7):
  tests: Add tests for "guix pack".
  vm: Allow control of deduplication in root-partition-initializer.
  system: Allow customization of the initrd's Guile.
  docker: Allow the use of a custom temporary directory.
  docker: Allow the addition of extra files into the image.
  system: Add "guix system docker-image" command.
  tests: Add tests for "guix system disk-image" et al.                           |   1 +
 doc/guix.texi                         |  74 ++++++++++++++++------
 gnu/build/vm.scm                      |  12 ++--
 gnu/system/examples/docker-image.tmpl |  47 ++++++++++++++
 gnu/system/linux-initrd.scm           |  31 ++++++---
 gnu/system/vm.scm                     | 116 ++++++++++++++++++++++++++++++++++
 guix/docker.scm                       |  29 +++++++--
 guix/scripts/system.scm               |  10 ++-
 tests/                    |  43 +++++++++++++
 tests/                  |  13 ++++
 10 files changed, 333 insertions(+), 43 deletions(-)
 create mode 100644 gnu/system/examples/docker-image.tmpl
 create mode 100644 tests/


reply via email to

[Prev in Thread] Current Thread [Next in Thread]