Skip to content

Installation workflow improvements #238

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

Merged

Conversation

cmcgee1024
Copy link
Member

@cmcgee1024 cmcgee1024 commented Feb 28, 2025

If the user happens to use the GUI installer to install the macOS pkg then
it will end up being install in /usr/local/bin/swift with no mechanism in place
for self-update. This patch redirects the GUI installer to install into the user
home directory.

For the easiest user experience swift.org may provide a copy-pastable set
of shell commands that can take the user directly to the swiftly init routine.
In this case there may be commands that can also take care of updating the
shell environment. In this case the init subcommand can take an option flag
to quiet the shell instructions.

@kateinoigakukun
Copy link
Member

I'm interested in the desired installation workflow you are thinking, so would you mind sharing the shape of the copy-pastable script? Are you planning to prohibit using the .pkg installer in GUI context?

@cmcgee1024
Copy link
Member Author

Are you planning to prohibit using the .pkg installer in GUI context?

Not prohibit it, just trying to direct the GUI installer to try and install in the current user's home directory for a better self-update experience.

@cmcgee1024
Copy link
Member Author

@swift-ci test macOS

@shahmishal
Copy link
Member

@swift-ci test

@cmcgee1024 cmcgee1024 merged commit 222d5da into swiftlang:release/1.0 Mar 12, 2025
19 checks passed
@shahmishal
Copy link
Member

Is there a PR for main?

@cmcgee1024
Copy link
Member Author

@shahmishal it's here: #249

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.

3 participants