#26499 closed defect (fixed)
sagenb 1.1.0
Reported by:  dimpase  Owned by:  

Priority:  major  Milestone:  sage8.5 
Component:  packages: standard  Keywords:  
Cc:  fbissey  Merged in:  
Authors:  Dima Pasechnik  Reviewers:  François Bissey 
Report Upstream:  N/A  Work issues:  
Branch:  20b0980 (Commits)  Commit:  
Dependencies:  #24692, #25382  Stopgaps: 
Description (last modified by )
a routine update, incorporating most urgent PRs.
Also, in view of #25382
tarball: https://github.com/sagemath/sagenb/releases/download/1.1.0/sagenb1.1.0.tar.bz2
Change History (22)
comment:1 Changed 2 years ago by
comment:2 Changed 2 years ago by
 Status changed from new to needs_review
comment:3 followup: ↓ 5 Changed 2 years ago by
should we also do the downgrade to optional here?
You mean downgrading the spkg to optional? Is that actually possible already?
comment:4 followup: ↓ 6 Changed 2 years ago by
Could someone with python3 sage test this?
comment:5 in reply to: ↑ 3 Changed 2 years ago by
Replying to ghtimokau:
should we also do the downgrade to optional here?
You mean downgrading the spkg to optional? Is that actually possible already?
Sorry, no, not yet. I forgot about #25837.
comment:6 in reply to: ↑ 4 Changed 2 years ago by
Replying to dimpase:
Could someone with python3 sage test this?
There is no need to test with python3: I know that launching sagenb will fail when trying to import "email" module. I would say that py3compatibility of sagenb has never been closer, but there is still work, and no worker to handle the task..
comment:7 Changed 2 years ago by
 Status changed from needs_review to needs_work
I'm adding docs, please stand by...
comment:8 Changed 2 years ago by
 Cc fbissey added
comment:9 Changed 2 years ago by
 Description modified (diff)
 Status changed from needs_work to needs_review
 Summary changed from sagenb 1.0.4 to sagenb 1.1.0
comment:10 Changed 2 years ago by
 Branch set to u/dimpase/packages/sagenb110
 Commit set to 0f1fbf24fc301dc0afb7a0ce76b053c3b502ca2e
New commits:
7adac2c  update sagenb to version 1.0.4

059de77  trac 25382: remove the sagenb docs from the reference manual.

591e91a  trac 25382 remove remaining crosslinks

e2ff902  Merge branch 'public/25382' of trac.sagemath.org:sage into nb104

80beebb  Upgrade GDB

6769112  updated SPKG.txt to note the need for makeinfo

17ab2ce  Merge branch 'public/packages/optional/gdb82' of trac.sagemath.org:sage into nb104

0f1fbf2  added docs of sagenb

comment:11 Changed 2 years ago by
 Dependencies set to #24692, #25382
comment:12 Changed 2 years ago by
admittedly, docs are packed with few short autogenerated .rst files; if anyone is willing to tell me how to use sphinx to generate these, be my guest, I can fix this then.
comment:13 followup: ↓ 15 Changed 2 years ago by
I presume sagenb should now also depend on sphinx, there may be transitive dependency from something else but it is not visible.
comment:14 Changed 2 years ago by
 Commit changed from 0f1fbf24fc301dc0afb7a0ce76b053c3b502ca2e to 20b09806a536f6e6a585b9d226c38af58d24b216
Branch pushed to git repo; I updated commit sha1. New commits:
20b0980  sphinx must be a dependency

comment:15 in reply to: ↑ 13 Changed 2 years ago by
 Milestone changed from sage8.4 to sage8.5
Replying to fbissey:
I presume sagenb should now also depend on sphinx, there may be transitive dependency from something else but it is not visible.
Indeed. Thanks for pointing this out. Fixed in the last commit.
comment:16 Changed 2 years ago by
Do we go ahead with this or are we waiting for further changes?
comment:17 Changed 2 years ago by
Well, I am waiting for a review on this ticket... :)
comment:18 Changed 2 years ago by
 Reviewers set to François Bissey
 Status changed from needs_review to positive_review
Looks good to me then :)
comment:19 Changed 2 years ago by
 Branch changed from u/dimpase/packages/sagenb110 to 20b09806a536f6e6a585b9d226c38af58d24b216
 Resolution set to fixed
 Status changed from positive_review to closed
comment:20 Changed 2 years ago by
 Commit 20b09806a536f6e6a585b9d226c38af58d24b216 deleted
How was the "source tarball" https://github.com/sagemath/sagenb/releases/download/1.1.0/sagenb1.1.0.tar.bz2 created? It doesn't correspond with the github repo. In particular, the repo does not contain a bundled mathjax (as it should be), while the "source tarball" does.
Please comment at #26641.
comment:21 Changed 2 years ago by
I assume that the "source tarball" from this ticket was created from a nonclean checkout of the sagenb repo. I'll fix that on #26641.
should we also do the downgrade to optional here?