Python 3 preparation: Change backticks into repr()

[wluebbe]

Backticks are deprecated since long ago and not accepted by Python 3.

Changes according to lib2to3/fixes/fix_repr.py:

transforms `xyzzy` into repr(xyzzy)

This ticket is tracked as a dependency of meta-ticket ticket:15980.

Component: distribution

Author: Wilfried Luebbe

Branch/Commit: 7daa857

Reviewer: Frédéric Chapoton, Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/15992

[wluebbe]

comment:1

2to3 changed 4 affected .py modules.

I have not found a way to identify the relevant backtick usages (too many backticks in comments and doc-test):

• .pyx modules may be affected. But cython is said to be Python 3 enabled, so that should be no problem.
• If there backticks in doc-tests they will be found during tests in a Python 3 environment.

---

New commits:

289b346

changes the deprecated backticks usage to repr() function calls

[wluebbe]

Branch: u/wluebbe/ticket/15992

[wluebbe]

Commit: 289b346

[wluebbe]

comment:2

Apparently the automerge fails. How to locate the conflicting parts?

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

8c99a20

changes the deprecated backticks usage to repr() function calls

[sagetrac-git]

Changed commit from 289b346 to 8c99a20

[wluebbe]

comment:4

branch was rebased to resolve merge conflict

[wluebbe]

comment:5

I did

git merge u/chapoton/15991 u/wluebbe/ticket/15986 u/wluebbe/ticket/15987

on top of u/wluebbe/ticket/15992 (which is rebased on 6.2.beta5)

Test report:

./sage -t -p --all --long --logfile=logs/ptestlong-alle4.log
...
All tests passed!

[fchapoton]

Reviewer: Frédéric Chapoton

[fchapoton]

Changed branch from u/wluebbe/ticket/15992 to public/ticket/15992

[fchapoton]

Changed commit from 8c99a20 to 7daa857

[fchapoton]

comment:6

Here is a reviewer branch. I have taken the opportunity to make small changes in the close nieghborhood of the modified code, and to add some doctests.

If you agree with my changes, you can set this to positive review.

---

New commits:

7daa857

trac #15992 reviewer patch

[tscrim]

comment:7

Eeek, I'm surprised I've never noticed that not raising an error message in NCSF. I agree with the changes, so positive review.

[tscrim]

Author: Wilfried Luebbe

[tscrim]

Changed reviewer from Frédéric Chapoton to Frédéric Chapoton, Travis Scrimshaw

[vbraun]

Changed branch from public/ticket/15992 to 7daa857