From f4bec0b31dfe117aca94c2d72d489460c1f51057 Mon Sep 17 00:00:00 2001 From: Boris Kolpackov Date: Mon, 13 Jun 2022 15:13:46 +0200 Subject: Fix bug in originating dependent confirmation logic --- bpkg/package-skeleton.cxx | 4 ++-- bpkg/pkg-build.cxx | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/bpkg/package-skeleton.cxx b/bpkg/package-skeleton.cxx index 49d375a..fb76f67 100644 --- a/bpkg/package-skeleton.cxx +++ b/bpkg/package-skeleton.cxx @@ -343,7 +343,7 @@ namespace bpkg // Note also that on the first call we will have no configuration. And // so to keep things simple, we merge variable of the buildfile origin // into cmd_vars and then rebuild things from scratch. Note, however, - // that below we need to sort our these merged overrides into user and + // that below we need to sort out these merged overrides into user and // dependent, so we keep the old configuration for reference. // // Note also that dependent values do not clash with user overrides by @@ -416,7 +416,7 @@ namespace bpkg v.origin = variable_origin::buildfile; v.dependent = move (ov->dependent); - v.confirmed = true; + v.confirmed = ov->confirmed; } } diff --git a/bpkg/pkg-build.cxx b/bpkg/pkg-build.cxx index 9e3ad26..ca3387f 100644 --- a/bpkg/pkg-build.cxx +++ b/bpkg/pkg-build.cxx @@ -6972,7 +6972,7 @@ namespace bpkg dept = move (v.dependent); if (cycle) - dr << info << v.serialize_cmdline (); + dr << "\n " << v.serialize_cmdline (); else v.undefine (); } -- cgit v1.1