Fix python usage for arch's broken maintainers #5384

pull TheBlueMatt wants to merge 1 commits into bitcoin:master from TheBlueMatt:python2 changing 14 files +14 −14

TheBlueMatt commented at 5:04 AM on November 27, 2014: member

arch's python maintainers dont bother to follow https://www.python.org/dev/peps/pep-0394/, so you have to use python2 on arch systems to get tests to run...This should work fine on every Linux system Ive ever seen, though I'd like to hear from someone at least on OSX about whether this work for them.
Fix python usage for arch's broken maintainers e3f315a0a5
fanquake commented at 6:56 AM on November 27, 2014: member

Builds an runs fine. OSX 10.9.5
TheBlueMatt referenced this in commit 07c92aa881 on Nov 27, 2014
laanwj commented at 8:12 AM on November 27, 2014: member

If we merge this, ideally we should also set up travis to not have python but only python2, so that future scripts that are added are automatically checked for this @theuni.

Edit: ACK, this is trivially correct.
laanwj added the label Tests on Nov 27, 2014
luke-jr commented at 8:15 AM on November 27, 2014: member

utACK, the PEP also says scripts should use 'python2' and not 'python' as implemented in this PR
sipa commented at 10:19 AM on November 27, 2014: member

utACK
laanwj merged this on Nov 27, 2014
laanwj closed this on Nov 27, 2014
laanwj referenced this in commit faa1623a44 on Nov 27, 2014
TheBlueMatt deleted the branch on Dec 1, 2014
DrahtBot locked this on Sep 8, 2021

Contributors

Labels