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

Handle starter-kit post-install for statamic/cli users in Windows #6830

Merged
merged 17 commits into from Oct 6, 2022

Conversation

jesseleite
Copy link
Member

@jesseleite jesseleite commented Oct 4, 2022

The problem

When working on statamic/cli#56 for #6792, we realized that asking the user for input doesn't work within a sub-process in Windows environments. This is due to the lack of TTY support in Windows, and thus is not supported by symfony/process in Windows environments.

To be clear, this only affects:

  • Starter kits which contain post-install hooks
  • Where those post-install hooks ask the end user for input
  • Where the end user is running Windows

The solution

The solution we came up with is to cache friendly post-install instructions for statamic/cli to show at the end of the install process...

CleanShot 2022-10-04 at 17 48 46

By having the user manually run the post-install hook, it can be run as a regular process, rather than as a sub-process (via symfony/process), which allows the post-install hook to ask the user for input without issues in Windows.

@jesseleite jesseleite changed the title Cache starter-kit post-install instructions for statamic/cli users in Windows Handle starter-kit post-install for statamic/cli users in Windows Oct 4, 2022
@jesseleite jesseleite marked this pull request as ready for review October 6, 2022 00:44
@jesseleite jesseleite merged commit 54d513c into 3.3 Oct 6, 2022
@jesseleite jesseleite deleted the fix/starter-kit-post-install-hook-tty-windows branch October 6, 2022 00:44
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.

None yet

1 participant