Skip to content

Commit 4474668

Browse files
Rollup merge of #156769 - fallofpheonix:fix-path-lldb-macos, r=jieyouxu
Use print instead of po in debuginfo path test Fixes #156660 This fixes `tests/debuginfo/path.rs` failing on newer Apple LLDB versions. Root cause: Apple LLDB 2100 reports missing Rust language plugin support: warning: `po` was unsuccessful, running `p` instead As a result, `po` falls back to `p`, producing expanded output that no longer matches the existing `lldb-check` directives. Changes: - replace `po` with `print` - relax `lldb-check` expectations using `[...]` - preserve validation of the displayed path value This also aligns `path.rs` with the rest of the debuginfo tests, as it appears to be the only test using `po`. --- EDIT(@jieyouxu): see digging #156769 (comment)
2 parents e1ff77d + 6ed5bf2 commit 4474668

1 file changed

Lines changed: 0 additions & 4 deletions

File tree

tests/debuginfo/path.rs

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,12 +8,8 @@
88

99
//@ lldb-command:print pathbuf
1010
//@ lldb-check:[...] "/some/path" { inner = "/some/path" { inner = { inner = size=10 { [0] = '/' [1] = 's' [2] = 'o' [3] = 'm' [4] = 'e' [5] = '/' [6] = 'p' [7] = 'a' [8] = 't' [9] = 'h' } } } }
11-
//@ lldb-command:po pathbuf
12-
//@ lldb-check:"/some/path"
1311
//@ lldb-command:print path
1412
//@ lldb-check:[...] "/some/path" { data_ptr = [...] length = 10 }
15-
//@ lldb-command:po path
16-
//@ lldb-check:"/some/path"
1713

1814
use std::path::Path;
1915

0 commit comments

Comments
 (0)