appimage: use original app desktop file as base for appimage one
ClosedPublic

Authored by kossebau on Nov 19 2018, 9:31 PM.

Details

Summary

Makes sure the strings & metadata are consistent, also brings in
translations.

Diff Detail

Repository
R32 KDevelop
Branch
reusenormaldesktopfile
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 5153
Build 5171: arc lint + arc unit
kossebau created this revision.Nov 19 2018, 9:31 PM
Restricted Application added a project: KDevelop. · View Herald TranscriptNov 19 2018, 9:31 PM
Restricted Application added a subscriber: kdevelop-devel. · View Herald Transcript
kossebau requested review of this revision.Nov 19 2018, 9:31 PM

Not exactly sure if " -b %U" is an appimage wrapper thing somehow? At least -b these days is (no longer?) a cmdl argument interpreted by kdevelop app.

Also not sure if the [Desktop Action OpenSession] will be picked up, but I expect appimaged to deploy the desktop file somehow to the system, so there is a chance it also gets picked up. Should not do a harm otherwise.

kossebau planned changes to this revision.Nov 19 2018, 10:24 PM

Mh, actually the desktop action cannot work, given the AppRun is only valid inside the appimage env. That was bogus.
Need to first understand the purpose and usage of this very desktop file some more...

kossebau updated this revision to Diff 45862.Nov 20 2018, 2:29 AM
  • drop actions from desktop file
  • do not replace kdevelop with AppRun in the Exec entry, the latter is actually wrong I learned on #AppImage from TheAssasin
This revision is now accepted and ready to land.Nov 20 2018, 8:57 AM
This revision was automatically updated to reflect the committed changes.