Clarify bubblewrap install warning

This commit is contained in:
Adrian Bravo
2026-04-06 14:46:23 -07:00
parent 03c07956cf
commit 72aae3df13
2 changed files with 3 additions and 1 deletions

View File

@@ -23,7 +23,7 @@ fn system_bwrap_warning_for_lookup(system_bwrap_path: Option<PathBuf>) -> Option
match system_bwrap_path {
Some(_) => None,
None => Some(
"Codex could not find system bubblewrap on PATH. Please install bubblewrap with your package manager. Codex will use the vendored bubblewrap in the meantime."
"Codex could not find system bubblewrap on PATH. Please install the Bubblewrap system package with your OS package manager (for example apt, dnf, or pacman), not npm. Upstream project: https://github.com/containers/bubblewrap. Codex will use the vendored bubblewrap in the meantime."
.to_string(),
),
}

View File

@@ -10,6 +10,8 @@ fn system_bwrap_warning_reports_missing_system_bwrap() {
.expect("missing system bwrap should emit a warning");
assert!(warning.contains("could not find system bubblewrap"));
assert!(warning.contains("not npm"));
assert!(warning.contains("https://github.com/containers/bubblewrap"));
}
#[test]