contrib: Fix capture_output in getcoins.py #26212

1. 
   willcl-ark commented at 12:21 PM on September 30, 2022: contributor

   Our minimum required Python version 3.6.12 does not support capture_output as a subprocess.run argument; this was added in python 3.7.

   We can emulate it by setting both stdout and stderr to subprocess.PIPE

2. contrib: Fix capture_output in getcoins.py

   Our required Python version 3.6.12 does not support `capture_output` as
   a subprocess.run argument; this was added in python 3.7.
   
   We can emulate it by setting stdout and stderr to subprocess.PIPE

   be59bd17ec
3. fanquake added the label Scripts and tools on Sep 30, 2022
4. 
   brunoerg commented at 6:53 PM on September 30, 2022: contributor

   Concept ACK

   Python 3.6: https://docs.python.org/3.6/library/subprocess.html#subprocess.run Python 3.7 (contains capture_output): https://docs.python.org/3.6/library/subprocess.html#subprocess.run

5. willcl-ark cross-referenced this on Sep 30, 2022 from issue v24.0 testing by fanquake
6. maflcko added this to the milestone 24.0 on Oct 2, 2022
7. maflcko added the label Needs backport (24.x) on Oct 2, 2022
8. maflcko merged this on Oct 2, 2022
9. maflcko closed this on Oct 2, 2022

   ---

10. willcl-ark deleted the branch on Oct 2, 2022
11. 
    maflcko commented at 3:18 PM on October 2, 2022: member

    Marked for backport and reverted in https://github.com/bitcoin/bitcoin/pull/26226

12. fanquake referenced this in commit 1db89aa9cc on Oct 2, 2022
13. fanquake cross-referenced this on Oct 2, 2022 from issue [24.x] Backports for rc2 by fanquake
14. fanquake removed the label Needs backport (24.x) on Oct 2, 2022
15. 
    fanquake commented at 3:42 PM on October 2, 2022: member

    Added to #26133 for backport to 24.x.

16. fanquake referenced this in commit 997faf6b6c on Oct 11, 2022
17. achow101 referenced this in commit 885366c67a on Oct 13, 2022
18. bitcoin locked this on Oct 13, 2023

Contributors

willcl-arkbrunoergmaflckofanquake

Milestone
24.0