ok/jj
1
0
Fork 0
forked from mirrors/jj
Code Pull requests Activity

cli: insert tracing at editor/pager invocation

Browse source 
This should help debug Windows mess.
This commit is contained in:
Yuya Nishihara 2024-05-12 09:38:22 +09:00
parent e3b3b5586f
commit db75e19751
2 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
cli/src/cli_util.rs
View file

@ -2170,7 +2170,10 @@ pub fn run_ui_editor(settings: &UserSettings, edit_path: &PathBuf) -> Result<(),
.config()
.get("ui.editor")
.map_err(|err| config_error_with_message("Invalid `ui.editor`", err))?;
let exit_status = editor.to_command().arg(edit_path).status().map_err(|err| {
let mut cmd = editor.to_command();
cmd.arg(edit_path);
tracing::info!(?cmd, "running editor");
let exit_status = cmd.status().map_err(|err| {
user_error_with_message(
format!(
// The executable couldn't be found or run; command-line arguments are not relevant

4
cli/src/ui.rs
View file

@ -105,7 +105,9 @@ impl UiOutput {
}
fn new_paged(pager_cmd: &CommandNameAndArgs) -> io::Result<UiOutput> {
let mut child = pager_cmd.to_command().stdin(Stdio::piped()).spawn()?;
let mut cmd = pager_cmd.to_command();
tracing::info!(?cmd, "spawning pager");
let mut child = cmd.stdin(Stdio::piped()).spawn()?;
let child_stdin = child.stdin.take().unwrap();
Ok(UiOutput::Paged { child, child_stdin })
}