diff options
| author | Boris Kolpackov <boris@codesynthesis.com> | 2022-05-13 10:02:44 +0200 |
|---|---|---|
| committer | Karen Arutyunov <karen@codesynthesis.com> | 2022-06-07 21:14:14 +0300 |
| commit | 6c5feb88ef2174120a647d213f4e469c2c6ea093 (patch) | |
| tree | 8eabeacbde7312697d57c2be78baf6a5cbb4d557 | |
| parent | 1086739bca164d5fe9fd97eb5d7f7aefe3b42206 (diff) | |
Review
| -rw-r--r-- | bpkg/pkg-build.cxx | 19 |
1 files changed, 14 insertions, 5 deletions
diff --git a/bpkg/pkg-build.cxx b/bpkg/pkg-build.cxx index e7a50f2..ab73ee3 100644 --- a/bpkg/pkg-build.cxx +++ b/bpkg/pkg-build.cxx @@ -1194,6 +1194,12 @@ namespace bpkg struct postponed_configuration { + // The id of the cluster plus the ids of all the clusters that have been + // merged into it. + // + size_t id; + small_vector<size_t, 1> merged_ids; + using packages = small_vector<config_package, 1>; class dependency: public packages @@ -1896,6 +1902,9 @@ namespace bpkg for (auto i (begin ()); i != end (); ++i, ++r) ; return r; } + + private: + size_t next_id_ = 1; }; static ostream& @@ -4850,7 +4859,7 @@ namespace bpkg // @@ Re-evaluate up-to the cluster's dependencies. - // @@ TMP + // @@ TMP: need proper implementation. // { b->dependencies = dependencies (); @@ -5005,9 +5014,7 @@ namespace bpkg assert (i != pcfg->dependents.end () && i->second.dependencies.size () == 1); - const pair<size_t, size_t>& dp ( - i->second.dependencies[0].position); - + pair<size_t, size_t> dp (i->second.dependencies[0].position); assert (dp.first == sdeps.size () + 1); build_package::dependency_alternatives_refs pdas ( @@ -5249,9 +5256,11 @@ namespace bpkg pc->set_shadow_cluster (move (shadow)); - // Force-merge into this cluster those non-negotiated clusters + // Pre-merge into this cluster those non-negotiated clusters // which are subsets of the shadow cluster. // + // @@ TODO: use cluster ids instead. + // for (postponed_configuration& c: postponed_cfgs) { if (&c != pc && |