[AppImage Thumbnailer] Avoid creating QTemporaryFile
ClosedPublic

Authored by broulik on Sep 11 2018, 10:48 AM.

Details

Summary

Instead use new API that returns a buffer to avoid writing to the file system

Test Plan

Built libappimage with https://github.com/AppImage/libappimage/pull/9
Thumbnails are created just fine

Diff Detail

Repository
R320 KIO Extras
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik requested review of this revision.Sep 11 2018, 10:48 AM
broulik created this revision.
TheAssassin accepted this revision.Sep 11 2018, 11:21 AM
This revision is now accepted and ready to land.Sep 11 2018, 11:21 AM

https://phabricator.kde.org/source/kio-extras/browse/master/thumbnail/CMakeLists.txt$10

It should be specified the version that contains appimage_read_file_into_buffer_following_symlinks

anthonyfieroni requested changes to this revision.Sep 17 2018, 11:37 AM
This revision now requires changes to proceed.Sep 17 2018, 11:37 AM

Maybe it's better to make new code conditional to version that satisfy appimage_read_file_into_buffer_following_symlinks

The AppImage project doesn't do any versioning right now. We've recently extracted libappimage into a separate project to be able to start versioning of it, as projects like KDE expect and somehow need that. We're discussing a suitable versioning scheme right now. Please feel free to participate in the discussion in https://github.com/AppImage/AppImageKit/issues/849, and add your point of view.

This revision was not accepted when it landed; it landed in state Needs Revision.Oct 10 2018, 3:09 PM
This revision was automatically updated to reflect the committed changes.
Restricted Application added projects: Dolphin, Frameworks. · View Herald TranscriptOct 10 2018, 3:09 PM
Restricted Application added subscribers: kfm-devel, kde-frameworks-devel. · View Herald Transcript