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

create-iso: Use non-branched endless-installer.exe #147

Merged
merged 1 commit into from
Jan 19, 2024

Conversation

wjt
Copy link
Member

@wjt wjt commented Jan 16, 2024

In effect, this reverts 4a167ae and
a2f1eb4.

In the past, the Endless Installer for Windows was under active
development, with corresponding changes in the OS. To be able to e.g.
test the installer from within an ISO built by our infrastructure before
releasing the installer changes to the public, we had to fetch the
installer from a branched URL.

Now the installer is essentially dead, this just adds extra steps to our
release process for no benefit. Fetch from an unbranched URL.

I stand by the TODO I wrote in 977b96d
though it seems vanishingly unlikely we will ever do that.

https://phabricator.endlessm.com/T35105

In effect, this reverts 4a167ae and
a2f1eb4.

In the past, the Endless Installer for Windows was under active
development, with corresponding changes in the OS. To be able to e.g.
test the installer from within an ISO built by our infrastructure before
releasing the installer changes to the public, we had to fetch the
installer from a branched URL.

Now the installer is essentially dead, this just adds extra steps to our
release process for no benefit. Fetch from an unbranched URL.

I stand by the TODO I wrote in 977b96d
though it seems vanishingly unlikely we will ever do that.

https://phabricator.endlessm.com/T35105
@starnight starnight merged commit ab7fb80 into master Jan 19, 2024
2 checks passed
@starnight starnight deleted the T35105-unbranch-endless-installer.exe branch January 19, 2024 03:30
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