Skip to content

compiletest: Use PYTHONPATH for lldb too, not only gdb#153810

Merged
rust-bors[bot] merged 1 commit intorust-lang:mainfrom
Enselic:lldb-pp
Mar 13, 2026
Merged

compiletest: Use PYTHONPATH for lldb too, not only gdb#153810
rust-bors[bot] merged 1 commit intorust-lang:mainfrom
Enselic:lldb-pp

Conversation

@Enselic
Copy link
Member

@Enselic Enselic commented Mar 13, 2026

That allows you to e.g. run debuginfo tests with lldb-15 on Ubuntu 24.04 like this:

sudo apt install lldb-15 python3-six
PYTHONPATH=/usr/lib/python3/dist-packages ./x --set build.lldb=lldb-15 test tests/debuginfo/basic-stepping.rs

See #153792 (comment) for context.

That allows you to e.g. run debuginfo tests with lldb-15 on Ubuntu 24.04 like this:

    sudo apt install lldb-15 python3-six
    PYTHONPATH=/usr/lib/python3/dist-packages ./x --set build.lldb=lldb-15 test tests/debuginfo/basic-stepping.rs
@rustbot
Copy link
Collaborator

rustbot commented Mar 13, 2026

Some changes occurred in src/tools/compiletest

cc @jieyouxu

@rustbot rustbot added A-compiletest Area: The compiletest test runner A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) labels Mar 13, 2026
@rustbot
Copy link
Collaborator

rustbot commented Mar 13, 2026

r? @jieyouxu

rustbot has assigned @jieyouxu.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @jieyouxu, @oli-obk, @wesleywiser, bootstrap
  • @jieyouxu, @oli-obk, @wesleywiser, bootstrap expanded to 8 candidates
  • Random selection from Mark-Simulacrum, clubby789, jieyouxu

Copy link
Member

@jieyouxu jieyouxu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jieyouxu
Copy link
Member

r=me once PR CI is green

@jieyouxu
Copy link
Member

@bors r+ rollup

@rust-bors
Copy link
Contributor

rust-bors bot commented Mar 13, 2026

📌 Commit bf631b3 has been approved by jieyouxu

It is now in the queue for this repository.

@rust-bors rust-bors bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 13, 2026
rust-bors bot pushed a commit that referenced this pull request Mar 13, 2026
…uwer

Rollup of 7 pull requests

Successful merges:

 - #153650 (Streamline active job collection.)
 - #153707 (Remove `CycleErrorHandling`.)
 - #153384 (Add missing safety doc for CString::from_vec_unchecked and async_drop_in_place)
 - #153752 (fix(delegation): Filter Out Module Segments in Generic Args Inheritance)
 - #153797 (Compiletest: Fix compare-output-by-lines directive)
 - #153810 (compiletest: Use PYTHONPATH for lldb too, not only gdb)
 - #153820 (Rename `opt_span_diag_lint` into `opt_span_lint` and remove `emit_diag_lint`)
@rust-bors rust-bors bot merged commit 341b42a into rust-lang:main Mar 13, 2026
11 checks passed
@rustbot rustbot added this to the 1.96.0 milestone Mar 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-compiletest Area: The compiletest test runner A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants