search

marioortizmanero / polybar-pulseaudio-control

A feature-full Polybar module to control PulseAudio
MIT License
471 stars 49 forks source link

Fix shebang #30

Closed M-Gregoire closed 4 years ago

M-Gregoire commented 4 years ago

Hi,

Thanks a lot for your script! However, it didn't work for me as bash isn't in /bin/ on my distribution. When using bash, it's recommended to use /usr/bin/env bash (See https://stackoverflow.com/questions/21612980/why-is-usr-bin-env-bash-superior-to-bin-bash/21613044).

This PR should allow more portability.

Side note: It's perfectly fine to use /bin/sh because POSIX requires to have /bin/sh available but bash isn't required to be there.

marioortizmanero commented 4 years ago

Yes, I should have done that a long time ago. Thanks for the PR!

M-Gregoire commented 4 years ago

I've just finished setting everything up, this is really awesome! Thank you for sharing :+1: