Merge lp:~libqtelegram-team/telegram-app/app-bundle-thumbnailer into lp:telegram-app/app
Proposed by
Michał Karnicki
on 2015-07-20
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Approved by: | Roberto Mier Escandón on 2015-07-20 | ||||
| Approved revision: | 113 | ||||
| Merged at revision: | 113 | ||||
| Proposed branch: | lp:~libqtelegram-team/telegram-app/app-bundle-thumbnailer | ||||
| Merge into: | lp:telegram-app/app | ||||
| Diff against target: |
46 lines (+12/-0) 2 files modified
CMakeLists.txt (+6/-0) src/main.cpp (+6/-0) |
||||
| To merge this branch: | bzr merge lp:~libqtelegram-team/telegram-app/app-bundle-thumbnailer | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Roberto Mier Escandón | 2015-07-20 | Approve on 2015-07-20 | |
|
Review via email:
|
|||
Description of the Change
Bundle the old thumbnailer, as it has been removed in recent image. This is a temporary workaround before we have a proper C++ thumbnailer API.
I tested this the following way.
1) mv /usr/lib/
2) record a video, confirm the gallery cannot generate a thumbnail (proves the shipped vs-thumb will not be used by Telegram app)
3) open Telegram, send the video, confirm the thumbnail is present
To post a comment you must log in.


lgtm
Tested ok in regular and secret chats