#26641 closed defect (fixed)
Repackage sagenb
Reported by: | jdemeyer | Owned by: | |
---|---|---|---|
Priority: | blocker | Milestone: | sage-8.5 |
Component: | packages: standard | Keywords: | |
Cc: | fbissey, jhpalmieri, dimpase | Merged in: | |
Authors: | Jeroen Demeyer | Reviewers: | John Palmieri, Dima Pasechnik |
Report Upstream: | N/A | Work issues: | |
Branch: | 8824d93 (Commits, GitHub, GitLab) | Commit: | |
Dependencies: | #26642 | Stopgaps: |
Description (last modified by )
Since a while, sagenb is not supposed to contain a bundled mathjax. In particular, the github repo of sagenb does not contain mathjax.
Mysteriously, the sagenb packaged in #26499 does contain mathjax, but it's not at all clear where this comes from.
So this ticket fixes the packaging procedure of SageNB and makes a new tarball from essentially the same sources.
Tarball: https://github.com/sagemath/sagenb/releases/download/1.1.1/sagenb-1.1.1.tar.bz2
Change History (25)
comment:1 follow-up: ↓ 2 Changed 4 years ago by
comment:2 in reply to: ↑ 1 Changed 4 years ago by
Replying to dimpase:
Please fix this in sagenb repo.
The problem is not in the sagenb repo, it's in the packaging of the sagenb repo. So in order to make sure that this doesn't happen again, it would be good to know how you created the sagenb package on #26499.
I will release sagenb-1.1.1 from essentially the same sources as 1.1.0, packaged from a clean checkout of the sagenb repo, using the dist.sh
script.
comment:3 follow-up: ↓ 4 Changed 4 years ago by
Sagenb's GitHub repo contains a script to build the release. please fix it so that it doesn't do what you find wrong.
comment:4 in reply to: ↑ 3 ; follow-up: ↓ 5 Changed 4 years ago by
Replying to dimpase:
Sagenb's GitHub repo contains a script to build the release. please fix it so that it doesn't do what you find wrong.
That's hard to say since I don't know the exact steps to reproduce the bad sources at #26499. But I'll try: https://github.com/sagemath/sagenb/pull/462
comment:5 in reply to: ↑ 4 Changed 4 years ago by
Replying to jdemeyer:
That's hard to say since I don't know the exact steps to reproduce the bad sources at #26499.
Do you still have the sagenb checkout that you used for #26499? If so, could you run git status
there and show the output?
comment:6 Changed 4 years ago by
- Branch set to u/jdemeyer/repackage_sagenb
comment:7 follow-up: ↓ 8 Changed 4 years ago by
- Commit set to d78718449183c27cef07c98b06b512218e060bcf
comment:8 in reply to: ↑ 7 Changed 4 years ago by
Replying to dimpase:
No, I don't have a directory I packaged 1.1.0 in, anymore.
Can't one add an appropriate call to
git clean
to./dist.sh
to avoid such a mess in the future?
Yes, that is what I did in https://github.com/sagemath/sagenb/pull/462
comment:9 Changed 4 years ago by
- Commit changed from d78718449183c27cef07c98b06b512218e060bcf to 8a0e01741fe7b263e9c8360a33d55175ac1e6490
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
8a0e017 | Upgrade to sagenb version 1.1.1
|
comment:10 Changed 4 years ago by
This seems to be working now, but I'll wait on feedback about https://github.com/sagemath/sagenb/pull/462 before creating the final tarball.
comment:11 Changed 4 years ago by
- Description modified (diff)
comment:12 Changed 4 years ago by
- Description modified (diff)
comment:13 Changed 4 years ago by
- Commit changed from 8a0e01741fe7b263e9c8360a33d55175ac1e6490 to 8824d9344a32107ba56d718eca6b930164149807
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
8824d93 | Upgrade to sagenb version 1.1.1
|
comment:14 Changed 4 years ago by
- Status changed from new to needs_review
comment:15 follow-up: ↓ 16 Changed 4 years ago by
How does one deal with mathjax after this? Have you tested it works with sagenb and Jupiter (after installing something?)
I can't test it now myself, sitting in a ready to depart plane...
comment:16 in reply to: ↑ 15 Changed 4 years ago by
comment:17 follow-ups: ↓ 18 ↓ 24 Changed 4 years ago by
NOTE: the section of spkg-install for sagenb that builds sagenb documentation is very fragile, and is breaking the build of sage from scratch (not an incremental build).
I would rather remove that part.
comment:18 in reply to: ↑ 17 Changed 4 years ago by
Replying to chapoton:
NOTE: the section of spkg-install for sagenb that builds sagenb documentation is very fragile, and is breaking the build of sage from scratch (not an incremental build).
I would rather remove that part.
That should probably be a separate ticket. I would like to see more details.
comment:19 Changed 4 years ago by
This works for me: I get the correct behavior with sage -n
, sage -n sagenb
, and sage -n jupyter
. Both notebooks work for simple calculations – I didn't try anything very complicated.
comment:20 Changed 4 years ago by
- Reviewers set to John Palmieri
I'm willing to give this a positive review. Any objections?
comment:21 Changed 4 years ago by
- Reviewers changed from John Palmieri to John Palmieri, Dima Pasechnik
- Status changed from needs_review to positive_review
Lgtm
comment:22 Changed 4 years ago by
- Dependencies set to #26642
comment:23 Changed 4 years ago by
- Branch changed from u/jdemeyer/repackage_sagenb to 8824d9344a32107ba56d718eca6b930164149807
- Resolution set to fixed
- Status changed from positive_review to closed
comment:24 in reply to: ↑ 17 Changed 4 years ago by
- Commit 8824d9344a32107ba56d718eca6b930164149807 deleted
comment:25 Changed 4 years ago by
It should be the same, just missing dependencies in sagenb. Mea culpa - I think #26686 fixes this.
Please fix this in sagenb repo.