-
Notifications
You must be signed in to change notification settings - Fork 887
coder templates pull
does not work as expected on Windows
#9291
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
Comments
First IssueI can pull a template just fine when I specify a path as instructed by PS C:\Users\matif\tmp> coder templates pull deeplearning
Extracting template to "deeplearning/"
PS C:\Users\matif\tmp> coder templates pull deeplearning PATH
Extracting template to "PATH"
PS C:\Users\matif\tmp> ls .\deeplearning\
PS C:\Users\matif\tmp> ls .\PATH\
Directory: C:\Users\matif\tmp\PATH
Mode LastWriteTime Length Name
---- ------------- ------ ----
-a--- 25/08/2023 10:17 am 10852 deeplearning-connect.png
-a--- 25/08/2023 10:17 am 11401 main.tf
-a--- 25/08/2023 10:17 am 2181 README.md When we do not provide a path, the folder is empty 2nd IssueWhen using the coder templates pull --tar deeplearning > zipped.tar.gz other alternatives I have tried, but with the same effect. coder templates pull --tar deeplearning | Out-File -Path zipped.tar.gz PS: Both of these pull methods work as expected on Linux(Ubuntu) and macOS(confirmed by @mtojek) |
coder templates pull
does not work on Windowscoder templates pull
does not work as expected on Windows
I investigated it a bit more and found out that by default, PowerShell converts the output to string and to keep the data as binary, we need to enable an experimental flag |
Using |
I think the 2nd issue that @matifali identified is out of scope of this GitHub issue. There isn't much we can do about Powershell breaking binary redirection. However, we could add an option to take a |
@spikecurtis For the second issue, I guess we should wait for PowerShell 7.4 to be widely available as it is not an issue with Coder. This issue can be closed if we solve 1st. |
Screen.Recording.2023-08-23.at.5.17.19.PM.mov
It creates a folder, but no files
The text was updated successfully, but these errors were encountered: