use python3 print #848

pull fchapoton wants to merge 2 commits into bitcoin-core:master from fchapoton:patch-1 changing 1 files +16 −16

fchapoton commented at 10:28 AM on November 20, 2020: contributor

No description provided.
use python3 print cbb29e7e6d
fchapoton commented at 12:38 PM on November 20, 2020: contributor

sage is using now python3, since january 2020 and version 9.0. So this file is no longer working.
real-or-random commented at 7:22 PM on November 20, 2020: contributor

Thanks, looks good!

I noticed that secp256k1.sage does not work either with the current sage. This should also be easy to fix. Feel free to add a commit or indicate here that you're not willing to do this (then we can do it in another PR).
xrange to range for python3 compatibility cd9a9f1736
fchapoton commented at 8:22 PM on November 20, 2020: contributor

Conversion to python3 is not an easy task. It usually requires care from somebody knowing the code.
real-or-random cross-referenced this on Nov 21, 2020 from issue Convert Sage code to Python 3 (as used by Sage >= 9) by real-or-random
jonasnick commented at 8:13 PM on November 23, 2020: contributor

Merged as part of #849
jonasnick closed this on Nov 23, 2020
fchapoton deleted the branch on Nov 23, 2020

Contributors