From 6cb9d0e810c3336106b6c1f3c8a80cdec6fbdcf0 Mon Sep 17 00:00:00 2001 From: Boris Kolpackov Date: Wed, 22 Nov 2017 11:31:35 +0200 Subject: Fix dist bug where missing source file would be silently ignored --- build2/dist/operation.cxx | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) (limited to 'build2/dist/operation.cxx') diff --git a/build2/dist/operation.cxx b/build2/dist/operation.cxx index f42217e..6692156 100644 --- a/build2/dist/operation.cxx +++ b/build2/dist/operation.cxx @@ -79,8 +79,8 @@ namespace build2 fail << "in-tree distribution of target " << t << info << "distribution requires out-of-tree build"; - // Make sure we have the necessary configuration before - // we get down to business. + // Make sure we have the necessary configuration before we get down to + // business. // auto l (rs->vars["dist.root"]); @@ -155,10 +155,9 @@ namespace build2 } } - // Add buildfiles that are not normally loaded as part of the - // project, for example, the export stub. They will still be - // ignored on the next step if the user explicitly marked them - // nodist. + // Add buildfiles that are not normally loaded as part of the project, + // for example, the export stub. They will still be ignored on the next + // step if the user explicitly marked them dist=false. // auto add_adhoc = [&trace] (const scope& rs, const path& f) { @@ -204,9 +203,8 @@ namespace build2 } } - // Collect the files. We want to take the snapshot of targets - // since updating some of them may result in more targets being - // entered. + // Collect the files. We want to take the snapshot of targets since + // updating some of them may result in more targets being entered. // action_targets files; const variable& dist_var (var_pool["dist"]); -- cgit v1.1