From 49755e15ae067471d7b6ccfbe9114951334f4aeb Mon Sep 17 00:00:00 2001 From: Karen Arutyunov Date: Tue, 11 Jan 2022 14:41:22 +0300 Subject: Fix assertion in build_package::merge() --- bpkg/pkg-build.cxx | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/bpkg/pkg-build.cxx b/bpkg/pkg-build.cxx index df4c079..f78a96d 100644 --- a/bpkg/pkg-build.cxx +++ b/bpkg/pkg-build.cxx @@ -720,6 +720,11 @@ namespace bpkg // assert (action && *action != drop && (!p.action || *p.action != drop)); + // We never merge two repointed dependent reconfigurations. + // + assert ((flags & build_repoint) == 0 || + (p.flags & build_repoint) == 0); + // Copy the user-specified options/variables. // if (p.user_selection ()) @@ -780,17 +785,10 @@ namespace bpkg // flags |= p.flags; + // Upgrade repoint to the full build. + // if (*action == build) - { - // We never merge two repointed dependent reconfigurations. - // - assert ((flags & build_repoint) == 0 || - (p.flags & build_repoint) == 0); - - // Upgrade repoint to the full build. - // flags &= ~build_repoint; - } // Note that we don't copy the build_package::system flag. If it was // set from the command line ("strong system") then we will also have -- cgit v1.1