Merge lp:~robru/ubuntu-ui-toolkit/gles-experiment into lp:ubuntu-ui-toolkit/gles

Proposed by Robert Bruce Park on 2015-10-08
Status: Merged
Merged at revision: 84
Proposed branch: lp:~robru/ubuntu-ui-toolkit/gles-experiment
Merge into: lp:ubuntu-ui-toolkit/gles
Diff against target: 45 lines (+9/-4)
3 files modified
debian/control (+0/-1)
debian/rules (+9/-0)
debian/watch (+0/-3)
To merge this branch: bzr merge lp:~robru/ubuntu-ui-toolkit/gles-experiment
Reviewer Review Type Date Requested Status
Robert Bruce Park (community) Approve on 2015-10-09
Zoltan Balogh 2015-10-09 Pending
Review via email: mp+273926@code.launchpad.net

Commit Message

Packaging experiment.

To post a comment you must log in.
86. By Robert Bruce Park on 2015-10-08

Try not hard-coding jenkins home.

Robert Bruce Park (robru) wrote :

This works for me locally, except for the fact that I don't actually have the files locally for it to work. I'm testing this in the train except it's blocked by an unrelated bug, so I'll come back to this later:

https://requests.ci-train.ubuntu.com/#/ticket/497

87. By Robert Bruce Park on 2015-10-08

Feedback from slangasek.

88. By Robert Bruce Park on 2015-10-08

More feedback from slangasek.

Robert Bruce Park (robru) wrote :

Unknowns:

I'm not sure if the train passes the $SILONAME env var down into the chroot where this code will run. I might need to modify the train to pass that through if it doesn't already work.

I'm also not sure if ~ will expand to the right thing inside the chroot.

Hopefully these things aren't problems / are easily overcome so that we can stop hard-coding silonames into the packaging, so it works without changes in any silo.

89. By Robert Bruce Park on 2015-10-09

Let train pick versions.

Robert Bruce Park (robru) wrote :

Quite strange, the build log distinctly shows this working:

cp -v ~/silos/ubuntu/landing-060/ubuntu-ui-toolkit_1.3.1676+15.10.20151009.orig.tar.gz \
      ~/silos/ubuntu/landing-060/ubuntu-ui-toolkit-gles_1.3.1676+15.10.20151009.orig.tar.gz
'/var/lib/jenkins/silos/ubuntu/landing-060/ubuntu-ui-toolkit_1.3.1676+15.10.20151009.orig.tar.gz' -> '/var/lib/jenkins/silos/ubuntu/landing-060/ubuntu-ui-toolkit-gles_1.3.1676+15.10.20151009.orig.tar.gz'

But then the log later goes on to say:

Trying to use get-orig-source to retrieve needed tarball (deprecated).
get-orig-source did not create file for ubuntu-ui-toolkit-gles version 1.3.1676+15.10.20151009
Using uscan to look for the upstream tarball.
No watch file to use to retrieve upstream tarball.
bzr: ERROR: Unable to find the needed upstream tarball for package ubuntu-ui-toolkit-gles, version 1.3.1676+15.10.20151009.

But the train definitely has the files:

ls /var/lib/jenkins/silos/ubuntu/landing-060/*.orig.tar.gz
/var/lib/jenkins/silos/ubuntu/landing-060/ubuntu-ui-toolkit_1.3.1676+15.10.20151009.orig.tar.gz
/var/lib/jenkins/silos/ubuntu/landing-060/ubuntu-ui-toolkit-gles_1.3.1676+15.10.20151009.orig.tar.gz

Robert Bruce Park (robru) wrote :

Doesn't seem to be a permissions issue either, both files are 644 and owned by jenkins:jenkins.

Here's the original log:

https://ci-train.ubuntu.com/job/ubuntu-landing-060-1-build/16/console

90. By Robert Bruce Park on 2015-10-09

Maybe if I...

91. By Robert Bruce Park on 2015-10-09

But what about...

Robert Bruce Park (robru) wrote :

Ok this is now working.

review: Approve
Robert Bruce Park (robru) wrote :

No need to release this to ubuntu as it doesn't have any user-facing effect but it should be merged to trunk so we have it for the next release.

92. By Robert Bruce Park on 2015-10-09

Add comment.

93. By Robert Bruce Park on 2015-10-09

Stop hard-coding package name.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/control'
2--- debian/control 2015-09-28 15:49:18 +0000
3+++ debian/control 2015-10-09 16:56:18 +0000
4@@ -58,7 +58,6 @@
5 libqt5quickwidgets5-gles,
6 liblttng-ust-dev,
7 Standards-Version: 3.9.4
8-X-Auto-Uploader: no-rewrite-version
9 Homepage: https://launchpad.net/ubuntu-ui-toolkit
10
11 Package: qtdeclarative5-ubuntu-ui-toolkit-plugin-gles
12
13=== modified file 'debian/rules'
14--- debian/rules 2015-08-03 13:28:04 +0000
15+++ debian/rules 2015-10-09 16:56:18 +0000
16@@ -1,5 +1,8 @@
17 #!/usr/bin/make -f
18
19+VERSION := $(shell dpkg-parsechangelog | sed -n -e's/^Version: \(.*\)-[^-]\+/\1/p')
20+SOURCE := $(shell dpkg-parsechangelog | sed -n -e's/^Source: \(.*\)-gles/\1/p')
21+
22 # Uncomment this to turn on verbose mode.
23 # export DH_VERBOSE=1
24 export DPKG_GENSYMBOLS_CHECK_LEVEL=4
25@@ -10,6 +13,12 @@
26 CONFIGURE_SWITCHES = "-r" \
27 "ROOT_SOURCE_DIR=$(CURDIR)" \
28 "ROOT_BUILD_DIR=$(CURDIR)"
29+
30+# Don't try this at /home/: It will only work within https://ci-train.ubuntu.com
31+get-orig-source:
32+ cp -v ~/silos/$(SILONAME)/$(SOURCE)_$(VERSION).orig.tar.gz \
33+ $(CURDIR)/$(SOURCE)-gles_$(VERSION).orig.tar.gz
34+
35 %:
36 dh $@
37
38
39=== removed file 'debian/watch'
40--- debian/watch 2015-10-08 11:00:54 +0000
41+++ debian/watch 1970-01-01 00:00:00 +0000
42@@ -1,3 +0,0 @@
43-version=3
44-opts=dversionmangle=s/\-gles// \
45-http://ppa.launchpad.net/ci-train-ppa-service/landing-023/ubuntu/pool/main/u/ubuntu-ui-toolkit/ubuntu-ui-toolkit_([^-]*)\.orig.tar.gz

Subscribers

People subscribed via source and target branches