curl path download file extension added #1896
Conversation
| curl -Lo bicep https://github.com/Azure/bicep/releases/latest/download/bicep-linux-x64 | ||
| chmod +x ./bicep | ||
| sudo mv ./bicep /usr/local/bin/bicep | ||
| curl -Lo bicep.bin https://github.com/Azure/bicep/releases/latest/download/bicep-linux-x64 |
There was a problem hiding this comment.
@anthony-c-martin - any idea why @pinakighatak needed to add the .bin extension here?
There was a problem hiding this comment.
Any extension will work. I just put bin. Curl does not seem to work without this extension.
There was a problem hiding this comment.
I'm not sure - I've not had any problems running this command on OSX, Linux or WSL.
@pinakighatak Does the following work for you by any chance (specifying the current directory with ./)?
curl -Lo ./bicep https://github.com/Azure/bicep/releases/latest/download/bicep-linux-x64
It doesn't matter much, as this is just a temporary name before the exe is moved to /usr/local/bin/bicep
There was a problem hiding this comment.
I'm trying this directly on my GitHub workflow pipeline. Adding ./bicep does not work. See below:
But adding an extension (any) works smoothly.
There was a problem hiding this comment.
given the extension doesn't matter, I'm inclined to merge this.
Contributing a feature
The latest version of curl confuses between a directory and file without extension when downloading form a URL. So in this PR, have added an extension to the bicep binary download, and rename it back when moving to the folder. This works.
Also added an environment variable for the resource group parameter. The docs did not illustrate this correctly.