Opened 3 years ago

Closed 3 years ago

Last modified 2 years ago

#27684 closed enhancement (fixed)

Upgrade dot2tex to 2.11.3

Reported by: tscrim Owned by:
Priority: major Milestone: sage-8.8
Component: packages: optional Keywords: upgrade, dot2tex
Cc: jdemeyer, embray, chapoton, nthiery, aschilling, bsalisbury01 Merged in:
Authors: Travis Scrimshaw Reviewers: Anne Schilling, Nicolas M. Thiéry
Report Upstream: N/A Work issues:
Branch: 7add7a6 (Commits, GitHub, GitLab) Commit:
Dependencies: Stopgaps:

Status badges

Description

A straightforward upgrade, which includes support for Python3. I also fix something in spkg-check to use the Python installed by Sage.

tarball

Change History (11)

comment:1 Changed 3 years ago by tscrim

  • Branch set to public/packages/upgrade_dot2tex_2_11_3-27684
  • Commit set to 7add7a663481f22cea133b3b6905ce0cd94b2481
  • Status changed from new to needs_review

New commits:

7add7a6Upgrade dot2tex to 2.11.3 and additional changes for Python3.

comment:2 Changed 3 years ago by nthiery

I haven't tested it yet, but otherwise this looks fine to me! Thanks Travis. I got hit by that lack of support for Python3 two days ago!

comment:3 Changed 3 years ago by tscrim

I tested this both on Python2 and 3 (in part because I wrote the dot2tex port PR). I completely forgot to do this upgrade ticket until today when I was upgrading my python3 build. :P Sorry it was a little late for you.

comment:4 Changed 3 years ago by aschilling

Could you give some instructions on how to test your changes? Get your branch and then install the tarball?

comment:5 follow-up: Changed 3 years ago by tscrim

You will need to get the branch and download the tarball from the link in the description, putting it in the $SAGE_ROOT/upstream folder. Then just run make build or sage -i dot2tex. You can also run sage -i -c dot2tex if you want to run the spkg's test suite.

comment:6 in reply to: ↑ 5 Changed 3 years ago by aschilling

Replying to tscrim:

You will need to get the branch and download the tarball from the link in the description, putting it in the $SAGE_ROOT/upstream folder. Then just run make build or sage -i dot2tex. You can also run sage -i -c dot2tex if you want to run the spkg's test suite.

Ok, thanks! Now that I could finally build sage again, I tried out your updated dot2tex package and it seems to work great.

comment:7 Changed 3 years ago by aschilling

  • Reviewers set to Anne Schilling, Nicolas Thiery
  • Status changed from needs_review to positive_review

comment:8 Changed 3 years ago by tscrim

Thank you.

comment:9 Changed 3 years ago by vbraun

  • Branch changed from public/packages/upgrade_dot2tex_2_11_3-27684 to 7add7a663481f22cea133b3b6905ce0cd94b2481
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:10 Changed 2 years ago by tmonteil

  • Commit 7add7a663481f22cea133b3b6905ce0cd94b2481 deleted

Followup : #28332

Please also run self-tests when upgrading a package.

comment:11 Changed 2 years ago by slelievre

  • Reviewers changed from Anne Schilling, Nicolas Thiery to Anne Schilling, Nicolas M. Thiéry
Note: See TracTickets for help on using tickets.