Compare commits

...

1 Commits

Author SHA1 Message Date
Adrian Bravo
72aae3df13 Clarify bubblewrap install warning 2026-04-06 14:46:23 -07:00
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]