Merge ~vorlon/germinate:feature-no-follow-build-depends-all into germinate:master

Proposed by Steve Langasek
Status: Merged
Merged at revision: 259c65ca918aeaacb976be60c190f68e3ddcf953
Proposed branch: ~vorlon/germinate:feature-no-follow-build-depends-all
Merge into: germinate:master
Diff against target: 37 lines (+12/-3)
1 file modified
germinate/germinator.py (+12/-3)
Reviewer Review Type Date Requested Status
Colin Watson Approve
Review via email: mp+376481@code.launchpad.net

Description of the change

This lets us drop an additional 106 packages (including nodejs) from the i386 set, and 7 from the set of packages needing to be autopkgtested.

To post a comment you must log in.
Revision history for this message
Steve Langasek (vorlon) wrote :

Fixed the merge conflict.

Revision history for this message
Steve Langasek (vorlon) wrote :

Noticed a problem with libprelude being left out of the seed despite being a build-dependency of audit (causing some binaries to be uninstallable, but more importantly making audit unbuildable). Fixed as in the latest commit.

Revision history for this message
Colin Watson (cjwatson) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/germinate/germinator.py b/germinate/germinator.py
index 9b4e340..946e1a7 100644
--- a/germinate/germinator.py
+++ b/germinate/germinator.py
@@ -1653,20 +1653,29 @@ class Germinator(object):
1653 if pkg_src in output._blacklist:1653 if pkg_src in output._blacklist:
1654 seed._blacklisted.add(pkg_src)1654 seed._blacklisted.add(pkg_src)
1655 else:1655 else:
1656 # Special-case not following build dependencies of arch: all
1657 # packages
1658 if not self._follow_build_depends_all(seed.structure, seed) \
1659 and self._packages[pkg]["Architecture"] == "all":
1660 continue
1656 seed._not_build_srcs.add(pkg_src)1661 seed._not_build_srcs.add(pkg_src)
1657 seed._sourcepkgs.add(pkg_src)1662 seed._sourcepkgs.add(pkg_src)
16581663
1659 output._all_srcs.add(pkg_src)
1660 seed._build_srcs.add(pkg_src)
1661
1662 # Special-case not following build dependencies of arch: all1664 # Special-case not following build dependencies of arch: all
1663 # packages1665 # packages
1664 if not self._follow_build_depends_all(seed.structure, seed) and \1666 if not self._follow_build_depends_all(seed.structure, seed) and \
1665 self._packages[pkg]["Architecture"] == "all":1667 self._packages[pkg]["Architecture"] == "all":
1666 continue1668 continue
16671669
1670 output._all_srcs.add(pkg_src)
1671 seed._build_srcs.add(pkg_src)
1672
1668 if self._follow_build_depends(seed.structure, seed):1673 if self._follow_build_depends(seed.structure, seed):
1669 for build_depends in BUILD_DEPENDS:1674 for build_depends in BUILD_DEPENDS:
1675 if not self._follow_build_depends_all(seed.structure,
1676 seed) and \
1677 build_depends == 'Build-Depends-Indep':
1678 continue
1670 self._add_dependency_tree(1679 self._add_dependency_tree(
1671 seed, pkg, self._sources[pkg_src][build_depends],1680 seed, pkg, self._sources[pkg_src][build_depends],
1672 build_depend=True)1681 build_depend=True)

Subscribers

People subscribed via source and target branches

to all changes: