Skip to content

Conversation

maflcko
Copy link
Member

@maflcko maflcko commented Oct 1, 2017

Currently ./configure && make check will look for python3, then python2. As long as we support python2 (and use it as fallback), make check should run fine with both python2 and python3.

Fixes #11352 by @Zenitur

@maflcko maflcko added the Tests label Oct 1, 2017
@maflcko maflcko added this to the 0.15.1 milestone Oct 1, 2017
@laanwj
Copy link
Member

laanwj commented Oct 2, 2017

utACK fafff12

@jnewbery
Copy link
Contributor

jnewbery commented Oct 3, 2017

Tested ACK fafff12. Works with py2 and py3 for me.

Out of interest, what are the plans for deprecating py2 for make && make check? Obviously we can't include any of the functional tests for make check while we still support py2.

@maflcko
Copy link
Member Author

maflcko commented Oct 3, 2017

I am not against deprecating py2, but we should do it properly such that ./configure fails early and warns about it. Also it should be mentioned in the release notes.

@maflcko maflcko merged commit fafff12 into bitcoin:master Oct 3, 2017
maflcko pushed a commit that referenced this pull request Oct 3, 2017
fafff12 qa: Restore bitcoin-util-test py2 compatibility (MarcoFalke)

Pull request description:

  Currently `./configure && make check` will look for python3, then python2. As long as we support python2 (and use it as fallback), `make check` should run fine with both python2 and python3.

  Fixes #11352 by @Zenitur

Tree-SHA512: a335ebdd224328d6f924fe52a9b97de196926476c9ee04ce3280743ea93bcae355eb2d5d4bed4050c01b2e904105595eac7db2eaa9307207581caa0a98ebcc0b
@maflcko maflcko deleted the Mf1710-qaUtilPy2 branch October 3, 2017 19:48
maflcko pushed a commit to maflcko/bitcoin-core that referenced this pull request Oct 4, 2017
@maflcko maflcko mentioned this pull request Dec 12, 2017
codablock pushed a commit to codablock/dash that referenced this pull request Sep 24, 2019
fafff12 qa: Restore bitcoin-util-test py2 compatibility (MarcoFalke)

Pull request description:

  Currently `./configure && make check` will look for python3, then python2. As long as we support python2 (and use it as fallback), `make check` should run fine with both python2 and python3.

  Fixes bitcoin#11352 by @Zenitur

Tree-SHA512: a335ebdd224328d6f924fe52a9b97de196926476c9ee04ce3280743ea93bcae355eb2d5d4bed4050c01b2e904105595eac7db2eaa9307207581caa0a98ebcc0b
barrystyle pushed a commit to PACGlobalOfficial/PAC that referenced this pull request Jan 22, 2020
fafff12 qa: Restore bitcoin-util-test py2 compatibility (MarcoFalke)

Pull request description:

  Currently `./configure && make check` will look for python3, then python2. As long as we support python2 (and use it as fallback), `make check` should run fine with both python2 and python3.

  Fixes bitcoin#11352 by @Zenitur

Tree-SHA512: a335ebdd224328d6f924fe52a9b97de196926476c9ee04ce3280743ea93bcae355eb2d5d4bed4050c01b2e904105595eac7db2eaa9307207581caa0a98ebcc0b
@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.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Failed to make check on an old Linux: python3: no such file or directory
3 participants