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

'spotify quit' quits closes terminal window as well #97

Closed
ryanyz10 opened this issue Apr 11, 2018 · 5 comments
Closed

'spotify quit' quits closes terminal window as well #97

ryanyz10 opened this issue Apr 11, 2018 · 5 comments
Labels

Comments

@ryanyz10
Copy link

Some weird behavior I've bee running into: 'spotify quit' also closes my terminal window that I'm working in. Is this expected behavior or a bug?

Shell is zsh, terminal is iTerm2.

kapture 2018-04-11 at 15 51 09

@hnarayanan
Copy link
Owner

It is not the intended behaviour, and I am not able to reproduce it in my setup (where it simply closes spotify as it should). I am running zsh on Apple's default Terminal.app.

@ryanyz10
Copy link
Author

ryanyz10 commented Apr 12, 2018

Maybe it's a problem with iTerm then, I'll see if I can look into it but I don't know much, if any, shell.

Edit: I changed the "exit 1" to "break" in the case (switch?) statement where all the commands are handled and it seems to still work fine.

@dmofot
Copy link
Collaborator

dmofot commented Apr 24, 2018

In iTerm2 v3.1.6 using bash shell, spotify quit does not close iTerm2. So it's possibly a combination of iTerm2 and zsh, but it could also be something with @ryanyz10 specific terminal/shell settings.

@hnarayanan
Copy link
Owner

hnarayanan commented Mar 24, 2019

I recently merged a PR that prompted me to review and clean up the use of breaks and exits.

This particular case, I've used exit 0 as technically that's what this shell script should do when attempting to quit the Spotify app.

Hopefully this fixes your case too.

@ZachAttackMLR
Copy link

ZachAttackMLR commented Apr 26, 2019

I ran into this issue yesterday using the built in terminal app on macOS running zsh. I used @ryanyz10 's fix and made a gif of it in case anyone else runs into this issue.

shpotifyQuitFix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants