Opened 9 years ago
Closed 9 years ago
#13794 closed defect (fixed)
sage n is broken in 5.5rc0
Reported by:  ppurka  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 in reply to: ↑ description Changed 9 years ago by
comment:2 followup: ↓ 3 Changed 9 years ago by
See also #11409, in particular the scripts patch.
comment:3 in reply to: ↑ 2 Changed 9 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 9 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 9 years ago by
 Status changed from new to needs_review
comment:6 Changed 9 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 9 years ago by
 Reviewers set to Punarbasu Purkayastha
 Status changed from needs_review to positive_review
Thanks.
comment:9 Changed 9 years ago by
 Merged in set to sage5.5.rc1
 Resolution set to fixed
 Status changed from positive_review to closed
Replying to ppurka:
... or posted to sagerelease in the first place ... ;)