-
-
Notifications
You must be signed in to change notification settings - Fork 510
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
update sagetex to version 3.6.1 #32887
Comments
Author: Dima Pasechnik |
New commits:
|
Commit: |
Branch: u/dimpase/packages/sagetex/v3.6 |
comment:2
And the 3.6 release is defective - it still identifies itself as 3.5 - https://github.com/sagemath/sagetex/blob/master/setup.py#L7 |
comment:3
Or, rather, what is on https://github.com/sagemath/sagetex/ does not match what is packaged up in the release tarball |
Branch pushed to git repo; I updated commit sha1. New commits:
|
comment:5
should all be fixed now, sorry. To much different metadata to tie up manually here... |
comment:7
What is the tarball doing on GitHub? |
comment:8
Replying to @mkoeppe:
there is something called sagetex.zip on CTAN - but the process of publishing there is manual, and I have not put this there yet. And I don't know anything about PyPI publishing. |
comment:9
Unless something has changed, we do not want it on CTAN, because that will not match versions with Sage. Dan always made sure that the versions matched (indeed, I think this is in the release instructions for SageTeX?) and so one has to manually move the .sty file, or did have to do. |
comment:10
Hmm, what do you mean by "we don't want it on CTAN"? Have you forgotten how we started putting in on CTAN few years ago? (And I did it for versions 3.4 and 3.5, and will do for 3.6) |
comment:11
Replying to @dimpase:
Have you looked into rever? I use it to create tarballs for autoconfiscated packages, release them on GitHub and on PyPI. See, e.g., https://github.com/flatsurf/flatsurf/blob/master/rever.xsh. |
comment:12
Apparently it has.
That is good news, and I hope it will keep track with the Sage versions well. |
comment:13
Replying to @kcrisman:
Karl, it's almost 2 years old news:
|
comment:14
I give up. I clearly can no longer keep 'Trac'. |
comment:15
Replying to @dimpase:
The package's Python code (apart from setup.py) is generated by TeX from a .dtx file.
Perhaps some metadata is missing, I don't know. There is no sagetex project on PyPI as far as I can tell. Let's get this in as is, and deal with a better distribution mechanism later. |
comment:17
Is sagetex actually used by sage? TBH I've managed to avoid it until now. |
comment:18
No, sagetex is a !Sage/Latex package. |
comment:136
|
comment:137
OK, that was pointed out in comment:41 already. |
Branch pushed to git repo; I updated commit sha1. New commits:
|
comment:141
Replying to @mkoeppe:
The centaur runs ... and passes ==> Thanks a lot ! |
comment:142
Does
pass? |
comment:143
Sorry for the wait (family obligations... ;-) Replying to @dimpase:
Yep ! HTH, |
comment:144
Thanks for the review! |
Reviewer: Emmanuel Charpentier |
Changed branch from u/mkoeppe/packages/sagetex/v3.6 to |
Previous update: #30342, merged 2020-08-20.
Depends on #34219
CC: @kcrisman @mkoeppe @orlitzky @slel @jhpalmieri @EmmanuelCharpentier
Component: packages: standard
Author: Dima Pasechnik, Matthias Koeppe
Branch/Commit:
add10cd
Reviewer: Emmanuel Charpentier
Issue created by migration from https://trac.sagemath.org/ticket/32887
The text was updated successfully, but these errors were encountered: