Skip to content

Commit

Permalink
Trac #29923: Deprecate deprecated sage-cython script harder, remove u…
Browse files Browse the repository at this point in the history
…se of it in the main sage script

The script `sage-cython` has been deprecated for about 18 months, so
let's remove its use within sage.

URL: https://trac.sagemath.org/29923
Reported by: jhpalmieri
Ticket author(s): John Palmieri
Reviewer(s): Matthias Koeppe
  • Loading branch information
Release Manager committed Jul 10, 2020
2 parents 16c2328 + 648e327 commit 613162e
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
2 changes: 1 addition & 1 deletion src/bin/sage
Original file line number Diff line number Diff line change
Expand Up @@ -548,7 +548,7 @@ fi

if [ "$1" = "-cython" -o "$1" = '--cython' -o "$1" = '-pyrex' -o "$1" = "--pyrex" ]; then
shift
exec sage-cython "$@"
exec cython "$@"
fi

if [ "$1" = '-gap' -o "$1" = '--gap' ]; then
Expand Down
4 changes: 4 additions & 0 deletions src/bin/sage-cython
Original file line number Diff line number Diff line change
Expand Up @@ -4,11 +4,15 @@
# It is deprecated since Trac #27041 (Sage 8.7) because one should
# simply use "cython" directly. We display deprecation messages whenever
# "sage-cython" does something different from plain "cython".
#
# A stronger deprecation warning was added in #29923 (Sage 9.2).

import os
import sys
args = sys.argv[1:]

sys.stderr.write("WARNING: the script sage-cython is deprecated; use cython instead.\n")

from sage.env import SAGE_SRC

# args can have length 0, in case we're printing a usage message (see trac 12207)
Expand Down

0 comments on commit 613162e

Please sign in to comment.