Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#13598 closed enhancement (fixed)

'x' should be defined when using 'sage -c'

Reported by: jhpalmieri Owned by: leif
Priority: major Milestone: sage-5.4.1
Component: scripts Keywords:
Cc: kcrisman Merged in: sage-5.4.1.rc0
Authors: John Palmieri Reviewers: Punarbasu Purkayastha
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by jhpalmieri)

Sage currently doesn't defined the variable 'x' when using sage -c:

$ sage -c "print x"
Traceback (most recent call last):
  File "/Users/palmieri/Desktop/Sage_stuff/sage_builds/sage-5.4.rc1/local/bin/sage-eval", line 13, in <module>
    eval(compile(s,'<cmdline>','exec'))
  File "<cmdline>", line 1, in <module>
NameError: name 'x' is not defined

We should fix this.

Apply trac_13598-x.patch to the scripts repository.

Attachments (1)

trac_13598-x.patch (498 bytes) - added by jhpalmieri 8 years ago.
scripts repo

Download all attachments as: .zip

Change History (6)

Changed 8 years ago by jhpalmieri

scripts repo

comment:1 Changed 8 years ago by jhpalmieri

  • Description modified (diff)
  • Status changed from new to needs_review

comment:2 Changed 8 years ago by jhpalmieri

The line

from sage.calculus.predefined import x

is present in both all_cmdline.py and all_notebook.py, so it is executed during any normal Sage session.

comment:3 Changed 8 years ago by ppurka

  • Reviewers set to Punarbasu Purkayastha
  • Status changed from needs_review to positive_review

Looks good to me. Maybe this was just an oversight earlier?

comment:4 Changed 8 years ago by jdemeyer

  • Merged in set to sage-5.5.beta0
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:5 Changed 8 years ago by jdemeyer

  • Merged in changed from sage-5.5.beta0 to sage-5.4.1.rc0
  • Milestone changed from sage-5.5 to sage-5.4.1
Note: See TracTickets for help on using tickets.