subprocess: Backport upstream changes #32567

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:250520-subprocess-backports changing 1 files +6 −1
  1. hebasto commented at 10:56 am on May 20, 2025: member

    A list of the backported PRs:

    The following PRs were skipped for backporting:

    Required for #32566.

  2. DrahtBot commented at 10:56 am on May 20, 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/32567.

    Reviews

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

  3. hebasto commented at 10:56 am on May 20, 2025: member
    cc @laanwj
  4. hebasto force-pushed on May 20, 2025
  5. fanquake commented at 11:03 am on May 20, 2025: member
    Can’t this just go in with #32566? Why does it need it’s own PR?
  6. subprocess: Don't add an extra whitespace at end of Windows command line
    The windows code adds an unnecessary extra space to the command line.
    This can cause subtle issues, so avoid it.
    
    Github-Pull: arun11299/cpp-subprocess#119
    Rebased-From: 777cfa77d1f84bb08b3e445d5f7fc6c87282223b
    e63a7034f0
  7. hebasto force-pushed on May 20, 2025
  8. hebasto commented at 11:11 am on May 20, 2025: member

    Can’t this just go in with #32566? Why does it need it’s own PR?

    It would be easier to track backports.


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-05-20 18:12 UTC

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