fabiospampinato / vscode-terminals

An extension for setting-up multiple terminals at once, or just running some commands.
MIT License
121 stars 19 forks source link

zsh support #25

Closed spenoir closed 5 years ago

spenoir commented 5 years ago

I have zsh installed and vscode terminal uses zsh with the config:

"terminal.integrated.shell.osx": "/bin/zsh",

When I open my project with the extension enabled and a terminals.json file I get the following error:

The terminal process command '/bin/zsh' failed to launch (exit code: 1)

fabiospampinato commented 5 years ago

I don't think this has anything to do with zsh, since I'm using it too, unless you have configured it badly or something.

Could you paste your terminals.json configuration?

spenoir commented 5 years ago

Good to know it works with zsh. What setting do you have for: terminal.integrated.shell?

my terminals.json

{
    "autorun": true,
    "autokill": true,
    "env": {
        "name": "value"
    },
    "terminals": [
        {
            "name": "Foo",
            "description": "Bar",
            "icon": "code",
            "cwd": "/Users/me/project",
            "commands": [
                "./run.sh",
                "docker-compose up"
            ]
        }
    ]
}    
fabiospampinato commented 5 years ago

What setting do you have for: terminal.integrated.shell?

I have:

"terminal.integrated.shell.osx": "/bin/zsh",

Are you sure the cwd path is right? because I can only reproduce this if that points to a non existent directory.

spenoir commented 5 years ago

You're right, some of my cwd paths were wrong, it works perfectly now. Thanks for your help

fabiospampinato commented 5 years ago

Maybe we should detect this sort of errors in the configuration 🤔

fabiospampinato commented 5 years ago

Maybe we should detect this sort of errors in the configuration 🤔

Implemented in v1.12.5 👍