search

mpv-player / mpv

🎥 Command line video player
https://mpv.io
Other
26.67k stars 2.83k forks source link

bash-completion: remove hashbang #14374

Closed diederikdehaas closed 1 week ago

diederikdehaas commented 2 weeks ago

Hashbangs are meant for scripts that are executed, but a bash completion script is meant to be sourced and therefor shouldn't have a hashbang.

Remarked by Debian's lintian tool with the bash-completion-with-hashbang tag.

github-actions[bot] commented 2 weeks ago

Download the artifacts for this pull request:

Windows * [mpv-i686-w64-mingw32](https://nightly.link/mpv-player/mpv/actions/artifacts/1608708358.zip) * [mpv-x86_64-w64-mingw32](https://nightly.link/mpv-player/mpv/actions/artifacts/1608716055.zip) * [mpv-x86_64-windows-msvc](https://nightly.link/mpv-player/mpv/actions/artifacts/1608745300.zip)
macOS * [mpv-macos-12-intel](https://nightly.link/mpv-player/mpv/actions/artifacts/1608716194.zip) * [mpv-macos-13-intel](https://nightly.link/mpv-player/mpv/actions/artifacts/1608712610.zip) * [mpv-macos-14-arm](https://nightly.link/mpv-player/mpv/actions/artifacts/1608705900.zip)