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

Windows installation script does not install pixi #150

Closed
traversaro opened this issue Jun 27, 2023 · 3 comments · Fixed by #152
Closed

Windows installation script does not install pixi #150

traversaro opened this issue Jun 27, 2023 · 3 comments · Fixed by #152

Comments

@traversaro
Copy link
Contributor

I am following the instructions in the README to install pixi on Windows, and the command closes the Powersehll windows, on both Powershell and Powershell with Admin Permissions. This is the output before the windows is closed:

PS C:\WINDOWS\system32> iwr -useb https://raw.githubusercontent.com/prefix-dev/pixi/main/install/install.ps1 | iex
This script will automatically download and install Pixi (latest) for you.
Getting it from this url: https://github.com/prefix-dev/pixi/releases/latest/download/pixi-x86_64-pc-windows-msvc.zip
The binary will be installed into 'C:\Users\STraversaro\AppData\Local\pixi\bin'
@traversaro
Copy link
Contributor Author

I manually removed the try/catch, and the error is:

Expand-Archive : .tmp is not a supported archive file format. .zip is the only supported archive file format.
At C:\Users\STraversaro\test2\install.ps1:35 char:5
+     Expand-Archive -Path $TEMP_FILE -DestinationPath $PIXI_DIR -Force
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidArgument: (.tmp:String) [Expand-Archive], IOException
    + FullyQualifiedErrorId : NotSupportedArchiveFileExtension,Expand-Archive

Apparently [System.IO.Path]::GetTempFileName() returns a filename that ends with .tmp, and Expand-Archive does not like this.

@ruben-arts
Copy link
Contributor

Thanks for the issue! Fixed it in #152

@traversaro
Copy link
Contributor Author

Thanks for the fix!

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 a pull request may close this issue.

2 participants