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
=== modified file 'debian/changelog'
--- debian/changelog 2012-07-08 23:15:42 +0000
+++ debian/changelog 2012-07-09 10:19:20 +0000
@@ -1,3 +1,4 @@
1<<<<<<< TREE
1autopkgtest (2.2.3) unstable; urgency=medium2autopkgtest (2.2.3) unstable; urgency=medium
23
3 [ Martin Pitt ]4 [ Martin Pitt ]
@@ -23,6 +24,48 @@
23 * debian/control: add Vcs-* headers pointing to Alioth Git repository24 * debian/control: add Vcs-* headers pointing to Alioth Git repository
2425
25 -- Ian Jackson <ijackson@chiark.greenend.org.uk> Wed, 27 Jun 2012 21:36:56 +010026 -- Ian Jackson <ijackson@chiark.greenend.org.uk> Wed, 27 Jun 2012 21:36:56 +0100
27=======
28autopkgtest (2.2.0ubuntu5) UNRELEASED; urgency=low
29
30 * runner/adt-run: Some stylistic updates mentioned by Ian.
31 * runner/adt-run: Split out a new read_stanzas() function from
32 read_control() to read a debian/control-like file, and reimplement
33 packages_from_source() to use that. This makes the function more robust
34 about different field orders and more in line with the existing code.
35
36 -- Martin Pitt <martin.pitt@ubuntu.com> Wed, 04 Jul 2012 17:54:42 +0200
37
38autopkgtest (2.2.0ubuntu4) quantal; urgency=low
39
40 * runner/adt-run: Filter out udebs from the '@' expansion.
41
42 -- Martin Pitt <martin.pitt@ubuntu.com> Thu, 21 Jun 2012 18:06:35 +0200
43
44autopkgtest (2.2.0ubuntu3) quantal; urgency=low
45
46 * runner/adt-run: Fix for running in an already unpacked tree.
47
48 -- Martin Pitt <martin.pitt@ubuntu.com> Thu, 21 Jun 2012 11:10:01 +0200
49
50autopkgtest (2.2.0ubuntu2) quantal; urgency=low
51
52 * runner/adt-run: Expand '@' test Depends: from the test package's
53 debian/control instead from the list of built .debs. The latter does not
54 work if we do not actually build the source. This uses dh_listpackage to
55 avoid duplicating all the "Architecture:" parsing magic, so add a
56 debhelper dependency. (LP: #1015400, Closes: #678359)
57
58 -- Martin Pitt <martin.pitt@ubuntu.com> Thu, 21 Jun 2012 10:51:00 +0200
59
60autopkgtest (2.2.0ubuntu1) quantal; urgency=low
61
62 * Merge with Debian unstable. Remaining Ubuntu changes:
63 - Fix hashbang lines to use python instead of python2.6, and update
64 debian/control to only depend on python (>= 2.6) instead of
65 python2.6. (Debian #644332)
66
67 -- Martin Pitt <martin.pitt@ubuntu.com> Tue, 19 Jun 2012 18:35:05 +0200
68>>>>>>> MERGE-SOURCE
2669
27autopkgtest (2.2.0) unstable; urgency=low70autopkgtest (2.2.0) unstable; urgency=low
2871
2972
=== modified file 'runner/adt-run'
--- runner/adt-run 2012-07-08 23:15:42 +0000
+++ runner/adt-run 2012-07-09 10:19:20 +0000
@@ -1355,13 +1355,18 @@
13551355
1356def packages_from_source(act, tree):1356def packages_from_source(act, tree):
1357 (rc, output) = subprocess_cooked(['dh_listpackages'],1357 (rc, output) = subprocess_cooked(['dh_listpackages'],
1358<<<<<<< TREE
1358 stdout=subprocess.PIPE, cwd=tree.read())1359 stdout=subprocess.PIPE, cwd=tree.read())
1360=======
1361 stdout=subprocess.PIPE, cwd=tree.read(True))
1362>>>>>>> MERGE-SOURCE
1359 if rc: badpkg('failed to parse packages built from source, code %d' % rc)1363 if rc: badpkg('failed to parse packages built from source, code %d' % rc)
13601364
1361 # filter out empty lines1365 # filter out empty lines
1362 packages = [p for p in output.split() if p] 1366 packages = [p for p in output.split() if p]
13631367
1364 # filter out udebs1368 # filter out udebs
1369<<<<<<< TREE
1365 control_af = RelativeInputFile(act.what+'-control',1370 control_af = RelativeInputFile(act.what+'-control',
1366 tree, 'debian/control')1371 tree, 'debian/control')
1367 for st in read_stanzas(control_af):1372 for st in read_stanzas(control_af):
@@ -1373,6 +1378,19 @@
1373 packages.remove(st['Package'][0][1])1378 packages.remove(st['Package'][0][1])
1374 except ValueError:1379 except ValueError:
1375 pass1380 pass
1381=======
1382 control_af = RelativeInputFile(act.what+'-control',
1383 tree, 'debian/control')
1384 for st in read_stanzas(control_af):
1385 if 'Package' not in st:
1386 # source stanza
1387 continue
1388 if 'Xc-package-type' in st:
1389 try:
1390 packages.remove(st['Package'][0][1])
1391 except ValueError:
1392 pass
1393>>>>>>> MERGE-SOURCE
13761394
1377 return packages1395 return packages
13781396

Subscribers

People subscribed via source and target branches

to all changes: