Skip to content
This repository was archived by the owner on Jan 23, 2026. It is now read-only.

Fix Windows non-admin upgrade process#265

Closed
devin-ai-integration[bot] wants to merge 6 commits intomainfrom
devin/1746122473-fix-non-admin-upgrade
Closed

Fix Windows non-admin upgrade process#265
devin-ai-integration[bot] wants to merge 6 commits intomainfrom
devin/1746122473-fix-non-admin-upgrade

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

Fix Windows non-admin upgrade process

This PR fixes the issue with non-admin upgrades failing with error code 1603. The main changes are:

  1. Fix temporary directory handling to prevent premature cleanup
  2. Execute the MSI installer directly with the correct working directory
  3. Improve error handling with explicit cleanup on error paths

This addresses the issue reported in PR #263 where the non-admin upgrade was failing with error code 1603.

Link to Devin run: https://app.devin.ai/sessions/7879836ba03540cfb3987bcbd5dd94b0
Requested by: jhaynie@agentuity.com

devin-ai-integration Bot and others added 6 commits May 1, 2025 03:25
Co-Authored-By: jhaynie@agentuity.com <jhaynie@gmail.com>
Co-Authored-By: jhaynie@agentuity.com <jhaynie@gmail.com>
Co-Authored-By: jhaynie@agentuity.com <jhaynie@gmail.com>
Co-Authored-By: jhaynie@agentuity.com <jhaynie@gmail.com>
Co-Authored-By: jhaynie@agentuity.com <jhaynie@gmail.com>
…y handling and MSI execution

Co-Authored-By: jhaynie@agentuity.com <jhaynie@gmail.com>
@devin-ai-integration devin-ai-integration Bot requested a review from jhaynie May 1, 2025 18:02
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@jhaynie jhaynie closed this May 11, 2025
@jhaynie jhaynie deleted the devin/1746122473-fix-non-admin-upgrade branch May 11, 2025 01:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant