D5510: Improve how we set up the flatpak backend

Aleix Pol Gonzalez noreply at phabricator.kde.org
Thu Apr 20 10:18:59 UTC 2017


apol added a comment.


  In https://phabricator.kde.org/D5510#103509, @jgrulich wrote:
  
  > Overall looks good to me, I also now understand why you don't use appScope as part of the unique id, hovewer I've been suggested by Alex to use flatpak_installation_get_display_name() or flatpak_installation_get_id(), which would result into a more readable unique id.
  
  
  I wanted to do that, but they don't seem very reliable. See:
  `qDebug() << "ggggg" << flatpakInstallation << g_file_get_path(file) << "." << flatpak_installation_get_id(flatpakInstallation) << "." << flatpak_installation_get_display_name(flatpakInstallation);`
  
    $ plasma-discover --backends flatpak-backend |& grep ggg | sort -u
    discover(3265)/(default) FlatpakResource::installationPath: ggggg 0x7f5170009160 /var/lib/flatpak . default . Default system directory
    discover(3265)/(default) FlatpakResource::installationPath: ggggg 0x7f51700092b0 /home/apol/.local/share/flatpak .  . 
  
  These identifiers don't seem to be very reliable, at least on my system. Using _get_id was actually my first approach.

REPOSITORY
  R134 Discover Software Store

REVISION DETAIL
  https://phabricator.kde.org/D5510

To: apol, jgrulich
Cc: plasma-devel, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170420/0c1acb87/attachment.html>


More information about the Plasma-devel mailing list