Make POSIX-compliant to increase portability and remove bash depend…#100145
Make POSIX-compliant to increase portability and remove bash depend…#100145joaomoreno merged 4 commits intomicrosoft:masterfrom nothingnesses:master
bash depend…#100145Conversation
…ency on Linux. Fix causes of `shellcheck` warnings.
nothingnesses
left a comment
There was a problem hiding this comment.
Should the VS Code on lines 8 and 33 be replaced with @@PRODNAME@@?
|
Does this fix an actual issue? |
|
I'm unable to start VS Code on my system which doesn't have |
|
Can you give me an example of said system? I'm just looking for a specific practical example. |
|
Sure thing. There are certain Linux distributions that don't enforce the usage of, and don't come pre-packaged with |
|
I'm actually surprised VS Code runs in said systems. 👌 |
Co-authored-by: VoidNoire <[email protected]>
Co-authored-by: VoidNoire <[email protected]>
…ency on Linux. Fix causes of
shellcheckwarnings.This PR fixes #100144 by removing bashisms and replacing them with POSIX-compliant alternatives.