D7177: Do not refresh appstream metadata everytime

2017-08-07 Thread Jan Grulich
This revision was automatically updated to reflect the committed changes. Closed by commit R134:64ad5fc8d6c7: Do not refresh appstream metadata everytime (authored by jgrulich). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D7177?vs=17806&id=17821#toc REPOSITORY R134 Discover Software

D7177: Do not refresh appstream metadata everytime

2017-08-07 Thread Aleix Pol Gonzalez
apol added inline comments. INLINE COMMENTS > FlatpakBackend.cpp:509 > FlatpakRemote *remote = FLATPAK_REMOTE(g_ptr_array_index(remotes, > i)); > +fileTimestamp = flatpak_remote_get_appstream_timestamp(remote, > nullptr); > Can you declare and set in the same line? No need t

D7177: Do not refresh appstream metadata everytime

2017-08-07 Thread Aleix Pol Gonzalez
apol accepted this revision. This revision is now accepted and ready to land. REPOSITORY R134 Discover Software Store BRANCH appstream-refreshing REVISION DETAIL https://phabricator.kde.org/D7177 To: jgrulich, apol Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterb

D7177: Do not refresh appstream metadata everytime

2017-08-07 Thread Jan Grulich
jgrulich added a reviewer: apol. REPOSITORY R134 Discover Software Store REVISION DETAIL https://phabricator.kde.org/D7177 To: jgrulich, apol Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas

D7177: Do not refresh appstream metadata everytime

2017-08-07 Thread Jan Grulich
jgrulich created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY Refresh appstream metadata only in case they have never been refreshed or when the cache is older than 6 hours. REPOSITORY R134 Discove