Skip to content

Git fetcher: Restore progress indication#14487

Merged
xokdvium merged 1 commit intomasterfrom
git-show-progress
Nov 5, 2025
Merged

Git fetcher: Restore progress indication#14487
xokdvium merged 1 commit intomasterfrom
git-show-progress

Conversation

@edolstra
Copy link
Member

@edolstra edolstra commented Nov 5, 2025

Motivation

This fixes Nix appearing to hang while fetching a Git repo (with no indication that it's doing that).

We were calling git with --quiet in order not to mess up Nix's progress bar. However, runProgram() already suspends the progress bar (since git may be interactive) so that's no longer an issue. So we can just run with --progress instead.

Context


Add 👍 to pull requests you find important.

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

We were calling git with `--quiet` in order not to mess up Nix's
progress bar. However, `runProgram()` already suspends the progress
bar (since git may be interactive) so that's no longer an issue. So we
can just run with `--progress` instead.
@github-actions github-actions bot added the fetching Networking with the outside (non-Nix) world, input locking label Nov 5, 2025
@xokdvium xokdvium enabled auto-merge November 5, 2025 21:26
@xokdvium xokdvium added this pull request to the merge queue Nov 5, 2025
Merged via the queue into master with commit b5302fc Nov 5, 2025
20 checks passed
@xokdvium xokdvium deleted the git-show-progress branch November 5, 2025 22:13
@edolstra edolstra mentioned this pull request Dec 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fetching Networking with the outside (non-Nix) world, input locking

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants