diff options
author | Karen Arutyunov <karen@codesynthesis.com> | 2023-04-07 12:12:31 +0300 |
---|---|---|
committer | Karen Arutyunov <karen@codesynthesis.com> | 2023-04-07 12:30:13 +0300 |
commit | 9ad939e65513fb9d367ba039484febcd63423304 (patch) | |
tree | 37de25d685432cb61ad773784e3bfda8c72aeaea /bbot/worker/worker.cxx | |
parent | b92c1c200deef9a4e2bbd686080ecc4c64c22d10 (diff) |
Add support for bbot.install.ldconfig step in worker
Diffstat (limited to 'bbot/worker/worker.cxx')
-rw-r--r-- | bbot/worker/worker.cxx | 100 |
1 files changed, 81 insertions, 19 deletions
diff --git a/bbot/worker/worker.cxx b/bbot/worker/worker.cxx index b33fb9f..98c17d4 100644 --- a/bbot/worker/worker.cxx +++ b/bbot/worker/worker.cxx @@ -214,9 +214,11 @@ enum class step_id // bpkg_install, + bbot_install_ldconfig, // Note: disabled by default. + // Note that the bpkg_bindist_* steps are mutually exclusive and the latest // status change for them (via the leading +/- characters in the prefix) - // overrides all the previous ones. + // overrides all the previous ones. Disabled by default. // bpkg_bindist_debian, bpkg_bindist_fedora, @@ -299,6 +301,8 @@ static const strings step_id_str { "bpkg.install", + "bbot.install.ldconfig", + "bpkg.bindist.debian", "bpkg.bindist.fedora", "bpkg.bindist.archive", @@ -801,6 +805,27 @@ run_b (step_id step, verbosity, buildspec, forward<A> (a)...); } +template <typename... A> +static result_status +run_ldconfig (step_id step, + tracer& t, + string& log, const regexes& warn_detect, + const optional<step_id>& bkp_step, + const optional<result_status>& bkp_status, + string& last_cmd, + A&&... a) +{ + return run_cmd (step, + t, + log, + nullptr /* out_str */, path () /* out_file*/, + warn_detect, + "sudo ldconfig", + bkp_step, bkp_status, last_cmd, + process_env ("sudo"), + "ldconfig", forward<A> (a)...); +} + // Upload compressed manifest to the specified TFTP URL with curl. Issue // diagnostics and throw failed on invalid manifest or process management // errors and throw io_error for input/output errors or non-zero curl exit. @@ -1089,6 +1114,7 @@ build (size_t argc, const char* argv[]) prefix != "bpkg.test-separate.update" && prefix != "bpkg.test-separate.test" && prefix != "bpkg.install" && + prefix != "bbot.install.ldconfig" && prefix != "bpkg.bindist.debian" && prefix != "bpkg.bindist.fedora" && prefix != "bpkg.bindist.archive" && @@ -3340,33 +3366,69 @@ build (size_t argc, const char* argv[]) if (!target_pkg && create_target) rm_r (trace, &r.log, rwd / target_conf); - // bpkg install <env-config-args> <tgt-config-args> <pkg-config-args> - // <package-name> + // Install. // - step_id b (step_id::bpkg_install); - step_id s (step_id::bpkg_install); + { + // bpkg install <env-config-args> <tgt-config-args> <pkg-config-args> + // <package-name> + // + step_id b (step_id::bpkg_install); + step_id s (step_id::bpkg_install); - r.status |= run_bpkg ( - b, - trace, r.log, wre, - bkp_step, bkp_status, last_cmd, - "-v", - "install", - step_args (env_args, s), - step_args (tgt_args, s), - step_args (pkg_args, s), - pkg); + r.status |= run_bpkg ( + b, + trace, r.log, wre, + bkp_step, bkp_status, last_cmd, + "-v", + "install", + step_args (env_args, s), + step_args (tgt_args, s), + step_args (pkg_args, s), + pkg); - if (!r.status) + if (!r.status) + break; + } + + // Run ldconfig. + // + if (step_enabled (step_id::bbot_install_ldconfig)) + { + // sudo ldconfig <env-config-args> <tgt-config-args> <pkg-config-args> + // + step_id b (step_id::bbot_install_ldconfig); + step_id s (step_id::bbot_install_ldconfig); + + r.status |= run_ldconfig ( + b, + trace, r.log, wre, + bkp_step, bkp_status, last_cmd, + step_args (env_args, s), + step_args (tgt_args, s), + step_args (pkg_args, s)); + + if (!r.status) + break; + } + // + // Fail if the breakpoint refers to the bbot.install.ldconfig step but + // this step is disabled. + // + else if (bkp_step && *bkp_step == step_id::bbot_install_ldconfig) + { + fail_unreached_breakpoint (r); break; + } rm.status |= r.status; } // - // Fail if the breakpoint refers to the bpkg.install step but the - // package is not supposed to be installed from source. + // Fail if the breakpoint refers to the bpkg.install related steps but + // the package is not supposed to be installed from source. // - else if (bkp_step && *bkp_step == step_id::bpkg_install) + else if (bkp_step && + *bkp_step >= step_id::bpkg_install && + *bkp_step <= step_id::bbot_install_ldconfig) { fail_unreached_breakpoint (add_result ("install")); break; |