-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Please considering remove smb
from X-KDE-Protocols
in desktop entry.
#14124
Labels
Comments
duplicate of #8731 |
As a note of correction, this is not a crash. mpv exits on its own volition. |
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 14, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 14, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 14, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 14, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 14, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 14, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 15, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 15, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 15, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 16, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 16, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 16, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 16, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 16, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 20, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 20, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 20, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 20, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 20, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 21, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Dudemanguy
added a commit
to Dudemanguy/mpv
that referenced
this issue
May 22, 2024
If we can run the built mpv binary, then it is possible to use a custom target that reads the protocols we have available in mpv and write the mpv.desktop file based on the output. For cases where this is not possible (e.g. cross compiling), then just install the unmodified mpv.desktop file like before. Fixes mpv-player#8731 and fixes mpv-player#14124.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Some distros don't include --enable-libsmbclient in ffmpeg build. File managers like Dolphin will try use smb:// as target as mpv's current desktop entry includes smb in X-KDE-Protocols, and mpv crashes because local ffmpeg lacks smb support. As --enable-libsmbclient isn't listed as dependency here, I suggest smb should be removed from X-KDE-Protocols, to make the entry file universal on various distros.
References (may be more distros to be added here):
Important Information
If known which version of mpv introduced the problemGPU model, driver and versionPossible screenshot or video of visual glitchesReproduction steps
Expected behavior
Target file should be provided with
kio-fuse
(under/run/user/1000/kio-fuse-*/
) instead of usingsmb://
directly.Actual behavior
Dolphin directly passes
smb://
link to mpv, and mpv crashes because local ffmpeg build doesn't support it.Log file
Sample filesThe text was updated successfully, but these errors were encountered: