[Updated] D11813: rhg: don't run `blackbox` if not activated
Alphare (Raphaël Gomès)
phabricator at mercurial-scm.org
Tue Dec 7 14:44:36 UTC 2021
Closed by commit rHG9ff246cd6200: rhg: don't run `blackbox` if not activated (authored by Alphare).
This revision was automatically updated to reflect the committed changes.
This revision was not accepted when it landed; it landed in state "Needs Review".
REPOSITORY
rHG Mercurial
CHANGES SINCE LAST UPDATE
https://phab.mercurial-scm.org/D11813?vs=31137&id=31348
CHANGES SINCE LAST ACTION
https://phab.mercurial-scm.org/D11813/new/
REVISION DETAIL
https://phab.mercurial-scm.org/D11813
AFFECTED FILES
rust/rhg/src/main.rs
CHANGE DETAILS
diff --git a/rust/rhg/src/main.rs b/rust/rhg/src/main.rs
--- a/rust/rhg/src/main.rs
+++ b/rust/rhg/src/main.rs
@@ -110,18 +110,23 @@
}
}
- let blackbox = blackbox::Blackbox::new(&invocation, process_start_time)?;
- blackbox.log_command_start();
- let result = run(&invocation);
- blackbox.log_command_end(exit_code(
- &result,
- // TODO: show a warning or combine with original error if `get_bool`
- // returns an error
- config
- .get_bool(b"ui", b"detailed-exit-code")
- .unwrap_or(false),
- ));
- result
+ if config.is_extension_enabled(b"blackbox") {
+ let blackbox =
+ blackbox::Blackbox::new(&invocation, process_start_time)?;
+ blackbox.log_command_start();
+ let result = run(&invocation);
+ blackbox.log_command_end(exit_code(
+ &result,
+ // TODO: show a warning or combine with original error if
+ // `get_bool` returns an error
+ config
+ .get_bool(b"ui", b"detailed-exit-code")
+ .unwrap_or(false),
+ ));
+ result
+ } else {
+ run(&invocation)
+ }
}
fn main() {
To: Alphare, #hg-reviewers
Cc: mercurial-patches
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mercurial-scm.org/pipermail/mercurial-patches/attachments/20211207/37c7969c/attachment-0001.html>
More information about the Mercurial-patches
mailing list