Opened 10 years ago
Closed 10 years ago
#13794 closed defect (fixed)
sage n is broken in 5.5rc0
Reported by:  Punarbasu Purkayastha  Owned by:  jason, mpatel, was 

Priority:  blocker  Milestone:  sage5.5 
Component:  notebook  Keywords:  
Cc:  Merged in:  sage5.5.rc1  
Authors:  Jeroen Demeyer, John Palmieri  Reviewers:  Punarbasu Purkayastha 
Report Upstream:  N/A  Work issues:  
Branch:  Commit:  
Dependencies:  Stopgaps: 
Description (last modified by )
See this sagedevel thread. I should have opened the bug report here. This is a clear blocker for 5.5.
Apply trac_13794scripts.patch to the scripts repo.
Attachments (1)
Change History (10)
comment:1 Changed 10 years ago by
comment:3 Changed 10 years ago by
Replying to jhpalmieri:
See also #11409, in particular the scripts patch.
I remember having tried only the scripts patch. It didn't work unfortunately. Only the error message changed.
comment:4 Changed 10 years ago by
Maybe we also need this change to sagenotebook
and sagenotebookinsecure
:

sagenotebook
diff git a/sagenotebook b/sagenotebook
a b print open(os.environ['SAGE_ROOT'] + '/l 6 6 7 7 print "Please wait while the Sage Notebook server starts..." 8 8 9 from sage.all import * 9 10 from sagenb.notebook.all import notebook 10 11 11 12 print "..."
comment:5 Changed 10 years ago by
Authors:  → Jeroen Demeyer, John Palmieri 

Status:  new → needs_review 
comment:6 Changed 10 years ago by
Ok. This patch works by itself with the nb in 5.5rc0. Thanks for the fix.
Can you do the following:
 change the name of the patch  it has the wrong ticket number.
 change the commit message.
I would give it a positive review otherwise.
comment:8 Changed 10 years ago by
Reviewers:  → Punarbasu Purkayastha 

Status:  needs_review → positive_review 
Thanks.
comment:9 Changed 10 years ago by
Merged in:  → sage5.5.rc1 

Resolution:  → fixed 
Status:  positive_review → closed 
Note: See
TracTickets for help on using
tickets.
Replying to ppurka:
... or posted to sagerelease in the first place ... ;)