Skip to content

Conversation

Ericson2314
Copy link
Member

Motivation

This test ends up being skipped, since the bug has not yet been fixed. A
future commit will fix the bug.

Previous commit fixes a different bug (failure to clean up) that was getting in the way of my test! :)

Context

Progress on #13247, naturally.


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@Ericson2314 Ericson2314 requested a review from Mic92 September 26, 2025 04:34
@github-actions github-actions bot added the with-tests Issues related to testing. PRs with tests have some priority label Sep 26, 2025
@Ericson2314
Copy link
Member Author

Oh yikes, platform-specific failure?

…tput

In the case where the store object doesn't exist, we do correctly move
(rather than copy) the scratch data into place. In this case, the
destination store object already exists, but we still want to clean up
after ourselves.
delete

buildViaSubstitute () {
nix build -f issue-13247.nix "$1" --no-link --max-jobs 0 --substituters "$cache" --no-require-sigs --offline --substitute
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had to add --substitute to not make the lack of network access disable substituters. (I added --offline too simply to reproduce the issue in dev builds.)

I am curious why the other substitution tests are not affected by this.

Also, no network should not disable substitutes that don't use the network!

This test ends up being skipped, since the bug has not yet been fixed. A
future commit will fix the bug.

Progress on NixOS#13247, naturally.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
with-tests Issues related to testing. PRs with tests have some priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant