Skip to content

vscode/extensions/update_installed_exts.sh: log on download failure instead of early exit#388747

Open
schuelermine wants to merge 1 commit intoNixOS:masterfrom
schuelermine:patch/vscode/extensions/update/dont-fail
Open

vscode/extensions/update_installed_exts.sh: log on download failure instead of early exit#388747
schuelermine wants to merge 1 commit intoNixOS:masterfrom
schuelermine:patch/vscode/extensions/update/dont-fail

Conversation

@schuelermine
Copy link
Contributor

The update_installed_exts.sh script no longer fails completely when downloading fails.
This is useful if you have any extensions that aren't from the Microsoft VSCode extension marketplace.
Instead, the extension name is logged to stderr.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@github-actions github-actions bot added 6.topic: vscode A free and versatile code editor that supports almost every major programming language. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Mar 10, 2025
@drupol drupol requested a review from a user May 13, 2025 09:08
@drupol
Copy link
Contributor

drupol commented Jun 18, 2025

What's the status of this? Can I mark it as draft?

@ghost ghost removed their request for review July 12, 2025 21:09
@nixpkgs-ci nixpkgs-ci bot added the 9.needs: reviewer This PR currently has no reviewers requested and needs attention. label Jul 12, 2025
@nixpkgs-ci nixpkgs-ci bot added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Dec 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md 6.topic: vscode A free and versatile code editor that supports almost every major programming language. 9.needs: reviewer This PR currently has no reviewers requested and needs attention. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants