ci: Avoid cd into build dir #32880

pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2507-ci-less-cd changing 1 files +27 −8
  1. maflcko commented at 1:08 pm on July 5, 2025: member

    Changing into the build dir is confusing and brittle, because the following commands implicitly assume it. So they could break on unrelated changes.

    The changes are required for stuff like:

    So remove the cd and just make the build dir explicit.

  2. DrahtBot added the label Tests on Jul 5, 2025
  3. DrahtBot commented at 1:08 pm on July 5, 2025: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32880.

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #31349 (ci: detect outbound internet traffic generated while running tests by vasild)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  4. ci: Avoid cd into build dir
    Changing into the build dir is confusing and brittle.
    
    This can be reviewed using the git option `--word-diff-regex=.`.
    faeae97b7f
  5. maflcko force-pushed on Jul 5, 2025
  6. DrahtBot added the label CI failed on Jul 5, 2025
  7. DrahtBot removed the label CI failed on Jul 5, 2025


maflcko DrahtBot

Labels
Tests


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2025-07-07 21:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me