FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Install an add-on over files already in place by mattdawkins · Pull Request #1960 · Kitware/dive · GitHub

/ dive Public

Install an add-on over files already in place - #1960

Merged
BryonLewis merged 2 commits into
mainfrom
fix/addon-install-overwrites
Sep 23, 2026
Merged

BryonLewis merged 2 commits into
mainfrom
fix/addon-install-overwrites

Conversation

Copy link
Copy Markdown
Member
  • Installing an add-on whose marker or files are already present no longer errors (This add-on is already installed…); the installer is always run with --force and replaces the files. It already stages each file and backs up the previous copy, so an interrupted install still restores the old files.
  • The force request flag is gone from the desktop API and the Add-ons page.

🤖 Generated with Claude Code

mattdawkins and others added 2 commits September 21, 2026 23:17
The install always runs with --force: an add-on whose marker or files exist is replaced instead of refused, so a stale catalog status can no longer fail the install.
…ed copy

The Add-Ons page read download_viame_addons.csv from the installation's
bin folder only, so an installation without it could not list anything,
and one with it never saw packs published after its release. Fetch the
catalog VIAME maintains on GitHub first, keep that copy for the installer,
and use the bundled file only when GitHub cannot be reached, saying so on
the page.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
mattdawkins added a commit that referenced this pull request Sep 23, 2026
# Conflicts:
#	client/platform/desktop/backend/native/addons.ts
BryonLewis merged commit 7502c9f into main Sep 23, 2026
3 checks passed
BryonLewis deleted the fix/addon-install-overwrites branch September 23, 2026 16:57
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL