search

blue-systems / plasma-5.5

Plasma 5.2 - 5.5
0 stars 0 forks source link

[kio]: opening .mp3-file from folderview freezes desktop and returns error code on closing app #120

Closed star-buck closed 8 years ago

star-buck commented 9 years ago

while testing out "open with" mimetypes, I noticed this: https://www.dropbox.com/s/prx6e878asj4sv5/mp3-open-from-folderview.mkv?dl=0

eikehein commented 9 years ago

Seems like a dupe of the old "running Kate from any launcher gets an error after" bug that turned out to be a badly crafted Kate .desktop file ... anyone remember the bug number?

mck182 commented 9 years ago

This is the patch that fixed kate with the discussion --> https://git.reviewboard.kde.org/r/122498/ -- in there dfaure also suggests to use "X-DBUS-StartupType=None" in the .desktop file.

eikehein commented 9 years ago

@star-buck: can you attach the .desktop file for that music player?

star-buck commented 9 years ago

well, its happening with any of those apps seen in the screencast: audacious, vlc player, gmusicplayer...

eikehein commented 9 years ago

Thanks, I can reproduce it with VLC on Fedora. It doesn't happen in Dolphin (5), which is interesting since it should be the same library code producing both menus. Investigating.

star-buck commented 9 years ago

does not seem to happen though if you simply click on file to open as default application, only when choosing "open with"!

eikehein commented 9 years ago

After an hour of chasing this through 10 levels of classes I'm slowly closing in on a bug in the desktop:// KIO slave dropping local URL schemes on the floor. You'll notice the problem goes away when you switch the FV config from "Show the desktop folder" to custom folder + explicit path to Desktop dir.

eikehein commented 9 years ago

Patch up for review: https://git.reviewboard.kde.org/r/123781/

star-buck commented 9 years ago

@apachelogger : please ship in ci-updates.

eikehein commented 9 years ago

Pushed: http://commits.kde.org/kio-extras/fe1f50caaf24c47000938c9ac36a7f9a304f3c96

hsitter commented 9 years ago

Apparently I forgot to post the URL. Sorry about that

https://launchpad.net/~netrunner-os/+archive/ubuntu/netrunner-16-ci-updates/+sourcepub/5068703/+listing-archive-extra