dockerTools.buildLayeredImage: fix typo in comments
This commit is contained in:
parent
0d983f9f99
commit
da261e3631
2 changed files with 9 additions and 9 deletions
|
@ -625,11 +625,11 @@ rec {
|
|||
|
||||
echo "Cooking the image..."
|
||||
# tar exits with an exit code of 1 if files changed while it was
|
||||
# reading them. it considers a change in the number of hard links
|
||||
# reading them. It considers a change in the number of hard links
|
||||
# to be a "change", which can cause this to fail if images are being
|
||||
# built concurrently and auto-optimise-store is turned on. since
|
||||
# know the contents of these files will not change, we can reasonably
|
||||
# ignore this exit code
|
||||
# built concurrently and the auto-optimise-store nix option is turned on.
|
||||
# Since the contents of these files will not change, we can reasonably
|
||||
# ignore this exit code.
|
||||
set +e
|
||||
tar -C image --dereference --hard-dereference --sort=name \
|
||||
--mtime="@$SOURCE_DATE_EPOCH" --owner=0 --group=0 \
|
||||
|
|
|
@ -13,8 +13,8 @@ echo "Creating layer #$layerNumber for $storePath"
|
|||
|
||||
mkdir -p "$layerPath"
|
||||
|
||||
# make sure /nix and /nix/store appear first in the archive.
|
||||
# we create the directories here and use them because
|
||||
# Make sure /nix and /nix/store appear first in the archive.
|
||||
# We create the directories here and use them because
|
||||
# when there are other things being added to the
|
||||
# nix store, tar could fail, saying,
|
||||
# "tar: /nix/store: file changed as we read it"
|
||||
|
@ -25,16 +25,16 @@ tar -cf "$layerPath/layer.tar" \
|
|||
--transform='s,nix,/nix,' \
|
||||
nix
|
||||
|
||||
# we change into the /nix/store in order to avoid a similar
|
||||
# We change into the /nix/store in order to avoid a similar
|
||||
# "file changed as we read it" error as above. Namely,
|
||||
# if we use the absolute path of /nix/store/123-pkg
|
||||
# and something new it added to the nix store while tar
|
||||
# and something new is added to the nix store while tar
|
||||
# is running, it will detect a change to /nix/store and
|
||||
# fail. Instead, if we cd into the nix store and copy
|
||||
# the relative nix store path, tar will ignore changes
|
||||
# to /nix/store. In order to create the correct structure
|
||||
# in the tar file, we transform the relative nix store
|
||||
# path to the absolute store path
|
||||
# path to the absolute store path.
|
||||
n=$(basename "$storePath")
|
||||
tar -C /nix/store -rpf "$layerPath/layer.tar" \
|
||||
--hard-dereference --sort=name \
|
||||
|
|
Loading…
Reference in a new issue