Enhance 'install-powershell.ps1' to work on Linux/MacOS - #5411
Conversation
Aditya Patwardhan (adityapatwardhan)
left a comment
There was a problem hiding this comment.
Some non-blocking comments
| $userPath = [System.Environment]::GetEnvironmentVariable("Path", "User") | ||
| $userPath = $Destination + [System.IO.Path]::PathSeparator + $userPath | ||
| [System.Environment]::SetEnvironmentVariable("Path", $userPath, "User") | ||
| Write-Verbose "'$Destination' is added to the Path" -Verbose |
There was a problem hiding this comment.
I think, since we have CmdletBinding() we should not force -verbose
There was a problem hiding this comment.
These messages are actually intentionally written out so that a user roughly knows what the script is doing when running it.
|
|
||
| Expand-Archive -Path $packagePath -DestinationPath $Destination | ||
| if ($IsWinEnv) { | ||
| Expand-Archive -Path $packagePath -DestinationPath $Destination |
There was a problem hiding this comment.
Should we delete the compressed package after expansion?
There was a problem hiding this comment.
The compressed package will be deleted. The compressed package is downloaded to $tempDir which will be deleted in the finally block at the end of the script.
|
Linux job succeeded, but the macOS job was canceled again. |
) Enhance 'install-powershell.ps1' to work on Linux and macOS too. A user can install release version or daily powershell core on Linux and macOS. When `-AddToPath` is specified: - On Windows, add the absolute destination path to the 'User' scope environment variable 'Path'; - On Linux, make the symlink '/usr/bin/pwsh' points to "$Destination/pwsh"; - On MacOS, make the symlink '/usr/local/bin/pwsh' points to "$Destination/pwsh".
Enhance 'install-powershell.ps1' to work on Linux/MacOS.
When
-AddToPathis specified: