On Thu, Jun 29, 2017 at 8:11 AM, Arnav Dhamija <[email protected]> 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 validate'). > > I'll be uploading the tarball shortly. > > On Wed, Jun 28, 2017 at 9:05 PM, Jonathan Riddell <[email protected]> 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 >>> case? >> >> 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. >> >> Jonathan > > > > -- > arnav dhamija
