Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/auto update package type #6026

Merged
merged 8 commits into from
Jan 27, 2025
Merged

Conversation

timtmok
Copy link
Contributor

@timtmok timtmok commented Jan 16, 2025

Address #5538 and #5540

Modifies the Linux build to update product.json when building the installer. Adds a packageType to the product configuration, available on Linux only. Windows already adds in a target that is either system or user.

The update URLs will now check more specifically the exact installer that was used. This prepares Windows to download the correct installer for auto-updates.

Release Notes

New Features

Enables Positron to detect the type of install so that the update service knows which installer to use.

Bug Fixes

  • N/A

QA Notes

No noticeable changes other than the debug log on the main process should show an update url that matches the platform, arch, and install type (deb/rpm/system/user).

@timtmok timtmok requested a review from jmcphers January 16, 2025 21:26
Copy link

github-actions bot commented Jan 16, 2025

E2E Tests 🚀
This PR will run tests tagged with: @:critical

readme  valid tags

@timtmok timtmok force-pushed the feature/auto-update-package-type branch 2 times, most recently from 9d1ac8c to 39a43f1 Compare January 24, 2025 14:01
@timtmok timtmok force-pushed the feature/auto-update-package-type branch from 39a43f1 to d720fa9 Compare January 24, 2025 14:02
@timtmok timtmok merged commit 7a428dc into main Jan 27, 2025
8 checks passed
@timtmok timtmok deleted the feature/auto-update-package-type branch January 27, 2025 16:42
@github-actions github-actions bot locked and limited conversation to collaborators Jan 27, 2025
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.

2 participants