Bug 373976 - krusader and krusader root icons are not open in wayland.
Summary: krusader and krusader root icons are not open in wayland.
Status: RESOLVED FIXED
Alias: None
Product: krusader
Classification: Applications
Component: general (show other bugs)
Version: 2.5.0
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Krusader Bugs Distribution List
URL:
Keywords:
Depends on: 378612
Blocks:
  Show dependency treegraph
 
Reported: 2016-12-20 20:13 UTC by miku84
Modified: 2018-05-06 00:15 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In: 2.6.1
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description miku84 2016-12-20 20:13:01 UTC
krusader and krusader root icons are not working in wayland.
they are working from console with krusader and dbus-lauch krusader or kdesu krusader as well. with this message in the console
"No X authentication info set for display ":1""

I have modified the shortcuts to these working commands but krusader is not open. nothing happens.

so the shortcut tries to open it through X? 
is it an issue?
thanks
Comment 1 miku84 2017-02-10 18:26:17 UTC
Krusader root runs with 
XDG_RUNTIME_DIR=/run/user/1000 krusader -qwindowtitle %c %u
Comment 2 Alex Bikadorov 2017-02-10 18:35:58 UTC
Oh, I forgot about this.

I tested this already and can verify that icons are not shown. But I couldn't find out the reason and wasn't able to spend much time for it as Wayland constantly crashed my system with many forced reboots.
Comment 3 Andrius Štikonas 2017-04-10 10:24:18 UTC
The reason for krusader icons not working in wayland is because krusader homepage is krusader.org and not krusader.kde.org. Somehow kwin passes window id as org.krusader.krusader to plasma libtaskmanager and then it tries to find icon in org.krusader.krusader.desktop instead of org.kde.krusader.desktop.
Comment 4 Luigi Toscano 2017-04-10 11:53:16 UTC
So let's discuss on the main bug (378612), please do not change the domain in the meantime (why provide both homepage and organizaton domain if they need to be the same? It's probably a bug somewhere else in Plasma).
Comment 5 Andrius Štikonas 2017-04-10 12:01:47 UTC
(In reply to Luigi Toscano from comment #4)
> So let's discuss on the main bug (378612), please do not change the domain
> in the meantime (why provide both homepage and organizaton domain if they
> need to be the same? It's probably a bug somewhere else in Plasma).

Yeah, I'm not changing domain, it's just how I noticed what triggers the bug. tosky sais organization domain should be kde.org anyway. So I think the fix should be in kwin, it has to supply correct id but I don't know much about how kwin works.
Comment 6 Andrius Štikonas 2017-04-11 15:11:50 UTC
Oh, so there is a bug in KCoreAddons on which this bug depends. But Krusader should still setOrganizationDomain(QStringLiteral("kde.org")); in the meantime.
Comment 7 Andrius Štikonas 2017-04-12 20:50:47 UTC
Git commit e396c209073d830dcb0d2e9eb7fab16f36fbbd8d by Andrius Štikonas.
Committed on 12/04/2017 at 21:45.
Pushed by stikonas into branch 'master'.

Set a path to desktop file.

M  +1          krusader/main.cpp

https://commits.kde.org/krusader/e396c209073d830dcb0d2e9eb7fab16f36fbbd8d