From 90a6531951546ae88a695410013b908b50cd6fd0 Mon Sep 17 00:00:00 2001 From: Karen Arutyunov Date: Fri, 19 May 2017 16:19:29 +0300 Subject: Rename build states to unbuilt, building and built --- libbrep/build.cxx | 14 +++++++------- libbrep/build.hxx | 12 ++++++------ mod/mod-build-force.cxx | 2 +- mod/mod-build-log.cxx | 2 +- mod/mod-build-result.cxx | 4 ++-- mod/mod-build-task.cxx | 42 +++++++++++++++++++++--------------------- mod/mod-builds.cxx | 8 ++++---- 7 files changed, 42 insertions(+), 42 deletions(-) diff --git a/libbrep/build.cxx b/libbrep/build.cxx index 33aad45..ece2fd7 100644 --- a/libbrep/build.cxx +++ b/libbrep/build.cxx @@ -13,9 +13,9 @@ namespace brep { switch (s) { - case build_state::untested: return "untested"; - case build_state::testing: return "testing"; - case build_state::tested: return "tested"; + case build_state::unbuilt: return "unbuilt"; + case build_state::building: return "building"; + case build_state::built: return "built"; } return string (); // Should never reach. @@ -24,9 +24,9 @@ namespace brep build_state to_build_state (const string& s) { - if (s == "untested") return build_state::untested; - else if (s == "testing") return build_state::testing; - else if (s == "tested") return build_state::tested; + if (s == "unbuilt") return build_state::unbuilt; + else if (s == "building") return build_state::building; + else if (s == "built") return build_state::built; else throw invalid_argument ("invalid build state '" + s + "'"); } @@ -44,7 +44,7 @@ namespace brep configuration (id.configuration), toolchain_name (move (tnm)), toolchain_version (move (tvr)), - state (build_state::testing), + state (build_state::building), timestamp (timestamp_type::clock::now ()), forced (false), machine (move (mnm)), diff --git a/libbrep/build.hxx b/libbrep/build.hxx index 112100a..9258bb3 100644 --- a/libbrep/build.hxx +++ b/libbrep/build.hxx @@ -64,9 +64,9 @@ namespace brep // enum class build_state: std::uint8_t { - untested, - testing, - tested + unbuilt, + building, + built }; string @@ -117,7 +117,7 @@ namespace brep public: using timestamp_type = brep::timestamp; - // Create the build object with the testing state, non-existent status, + // Create the build object with the building state, non-existent status, // the timestamp set to now and the forced flag set to false. // build (string package_name, version package_version, @@ -144,11 +144,11 @@ namespace brep // bool forced; - // Present only if the state is 'tested'. + // Must present for the built state, may present for the building state. // optional status; - // Present only if the state is 'testing' or 'tested'. + // Present only for building and built states. // optional machine; optional machine_summary; diff --git a/mod/mod-build-force.cxx b/mod/mod-build-force.cxx index ed63146..9389566 100644 --- a/mod/mod-build-force.cxx +++ b/mod/mod-build-force.cxx @@ -169,7 +169,7 @@ handle (request& rq, response& rs) // Respond with 409 (conflict) if the package configuration is in // inappropriate state for being rebuilt. // - else if (b->state != build_state::tested) + else if (b->state != build_state::built) throw invalid_request (409, "state is " + to_string (b->state)); if (!b->forced) diff --git a/mod/mod-build-log.cxx b/mod/mod-build-log.cxx index a1e73ca..3daa1e3 100644 --- a/mod/mod-build-log.cxx +++ b/mod/mod-build-log.cxx @@ -194,7 +194,7 @@ handle (request& rq, response& rs) if (b == nullptr) config_expired ("no package configuration"); - else if (b->state != build_state::tested) + else if (b->state != build_state::built) config_expired ("state is " + to_string (b->state)); else build_db_->load (*b, b->results_section); diff --git a/mod/mod-build-result.cxx b/mod/mod-build-result.cxx index 0574996..d78c45f 100644 --- a/mod/mod-build-result.cxx +++ b/mod/mod-build-result.cxx @@ -238,7 +238,7 @@ handle (request& rq, response&) if (b == nullptr) warn_expired ("no package configuration"); - else if (b->state != build_state::testing) + else if (b->state != build_state::building) warn_expired ("package configuration state is " + to_string (b->state)); else if (b->timestamp != session_timestamp) warn_expired ("non-matching timestamp"); @@ -252,7 +252,7 @@ handle (request& rq, response&) prev_status = move (b->status); - b->state = build_state::tested; + b->state = build_state::built; b->status = rqm.result.status; b->forced = false; diff --git a/mod/mod-build-task.cxx b/mod/mod-build-task.cxx index ec236c0..911fb9b 100644 --- a/mod/mod-build-task.cxx +++ b/mod/mod-build-task.cxx @@ -139,14 +139,14 @@ handle (request& rq, response& rs) } // Go through packages until we find one that has no build configuration - // present in the database, or has the untested one, or in the testing state + // present in the database, or has the unbuilt one, or in the building state // but expired (collectively called unbuilt). If such a package - // configuration is found then put it into the testing state, set the + // configuration is found then put it into the building state, set the // current timestamp and respond with the task for building this package // configuration. // // While trying to find a non-built package configuration we will also - // collect the list of the tested package configurations which it's time to + // collect the list of the built package configurations which it's time to // rebuilt. So if no unbuilt package is found, we will pickup one to // rebuild. The rebuild preference is given in the following order: the // greater force flag, the greater overall status, the lower timestamp. @@ -197,7 +197,7 @@ handle (request& rq, response& rs) }; // Calculate the expiration time for package configurations being in the - // testing (build expiration) or the tested (rebuild expiration) state. + // building (build expiration) or the built (rebuild expiration) state. // timestamp now (timestamp::clock::now ()); @@ -280,7 +280,7 @@ handle (request& rq, response& rs) // configurations that have already been acted upon (initially empty). // // This is why we query the database for package configurations that - // should not be built (in the tested state, or in the testing state and + // should not be built (in the built state, or in the building state and // not expired). Having such a list we will select the first build // configuration that is not in the list (if available) for the response. // @@ -307,8 +307,8 @@ handle (request& rq, response& rs) toolchain_version, true) && - (bld_query::state == "tested" || - (bld_query::state == "testing" && + (bld_query::state == "built" || + (bld_query::state == "building" && bld_query::timestamp > build_expiration_ns))); connection_ptr bld_conn (build_db_->connection ()); @@ -354,7 +354,7 @@ handle (request& rq, response& rs) // configurations that remained can be built. We will take the first // one, if present. // - // Also save the tested package configurations for which it's time + // Also save the built package configurations for which it's time // to be rebuilt. // config_machines configs (cfg_machines); // Make a copy for this pkg. @@ -370,7 +370,7 @@ handle (request& rq, response& rs) assert (j != configs.end ()); configs.erase (j); - if (i->state == build_state::tested && + if (i->state == build_state::built && i->timestamp <= (i->forced ? forced_rebuild_expiration : normal_rebuild_expiration)) @@ -385,7 +385,7 @@ handle (request& rq, response& rs) shared_ptr b (build_db_->find (bid)); // If build configuration doesn't exist then create the new one - // and persist. Otherwise put it into the testing state, refresh + // and persist. Otherwise put it into the building state, refresh // the timestamp and update. // if (b == nullptr) @@ -403,12 +403,12 @@ handle (request& rq, response& rs) } else { - // The package configuration can be in the testing or untested + // The package configuration can be in the building or unbuilt // state, so the forced flag is false and the status is absent, - // unless in the testing state (in which case they may or may not - // be set/exist), and there are no results. + // unless in the building state (in which case they may or may + // not be set/exist), and there are no results. // - // Note that in the testing state the status can be present if + // Note that in the building state the status can be present if // the rebuild task have been issued. In both cases we keep the // status intact to be able to compare it with the final one in // the result request handling in order to decide if to send the @@ -420,15 +420,15 @@ handle (request& rq, response& rs) // build_db_->load (*b, b->results_section); - assert (b->state != build_state::tested && + assert (b->state != build_state::built && ((!b->forced && !b->status) || - (b->state == build_state::testing && + (b->state == build_state::building && (!b->forced || b->status))) && b->results.empty ()); - b->state = build_state::testing; + b->state = build_state::building; b->toolchain_name = move (tqm.toolchain_name); b->machine = mh.name; b->machine_summary = move (mh.summary); @@ -486,7 +486,7 @@ handle (request& rq, response& rs) if (x->forced != y->forced) return x->forced > y->forced; // Forced goes first. - assert (x->status && y->status); // Both tested. + assert (x->status && y->status); // Both built. if (x->status != y->status) return x->status > y->status; // Larger status goes first. @@ -501,7 +501,7 @@ handle (request& rq, response& rs) // Note that the configurations may not match the required criteria // anymore (as we have committed the database transactions that were // used to collect this data) so we recheck. If we find one that matches - // then put it into the testing state, refresh the timestamp and + // then put it into the building state, refresh the timestamp and // update. Note that we don't amend the status and the force flag to // have them available in the result request handling (see above). // @@ -513,7 +513,7 @@ handle (request& rq, response& rs) b = build_db_->find (b->id); - if (b != nullptr && b->state == build_state::tested && + if (b != nullptr && b->state == build_state::built && b->timestamp <= (b->forced ? forced_rebuild_expiration : normal_rebuild_expiration)) @@ -546,7 +546,7 @@ handle (request& rq, response& rs) { assert (b->status); - b->state = build_state::testing; + b->state = build_state::building; b->machine = mh.name; // Can't move from, as may need it on the next iteration. diff --git a/mod/mod-builds.cxx b/mod/mod-builds.cxx index adbd9bf..c831359 100644 --- a/mod/mod-builds.cxx +++ b/mod/mod-builds.cxx @@ -97,7 +97,7 @@ build_query (const C& configs, const brep::params::builds& params) query q ( query::id.configuration.in_range (configs.begin (), configs.end ()) && - (query::state == "testing" || query::state == "tested")); + (query::state == "building" || query::state == "built")); // Note that there is no error reported if the filter parameters parsing // fails. Instead, it is considered that no package builds match such a @@ -164,7 +164,7 @@ build_query (const C& configs, const brep::params::builds& params) if (rs == "pending") q = q && query::forced; else if (rs == "building") - q = q && query::state == "testing"; + q = q && query::state == "building"; else { query sq (query::status == rs); @@ -381,11 +381,11 @@ handle (request& rq, response& rs) << TD << SPAN(CLASS="value"); - if (b.state == build_state::testing) + if (b.state == build_state::building) s << "building"; else { - assert (b.state == build_state::tested); + assert (b.state == build_state::built); build_db_->load (b, b.results_section); -- cgit v1.1