diff options
author | fanquake <fanquake@gmail.com> | 2020-01-06 15:18:10 +0800 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2020-01-06 15:46:26 +0800 |
commit | e8e79958a7b2a0bf1b02adcce9f4d811eac37dfc (patch) | |
tree | 5ea0ec35eb2eda01a010ba22a55c1ddf09300682 /CONTRIBUTING.md | |
parent | b949ac9697a6cfe087f60a16c063ab9c5bf1e81f (diff) | |
parent | 71af793512100ee7d508c3fb815af47925fe80ba (diff) |
Merge #17857: scripts: fix symbol-check & security-check argument passing
71af793512100ee7d508c3fb815af47925fe80ba scripts: fix check-symbols & check-security argument passing (fanquake)
Pull request description:
The first argument in `bin_PROGRAMS` (`bitcoind`) was being silently consumed and never passed into the [`security-check.py`](https://github.com/bitcoin/bitcoin/blob/master/contrib/devtools/security-check.py) or [`symbol-check.py`](https://github.com/bitcoin/bitcoin/blob/master/contrib/devtools/symbol-check.py) scripts.
This seems to have been the case since the scripts were added to the makefile in https://github.com/bitcoin/bitcoin/commit/f3d3eaf78eb51238d799d8f20a585550d1567719.
Example of the behavior:
```python
# touch a, touch b, touch c
# python3 args.py < a b c
import sys
if __name__ == '__main__':
print(sys.argv)
# ['args.py', 'b', 'c']
# if you add some lines to "a",
# you'll see them here..
for line in sys.stdin:
print(line)
```
ACKs for top commit:
laanwj:
ACK 71af793512100ee7d508c3fb815af47925fe80ba
Tree-SHA512: 9d0b975a11f66fd87a76654d210808000a629c9cce4c760f71e8a2bcb4e99b9109419f2306db67cf9b12c28e40b96ae722b7c9b4569b2b8bacd469fb99db30c3
Diffstat (limited to 'CONTRIBUTING.md')
0 files changed, 0 insertions, 0 deletions