Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pack: Restore behavior with no passed trace dirs #3842

Merged
merged 1 commit into from
Oct 2, 2024

Commits on Oct 1, 2024

  1. pack: Restore behavior with no passed trace dirs

    This is a bit of an embarassing bug-fix, but as pointed out in
    rr-debugger#3834 (comment),
    I accidentally broke `rr pack` without trace dir specified (which is
    supposed to pack the latest dir). We do actually have a test for it,
    which does fail for me locally (which I didn't notice - sorry),
    but not on CI. I'm not entirely sure why it doesn't fail on CI, but
    I imagine we might be copying more things in the CI environment, rather
    than cloning/linking them. To try to catch similar regressions in the future,
    this also adds an extra check to the test to make sure that there is
    some `mmap_pack_*` file in the trace after packing. I think we should
    always be packing at least libc or ld-linux, so hopefully there
    shouldn't be any false positives there.
    Keno committed Oct 1, 2024
    Configuration menu
    Copy the full SHA
    e67c1ec View commit details
    Browse the repository at this point in the history