Summary: | Amarok crashed while doing "Organize Files" [@ Meta::Field::mpris20MapFromTrack] | ||
---|---|---|---|
Product: | [Applications] amarok | Reporter: | Talamona Francesco <ti.liame> |
Component: | Collections/Organize Files Dialog | Assignee: | Amarok Developers <amarok-bugs-dist> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | ralf-engels |
Priority: | NOR | ||
Version: | 2.7.0 | ||
Target Milestone: | 2.8 | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/amarok/ce337dd4db9807480b627406233063d4dd4aa232 | Version Fixed In: | 2.8 |
Sentry Crash Report: |
Description
Talamona Francesco
2013-04-07 15:49:36 UTC
While I agree it shouldn't crash, trying to move a file that is currently playing to another physical location is a bad idea anyway. I assume it only crashes when the file you try to move is playing, right? I agree, even if I do it very often. Actually it was the first time. You didn't answer my question: I assume it only crashes when the file you try to move is playing, right? Also: is this reproducible? I did, it was the first time it crashed, so I cannot know if it does this only when moving files that are being played. It wasn't the first file moved. (In reply to comment #4) > I did, it was the first time it crashed, so I cannot know if it does this > only when moving files that are being played. It wasn't the first file moved. That was actually a request to test :) A crash that is not reproducible at least by the reporter is of not much use to us... You were too quick to reply, then ;) I'll keep an eye on it. I think I found it and will submit a fix shortly. Should be hard to reproduce. A classical race condition. I tried yesterday several times, both while listening and not. It didn't crash. Git commit ce337dd4db9807480b627406233063d4dd4aa232 by Ralf Engels. Committed on 09/04/2013 at 13:40. Pushed by rengels into branch 'master'. Fix: Amarok crashed while doing "Organize Files" FIXED-IN: 2.8 M +1 -0 ChangeLog M +24 -21 src/core/meta/support/MetaUtility.cpp http://commits.kde.org/amarok/ce337dd4db9807480b627406233063d4dd4aa232 |