Merge lp:~jbicha/unity-lens-video/build-with-valac into lp:unity-lens-video

[Jeremy Bícha]

Build with default valac

This changes unity-lens-video to build against the default valac (for Ubuntu 13.10 this is currently valac 0.18 but we may switch to 0.20 as Debian testing already has). I have successfully test-built against valac-0.20.

[Paweł Stołowski (stolowski)]

For some reason it fails for me if I only have valac-0.20.1 (the current one in S), and uninstall valac-0.18:

saucy:/home/pawel/src/ubuntu/unity-lens-videos/build-with-valac% dpkg-buildpackage -us -uc -b -j5
dpkg-buildpackage: source package unity-lens-video
dpkg-buildpackage: source version 0.3.15daily13.06.21-0ubuntu1
dpkg-buildpackage: source changed by Ubuntu daily release <email address hidden>
dpkg-buildpackage: host architecture amd64
 dpkg-source --before-build build-with-valac
dpkg-checkbuilddeps: Unmet build dependencies: valac (>= 0.18)
dpkg-buildpackage: warning: build dependencies/conflicts unsatisfied; aborting
dpkg-buildpackage: warning: (Use -d flag to override.)
saucy:/home/pawel/src/ubuntu/unity-lens-videos/build-with-valac% valac --version
Vala 0.20.1

and btw, you'll need to merge trunk changes to fix issue with Gee.

review: Needs Fixing

[Jeremy Bícha (jbicha)]

No, it builds fine in a clean chroot. There is a package named "valac" that you need to have installed, not just valac-0.20.

My branch also merges into trunk without a problem.

[Paweł Stołowski (stolowski)]

You're right, I got confused. +1

review: Approve

[PS Jenkins bot (ps-jenkins)]

review: Approve (continuous-integration)

=== modified file 'debian/control'
--- debian/control	2013-05-16 16:51:18 +0000
+++ debian/control	2013-06-07 21:34:31 +0000
@@ -13,7 +13,7 @@
                libsoup-gnome2.4-dev,
                libunity-dev (>= 7.0.0),
                libzeitgeist-dev (>= 0.3.8),
-               valac-0.18,
+               valac (>= 0.18),
                xvfb,
 Standards-Version: 3.9.4
 Homepage: https://launchpad.net/unity-lens-video