Merge lp:~ubuntu-branches/ubuntu/quantal/autopkgtest/quantal-201207091011 into lp:ubuntu/quantal/autopkgtest

Proposed by Ubuntu Package Importer
Status: Needs review
Proposed branch: lp:~ubuntu-branches/ubuntu/quantal/autopkgtest/quantal-201207091011
Merge into: lp:ubuntu/quantal/autopkgtest
Diff against target: 99 lines (+61/-0) (has conflicts)
2 files modified
debian/changelog (+43/-0)
runner/adt-run (+18/-0)
Text conflict in debian/changelog
Text conflict in runner/adt-run
To merge this branch: bzr merge lp:~ubuntu-branches/ubuntu/quantal/autopkgtest/quantal-201207091011
Reviewer Review Type Date Requested Status
Ubuntu branches Pending
Review via email: mp+113943@code.launchpad.net

Description of the change

The package importer has detected a possible inconsistency between the package history in the archive and the history in bzr. As the archive is authoritative the importer has made lp:ubuntu/quantal/autopkgtest reflect what is in the archive and the old bzr branch has been pushed to lp:~ubuntu-branches/ubuntu/quantal/autopkgtest/quantal-201207091011. This merge proposal was created so that an Ubuntu developer can review the situations and perform a merge/upload if necessary. There are three typical cases where this can happen.
  1. Where someone pushes a change to bzr and someone else uploads the package without that change. This is the reason that this check is done by the importer. If this appears to be the case then a merge/upload should be done if the changes that were in bzr are still desirable.
  2. The importer incorrectly detected the above situation when someone made a change in bzr and then uploaded it.
  3. The importer incorrectly detected the above situation when someone just uploaded a package and didn't touch bzr.

If this case doesn't appear to be the first situation then set the status of the merge proposal to "Rejected" and help avoid the problem in future by filing a bug at https://bugs.launchpad.net/udd linking to this merge proposal.

(this is an automatically generated message)

To post a comment you must log in.

Unmerged revisions

39. By Martin Pitt

runner/adt-run: Split out a new read_stanzas() function from
read_control() to read a debian/control-like file, and reimplement
packages_from_source() to use that. This makes the function more robust
about different field orders and more in line with the existing code.

38. By Martin Pitt

runner/adt-run: Some stylistic updates mentioned by Ian.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2012-07-08 23:15:42 +0000
3+++ debian/changelog 2012-07-09 10:19:20 +0000
4@@ -1,3 +1,4 @@
5+<<<<<<< TREE
6 autopkgtest (2.2.3) unstable; urgency=medium
7
8 [ Martin Pitt ]
9@@ -23,6 +24,48 @@
10 * debian/control: add Vcs-* headers pointing to Alioth Git repository
11
12 -- Ian Jackson <ijackson@chiark.greenend.org.uk> Wed, 27 Jun 2012 21:36:56 +0100
13+=======
14+autopkgtest (2.2.0ubuntu5) UNRELEASED; urgency=low
15+
16+ * runner/adt-run: Some stylistic updates mentioned by Ian.
17+ * runner/adt-run: Split out a new read_stanzas() function from
18+ read_control() to read a debian/control-like file, and reimplement
19+ packages_from_source() to use that. This makes the function more robust
20+ about different field orders and more in line with the existing code.
21+
22+ -- Martin Pitt <martin.pitt@ubuntu.com> Wed, 04 Jul 2012 17:54:42 +0200
23+
24+autopkgtest (2.2.0ubuntu4) quantal; urgency=low
25+
26+ * runner/adt-run: Filter out udebs from the '@' expansion.
27+
28+ -- Martin Pitt <martin.pitt@ubuntu.com> Thu, 21 Jun 2012 18:06:35 +0200
29+
30+autopkgtest (2.2.0ubuntu3) quantal; urgency=low
31+
32+ * runner/adt-run: Fix for running in an already unpacked tree.
33+
34+ -- Martin Pitt <martin.pitt@ubuntu.com> Thu, 21 Jun 2012 11:10:01 +0200
35+
36+autopkgtest (2.2.0ubuntu2) quantal; urgency=low
37+
38+ * runner/adt-run: Expand '@' test Depends: from the test package's
39+ debian/control instead from the list of built .debs. The latter does not
40+ work if we do not actually build the source. This uses dh_listpackage to
41+ avoid duplicating all the "Architecture:" parsing magic, so add a
42+ debhelper dependency. (LP: #1015400, Closes: #678359)
43+
44+ -- Martin Pitt <martin.pitt@ubuntu.com> Thu, 21 Jun 2012 10:51:00 +0200
45+
46+autopkgtest (2.2.0ubuntu1) quantal; urgency=low
47+
48+ * Merge with Debian unstable. Remaining Ubuntu changes:
49+ - Fix hashbang lines to use python instead of python2.6, and update
50+ debian/control to only depend on python (>= 2.6) instead of
51+ python2.6. (Debian #644332)
52+
53+ -- Martin Pitt <martin.pitt@ubuntu.com> Tue, 19 Jun 2012 18:35:05 +0200
54+>>>>>>> MERGE-SOURCE
55
56 autopkgtest (2.2.0) unstable; urgency=low
57
58
59=== modified file 'runner/adt-run'
60--- runner/adt-run 2012-07-08 23:15:42 +0000
61+++ runner/adt-run 2012-07-09 10:19:20 +0000
62@@ -1355,13 +1355,18 @@
63
64 def packages_from_source(act, tree):
65 (rc, output) = subprocess_cooked(['dh_listpackages'],
66+<<<<<<< TREE
67 stdout=subprocess.PIPE, cwd=tree.read())
68+=======
69+ stdout=subprocess.PIPE, cwd=tree.read(True))
70+>>>>>>> MERGE-SOURCE
71 if rc: badpkg('failed to parse packages built from source, code %d' % rc)
72
73 # filter out empty lines
74 packages = [p for p in output.split() if p]
75
76 # filter out udebs
77+<<<<<<< TREE
78 control_af = RelativeInputFile(act.what+'-control',
79 tree, 'debian/control')
80 for st in read_stanzas(control_af):
81@@ -1373,6 +1378,19 @@
82 packages.remove(st['Package'][0][1])
83 except ValueError:
84 pass
85+=======
86+ control_af = RelativeInputFile(act.what+'-control',
87+ tree, 'debian/control')
88+ for st in read_stanzas(control_af):
89+ if 'Package' not in st:
90+ # source stanza
91+ continue
92+ if 'Xc-package-type' in st:
93+ try:
94+ packages.remove(st['Package'][0][1])
95+ except ValueError:
96+ pass
97+>>>>>>> MERGE-SOURCE
98
99 return packages
100

Subscribers

People subscribed via source and target branches

to all changes: