Skip to content

Commit 4047b35

Browse files
committed
Remove hostexec
This is a leftover from an original legacy when I was trying to support having bcvk itself be a container. And it's basically been dead code until now, but it actively breaks when trying to do nested podman. Signed-off-by: Colin Walters <[email protected]>
1 parent e170ed4 commit 4047b35

File tree

11 files changed

+36
-411
lines changed

11 files changed

+36
-411
lines changed

crates/kit/src/containerenv.rs

Lines changed: 0 additions & 57 deletions
This file was deleted.

crates/kit/src/envdetect.rs

Lines changed: 0 additions & 107 deletions
This file was deleted.

crates/kit/src/ephemeral.rs

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,14 @@
44
//! Ephemeral VMs are temporary, non-persistent VMs that are useful for testing, development,
55
//! and CI/CD workflows.
66
7+
use std::process::Command;
8+
79
use clap::Subcommand;
810
use color_eyre::{eyre::eyre, Result};
911
use comfy_table::{presets::UTF8_FULL, Table};
1012
use serde::{Deserialize, Serialize};
1113

1214
// Re-export the existing implementations
13-
use crate::hostexec;
1415
use crate::run_ephemeral;
1516
use crate::run_ephemeral_ssh;
1617
use crate::ssh;
@@ -160,7 +161,7 @@ impl EphemeralCommands {
160161
fn list_ephemeral_containers() -> Result<Vec<ContainerListEntry>> {
161162
use bootc_utils::CommandRunExt;
162163

163-
let containers: Vec<ContainerListEntry> = hostexec::command("podman", None)?
164+
let containers: Vec<ContainerListEntry> = Command::new("podman")
164165
.args([
165166
"ps",
166167
"--all",
@@ -214,7 +215,7 @@ fn remove_all_ephemeral_containers(force: bool) -> Result<()> {
214215
"Removing container {}",
215216
&container.id[..12.min(container.id.len())]
216217
);
217-
let result = hostexec::command("podman", None)?
218+
let result = Command::new("podman")
218219
.args(["rm", "-f", &container.id])
219220
.run();
220221

crates/kit/src/hostexec.rs

Lines changed: 0 additions & 166 deletions
This file was deleted.

0 commit comments

Comments
 (0)