Skip to content

Commit

Permalink
tests: copying only the out paths is not enough information for CA
Browse files Browse the repository at this point in the history
  • Loading branch information
yorickvP committed May 8, 2023
1 parent 12685ef commit 5e332aa
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
3 changes: 2 additions & 1 deletion tests/post-hook.sh
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,8 @@ fi
# Build the dependencies and push them to the remote store.
nix-build -o $TEST_ROOT/result dependencies.nix --post-build-hook "$pushToStore"
# See if all outputs are passed to the post-build hook by only specifying one
export BUILD_HOOK_ONLY_OUT_PATHS=1
# TODO: BUILD_HOOK_ONLY_OUT_PATHS does not work with CA tests
export BUILD_HOOK_ONLY_OUT_PATHS=$([ ! $NIX_TESTS_CA_BY_DEFAULT ])
nix-build -o $TEST_ROOT/result-mult multiple-outputs.nix -A a.first --post-build-hook "$pushToStore"

clearStore
Expand Down
1 change: 1 addition & 0 deletions tests/push-to-store.sh
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ set -e
echo Pushing "$OUT_PATHS" to "$REMOTE_STORE"
if [ -n "$BUILD_HOOK_ONLY_OUT_PATHS" ]; then
printf "%s" "$OUT_PATHS" | xargs nix copy --to "$REMOTE_STORE" --no-require-sigs
printf "%s" "$DRV_PATH" | xargs nix copy --to "$REMOTE_STORE" --no-require-sigs --derivation
else
printf "%s" "$DRV_PATH"^'*' | xargs nix copy --to "$REMOTE_STORE" --no-require-sigs
fi

0 comments on commit 5e332aa

Please sign in to comment.