Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Nov 16, 2015

Follow up of #6998?

@laanwj
Copy link
Member

laanwj commented Nov 17, 2015

  • Deleting the outdated test-patches: trivial ACK
  • Moving gen_base58_test_vectors.py: are these (conditionally) invoked anywhere in the build system? (if so, they belong in share instead, dev scripts aren't currently part of the tarball)

@maflcko maflcko force-pushed the MarcoFalke-2015-contribC branch from f07daba to 013a364 Compare November 17, 2015 22:16
@maflcko maflcko changed the title [contrib] Delete test-patches and move testgen to devtools [contrib] Delete test-patches Nov 17, 2015
@maflcko
Copy link
Member Author

maflcko commented Nov 17, 2015

@laanwj not within bitcoin but this would break external (hot)links. So better leaving this as is.

@laanwj laanwj merged commit 013a364 into bitcoin:master Nov 18, 2015
laanwj added a commit that referenced this pull request Nov 18, 2015
013a364 [contrib] Delete test-patches (MarcoFalke)
@maflcko maflcko deleted the MarcoFalke-2015-contribC branch November 18, 2015 08:52
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
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