Skip to content
New issue

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

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

Already on GitHub? # to your account

support both pwsh and powershell.exe in install/uninstall scripts #86

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Molier
Copy link

@Molier Molier commented Jan 18, 2025

the script didn't work on windows powershell 7 machine.
I think this makes it more versatile and work on more machines and also show an error.

fix: support both pwsh and powershell.exe in install/uninstall scripts

  • Add fallback to powershell.exe if pwsh not found
  • Prevent silent failures by checking for PowerShell availability
  • Add pause to show errors before window closes

fix: support both pwsh and powershell.exe in install/uninstall scripts

- Add fallback to powershell.exe if pwsh not found
- Prevent silent failures by checking for PowerShell availability
- Add pause to show errors before window closes
fix: support both pwsh and powershell.exe in install/uninstall scripts

- Add fallback to powershell.exe if pwsh not found
- Prevent silent failures by checking for PowerShell availability
- Add pause to show errors before window closes
@Nonary
Copy link
Owner

Nonary commented Jan 23, 2025

Can you submit this PR to the source repository?

https://github.com/Nonary/SunshineScriptInstaller

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants