Re: [kde-devel] Re: kio-stash is in KDE Review
- Date: Thu, 29 Jun 2017 11:04:11 +0200
- From: Elvis Angelaccio <elvis.angelaccio@xxxxxxx>
- Subject: Re: [kde-devel] Re: kio-stash is in KDE Review
On Thu, Jun 29, 2017 at 8:11 AM, Arnav Dhamija <arnav.dhamija@xxxxxxxxx> wrote:
> Thanks for the suggestions. I've made all these changes in the latest
> commits. One thing bugging me is that the ID in the AppStream file
> needs a .desktop file, but I don't have one as this is an application.
> Should I leave it as it is or create a .desktop file for the same?
You don't need a .desktop file, something like
<id>org.kde.kio-stash</id> would be ok (check it with 'appstreamcli
> I'll be uploading the tarball shortly.
> On Wed, Jun 28, 2017 at 9:05 PM, Jonathan Riddell <jr@xxxxxxxxxxxx> wrote:
>> Versioning entirely depends on the project in question.
>> For a library with API/ABI stability it's important to set the right version and update it on API/ABI changes.
>> For anything else it's not too important but it's useful to have one
>> so people can report bugs on a known version, however I'm not sure
>> there's a good way to do so with a kioslave so maybe you can get away
>> with not doing it at all.
>>> Also, I have some confusion about AppStream procedure.
>>> > Your desktop file should be org.kde.foo.desktop. This is also your desktop-id, so whenver you may need that. It's basically your file name.
>>> > If you have a D-Bus service ... make sure that your D-Bus service name matches this or change your desktop file to match the service address (whichever seems more appropriate)
>>> > The name part 'foo' MUST be the name of your binary.
>>> In my case, I have two binaries - 'filestash.so' and
>>> 'stashnotifier.so'. I also have a D-Bus service name
>>> 'org.kde.kio.StashNotifier'. What would be my org.kde.foo name in this
>> It's not an application so I don't think it needs to match to a .desktop file, you'll need a <component> AppStream file for it. I'm not sure what the D-Bus stuff is about I'm afraid.
>>> Also, as for the screenshot part - my application is not really a GUI
>>> application as it is an ioslave. Would a screenshot of Dolphin loading
>>> the stash:/ ioslave do for this purpose?
>> Yes Dolphin would be fine, it's used by Discover and other package managers when people are offered to install it.
>> The files in the Git repository are still +x permission, that should be removed.
> arnav dhamija