input-ipc-server command line switch overridden by config in mpv.net v7

[Et0h]

input-ipc-server command line switch overridden by config in mpv.net v7

[stax76]

I could reproduce it and think I understand now what's wrong, I fix it in the next couple of days.

[stax76]

A new release fixing it is available.

[stax76]

In mpv.net v6 most CLI options were set after the initialization of the mpv core, in v7 this was changed to all CLI options being set before the initialization of the mpv core, I was told by a mpv developer that's the way to do it. It's probably this change causing the problem. I'll try to better understand it and fix it, but it will take time, as I'm currently taking a development break.

[soredake]

@Et0h can this be fixed on syncplay side by reading input-ipc-server value from mpv.conf and just using it?

[Et0h]

@soredake It would be too complicated for Syncplay to do this. For example, there could be several different conf files and in those files there might be one than one input-ipc-server value, e.g. due to profiles. It is just way too hacky and unpredictable. It'd be far better to just tell the user to delete their input-ipc-server value.

@stax76 It's interesting that an mpv developer called for this behaviour as it results in different results to the main MPV.

Let's provide an easier to test example of osd-msg1.

In mpv-0.37.0-x86_64 if I put osd-msg1=Bob in mpv.conf but run with the argument osd-msg1=Alice, it will display Alice.
On the other hand, if I do the same in mpv.net v7 it will display Bob.

For most things this isn't a big deal as I can just override the value via the IPC connection once mpv.net has started, but if the input-ipc-server value is set then I don't have an IPC connection in the first place.

If this is a known issue then people can just be instructed to not set input-ipc-server in their mpv.net configuration if they want to use Syncplay, and instead to set it via Syncplay itself (a feature I'm working on).

As such, this probably doesn't need to be seen as a high priority issue, but is worth knowing about.