Arch Linux User Repository

0.5-4 fixes the syntax error in the install script.

As for the autoload-or-not discussion, there's plenty of plugin packages in the repos that are automatically loaded once installed (e.g. all the vim-* packages), because the respective applications look for plugins in /usr. mpv does it the other way with /etc. So it's not as clear-cut as in the case of systemd services, which are never auto-enabled (except for systemd's own units).

I think at this point all arguments have been exchanged, and I'm not seeing myself changing my mind yet again soon. You can always edit the install file before building if you really disagree with this behavior.

0.5-4 fixes the syntax error in the install script.

As for the autoload-or-not discussion, there's plenty of plugin packages in the repos that are automatically loaded once installed (e.g. all the vim-* packages), because the respective applications look for plugins in /usr. mpv does it the other way with /etc. So it's not as clear-cut as in the case of systemd services, which are never auto-enabled (except for systemd's own units).

I think at this point all arguments have been exchanged, and I'm not seeing myself changing my mind yet again soon. You can always edit the install file before building if you really disagree with this behavior.

FWIW, I agree with majewsky and doskoi. The correct place to put the symlink is $HOME/.config/mpv/scripts. It is not something that has a system wide effect, so should not be configured system wide. Also it would seem to be very much the arch philosophy to leave things almost as minimally configured as possible to allow users a lot of flexibility.

post_upgrade() in the install script is an empty function and it throws an error.
Please remove it

With 0.5.-3:

(1/1) upgrading mpv-mpris                                                                                                                   [-------------------------------------------------------------------------------------] 100%
/tmp/alpm_rbE2av/.INSTALL: line 9: syntax error near unexpected token `}'
/tmp/alpm_rbE2av/.INSTALL: line 9: `}'

Arch Linux is a multi-user system you know. You, as an administrator, can install it for other specific users to enable it.

If you don't want it, don't install it. mpv is not systemd.

Don't auto load this, it's the responsibility of the user. systemd services are not auto enabled either.

It fails on my system because /etc/mpv/scripts is not present. Did it manually, but maybe you have to create the folder first.

I did not install to /etc in order to not break users who don't want to have the plugin load every single time.

The install script now creates the symlink you want, but only once, and tells the user to remove it if they don't want it.

Joining on the train of condemning comments - if you install this package then why would you ever need the plugin not to be enabled? Being able to 'disable' the plugin by not specifying the command option has no point and simply does not look good.

So you really should just install the .so directly into /etc/mpv/scripts directory and everything will work nicely out of the box and like expected.