Summary: | KRunner master crashes with LightDM integration | ||
---|---|---|---|
Product: | [Plasma] krunner | Reporter: | Christoph Feck <cfeck> |
Component: | general | Assignee: | Plasma Bugs List <plasma-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | crash | CC: | agateau |
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: |
Description
Christoph Feck
2011-10-27 19:18:35 UTC
Aurelien, did you just push the same change to 4.7, or is it a different version of the patch? I do not run 4.7 branch, so I cannot test, but 450ff4903aceae6a8b275a9fb4c2696a87c76123 causes this crash with master. With the 4.7.3 release so near we should be cautious to not break branch. Aurelien, any update? Bug 283086 comment #6 looks related. this should hopefully be addressed in both the 4.7 and master branches of kde-workspace. i can't trigger it here (probably because i'm using a release build of libdbus, so asserts are likely compiled out?), so it's only a best-guess on my part that when the XDG_SEAT_PATH environment variable wasn't set, which causes an empty string to be passed as the path to QDBusInterface, that libdbus barfs on it. after my commit, it now checks to see if the env var has been set and only if so tries to detect lightdm. if anyone can confirm, that'd be great :) Thanks Aaron, looks fixed here. If I get a crash again, I will complain ;) |