Merge lp:~stolowski/unity8/pkg-version-into-varlib into lp:unity8

[Paweł Stołowski]

Store unity8 package version in /var/lib/.../version file for fast retrieval from the shell plugin.

* Are there any related MPs required for this MP to build/function as expected? Please list.
N/A
 * Did you perform an exploratory manual test run of your code change and any related functionality?
Y
 * Did you make sure that your branch does not contain spurious tags?
Y
 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
Y
 * If you changed the UI, has there been a design review?
N/A

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1980
http://jenkins.qa.ubuntu.com/job/unity8-ci/6410/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/4476
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-wily-touch/789/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/1122
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-wily/441
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/1017
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/1018
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-wily-amd64-ci/649
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-wily-i386-ci/650
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-mako/3641
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/4473
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/4473/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/23901
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-wily-mako/476/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-wily-armhf/789
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-wily-armhf/789/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/23904

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/6410/rebuild

review: Needs Fixing (continuous-integration)

[Albert Astals Cid (aacid)]

Please run strip-tags.py over lp:~stolowski/unity8/pkg-version-into-varlib

Also write https://wiki.ubuntu.com/Process/Merges/Checklists/Unity8 in the description

[Paweł Stołowski (stolowski)]

> Please run strip-tags.py over lp:~stolowski/unity8/pkg-version-into-varlib
>
> Also write https://wiki.ubuntu.com/Process/Merges/Checklists/Unity8 in the
> description
Ok, done.

[Albert Astals Cid (aacid)]

Why does the plugin need that?

Wouldn't be better using a call from unity8-dash to the plugin than a magic file somewhere?

review: Needs Information

[Paweł Stołowski (stolowski)]

> Why does the plugin need that?
>
> Wouldn't be better using a call from unity8-dash to the plugin than a magic
> file somewhere?

Plugin needs version numbers of itself, unity8, unity-scopes-api to send them to smart scopes server. Note, this may change in the future if smart scopes are ever abandoned (they are effectively in maintenence mode now).
In all cases we want real package versions and not artifical numbers (defined somewhere in the code) that would require manual updating. Currently we run dpkg-query during plugin startup, but this was proved to be very slow. The proposed solution is the fastest equivalent of running dpkg-query, without the associated cost. It's also a solution that gives version for all packages we're insterested in (I MP'ed same changes to scopes api and plugin), whereas making a special call for unity8-dash will make it look nicer for the dash only (and still, you need to get this version number in an efficient way somehow to pass it to the plugin).

[Albert Astals Cid (aacid)]

Ok, if this means the plugin doesn't need to call dpkg on runtime makes sense to have.

 * Did you perform an exploratory manual test run of the code change and any related functionality?
Yes

 * Did CI run pass?
Yes to the extend they do

 * Did you make sure that the branch does not contain spurious tags?
Yes

review: Approve

=== modified file 'data/CMakeLists.txt'
--- data/CMakeLists.txt	2015-03-12 13:59:07 +0000
+++ data/CMakeLists.txt	2015-10-01 13:10:43 +0000
@@ -11,5 +11,8 @@
     DESTINATION ${CMAKE_INSTALL_DATADIR}/applications
     )
 
+add_custom_target(pkgversion ALL COMMAND dpkg-parsechangelog -l${CMAKE_SOURCE_DIR}/debian/changelog --show-field version > ${CMAKE_CURRENT_BINARY_DIR}/version)
+
 install(FILES com.canonical.Unity8.gschema.xml DESTINATION ${CMAKE_INSTALL_DATADIR}/glib-2.0/schemas)
 install(FILES unity8-dash.url-dispatcher DESTINATION ${CMAKE_INSTALL_DATADIR}/url-dispatcher/urls)
+install(FILES ${CMAKE_CURRENT_BINARY_DIR}/version DESTINATION ${CMAKE_INSTALL_LOCALSTATEDIR}/lib/unity8)

=== modified file 'debian/unity8.install'
--- debian/unity8.install	2015-06-23 20:51:20 +0000
+++ debian/unity8.install	2015-10-01 13:10:43 +0000
@@ -1,3 +1,4 @@
+var/lib/unity8/version
 data/unity8-dash.conf usr/share/upstart/sessions/
 data/unity8-filewatcher.conf usr/share/upstart/sessions/
 data/unity8.conf usr/share/upstart/sessions/