Skip to content

Conversation

hebasto
Copy link
Member

@hebasto hebasto commented Mar 13, 2021

  1. Test source and executable files are neither ignored by .gitignore nor removed by make clean and make distclean.

  2. The get_machine function is no longer used since contrib: run test-symbol-check for RISC-V #21255.

@DrahtBot
Copy link
Contributor

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

Conflicts

Reviewers, this pull request conflicts with the following ones:

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.

@DrahtBot
Copy link
Contributor

🕵️ @fanquake has been requested to review this pull request as specified in the REVIEWERS file.

Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 0fc0c00

@fanquake fanquake merged commit a65e772 into bitcoin:master Mar 18, 2021
@hebasto hebasto deleted the 210313-check branch March 18, 2021 10:52
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Mar 18, 2021
0fc0c00 test: Drop unused get_machine function (Hennadii Stepanov)
61a0f8f test: Cleanup test files in test-{security,symbol}-check.py (Hennadii Stepanov)

Pull request description:

  1) Test source and executable files are neither ignored by `.gitignore` nor removed by `make clean` and `make distclean`.

  2) The `get_machine` function is no longer used since bitcoin#21255.

ACKs for top commit:
  fanquake:
    ACK 0fc0c00

Tree-SHA512: ef3fcf22d4a04b6e4f37f748bd4be57e09696d2a77982e26292843cb2a1297789c8325f5c4bdad37d8094fce7765c4cc9ab19809e07471487943361b2b1a252c
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 18, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants