D5405: Create desktop file name based on organization domain unless set explicitely

2017-06-13 Thread Leslie Zhai
This revision was automatically updated to reflect the committed changes. Closed by commit R467:ddd546251022: Set organization name for Wayland icon. (authored by lesliezhai). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D5405?vs=13342&id=15416#toc REPOSITORY R467 K3b CHANGES SINCE L

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-13 Thread Andrius Štikonas
stikonas added a comment. > ! In https://phabricator.kde.org/D5405#101683, @graesslin wrote: > My comment was not meant as a stop-this-effort comment. If we can improve here to do more sane things I'm all for it. No, it's not your comment. We were discussing this on #kde-devel and it l

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-12 Thread Martin Gräßlin
graesslin added a comment. In https://phabricator.kde.org/D5405#101628, @stikonas wrote: > Ok, so let us not change kcoreaddons now although it's still something that could have better default behaviour in KF6. My comment was not meant as a stop-this-effort comment. If we can impr

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-12 Thread Andrius Štikonas
stikonas abandoned this revision. stikonas added a comment. Ok, so let us not change kcoreaddons now although it's still something that could have better default behaviour in KF6. REPOSITORY R244 KCoreAddons REVISION DETAIL https://phabricator.kde.org/D5405 To: stikonas, mpyne, kossebau

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-12 Thread Martin Gräßlin
graesslin added a comment. In https://phabricator.kde.org/D5405#101626, @ltoscano wrote: > In https://phabricator.kde.org/D5405#101624, @graesslin wrote: > > > In https://phabricator.kde.org/D5405#101621, @ltoscano wrote: > > > > > In https://phabricator.kde.org/D5405#101615, @grae

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-12 Thread Luigi Toscano
ltoscano added a comment. In https://phabricator.kde.org/D5405#101624, @graesslin wrote: > In https://phabricator.kde.org/D5405#101621, @ltoscano wrote: > > > In https://phabricator.kde.org/D5405#101615, @graesslin wrote: > > > > > The desktop file name should follow the way how th

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-12 Thread Martin Gräßlin
graesslin added a comment. In https://phabricator.kde.org/D5405#101621, @ltoscano wrote: > In https://phabricator.kde.org/D5405#101615, @graesslin wrote: > > > The desktop file name should follow the way how the dbus name is created. If the applications are broken, then they are broke

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-12 Thread Nicolás Alvarez
nalvarez added a comment. Are you saying the d-bus service name should be equal to the .desktop filename? I don't see where this requirement comes from. REPOSITORY R244 KCoreAddons REVISION DETAIL https://phabricator.kde.org/D5405 To: stikonas, mpyne, kossebau, aacid, ltoscano Cc: nalva

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-12 Thread Luigi Toscano
ltoscano added a comment. In https://phabricator.kde.org/D5405#101615, @graesslin wrote: > The desktop file name should follow the way how the dbus name is created. If the applications are broken, then they are broken. Given that I mentioned this several times at KDE conferences, blogged

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-12 Thread Martin Gräßlin
graesslin added a comment. The desktop file name should follow the way how the dbus name is created. If the applications are broken, then they are broken. Given that I mentioned this several times at KDE conferences, blogged about it, sent mails to KDE devel lists I assume the application ma

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-12 Thread Luigi Toscano
ltoscano added a comment. One thing for sure: the code should be moved back to the constructor, and not dynamically at each call of desktopFileName(), or it would be a behavioral change. (Or at least it should be probably cached if we get an agreement in breaking this). And regarding th

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-12 Thread Luigi Toscano
ltoscano added a comment. In https://phabricator.kde.org/D5405#101577, @kossebau wrote: > In https://phabricator.kde.org/D5405#101574, @ltoscano wrote: > > > I'm still confused. Documentation or not, bad usage or not, is it correct that desktop file name is constructed from both the o

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-12 Thread Andrius Štikonas
stikonas added a comment. In fact API documentation says explicitely that > IMPORTANT: if the organization domain is set, the .desktop file that describes your application should have an entry like X-DBUS-ServiceName=reversed_domain.kmyapp For instance kwrite passes "http://www.kate-edi

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-12 Thread Luigi Toscano
ltoscano added a comment. In https://phabricator.kde.org/D5405#101575, @mak wrote: > In https://phabricator.kde.org/D5405#101574, @ltoscano wrote: > > > I'm still confused. Documentation or not, bad usage or not, is it correct that desktop file name is constructed from both the organi

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-12 Thread Andrius Štikonas
stikonas added a comment. In https://phabricator.kde.org/D5405#101575, @mak wrote: > In https://phabricator.kde.org/D5405#101574, @ltoscano wrote: > > > I'm still confused. Documentation or not, bad usage or not, is it correct that desktop file name is constructed from both the organi

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-12 Thread Friedrich W. H. Kossebau
kossebau added a comment. In https://phabricator.kde.org/D5405#101574, @ltoscano wrote: > I'm still confused. Documentation or not, bad usage or not, is it correct that desktop file name is constructed from both the organization domain AND the homepage? In the current KAboutData

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-12 Thread Matthias Klumpp
mak added a comment. In https://phabricator.kde.org/D5405#101574, @ltoscano wrote: > I'm still confused. Documentation or not, bad usage or not, is it correct that desktop file name is constructed from both the organization domain AND the homepage? Yes. See https://standards.fre

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-12 Thread Luigi Toscano
ltoscano added a comment. I'm still confused. Documentation or not, bad usage or not, is it correct that desktop file name is constructed from both the organization domain AND the homepage? REPOSITORY R244 KCoreAddons REVISION DETAIL https://phabricator.kde.org/D5405 To: stikonas, mpyn

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-12 Thread Friedrich W. H. Kossebau
kossebau requested changes to this revision. kossebau added a comment. This revision now requires changes to proceed. Having slept over this one night, I still think that this change should be not the way to go to fix the seen issues. The actual problem is bad usage in application code, u

D5405: Create desktop file name based on organization domain unless set explicitely

2017-04-11 Thread Andrius Štikonas
stikonas created this revision. stikonas added a project: Plasma on Wayland. Restricted Application added subscribers: Frameworks, plasma-devel. Restricted Application added a project: Frameworks. REVISION SUMMARY On Wayland icons are obtained from desktop files that are named reverseOrganizat