Skip to content
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

Upgrade PARI to 2.7.3 #16939

Closed
jdemeyer opened this issue Sep 5, 2014 · 15 comments
Closed

Upgrade PARI to 2.7.3 #16939

jdemeyer opened this issue Sep 5, 2014 · 15 comments

Comments

@jdemeyer
Copy link

jdemeyer commented Sep 5, 2014

Download http://pari.math.u-bordeaux.fr/pub/pari/snapshots/pari-2.7.2-pre2.tar.gz and rename to $SAGE_ROOT/upstream/pari-2.7.2.tar.gz.

Alternative: #16997

Depends on #15767
Depends on #16894

CC: @pjbruin @JohnCremona

Component: packages: standard

Branch/Commit: u/jdemeyer/ticket/16939 @ d9214c8

Reviewer: Jeroen Demeyer

Issue created by migration from https://trac.sagemath.org/ticket/16939

@jdemeyer jdemeyer added this to the sage-6.4 milestone Sep 5, 2014
@jdemeyer
Copy link
Author

jdemeyer commented Sep 8, 2014

Author: Jeroen Demeyer

@jdemeyer

This comment has been minimized.

@jdemeyer
Copy link
Author

jdemeyer commented Sep 8, 2014

Branch: u/jdemeyer/ticket/16939

@jdemeyer

This comment has been minimized.

@jdemeyer
Copy link
Author

Commit: ce51a08

@jdemeyer
Copy link
Author

New commits:

ce51a08Upgrade to PARI 2.7.2

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Sep 16, 2014

Changed commit from ce51a08 to d9214c8

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Sep 16, 2014

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

d9214c8Upgrade to PARI 2.7.2

@jdemeyer

This comment has been minimized.

@pjbruin
Copy link
Contributor

pjbruin commented Sep 25, 2014

comment:6

PARI 2.7.2 was released last week. If you want to finish this straightforward upgrade, I will be happy to review it; I don't currently want to spend a lot of time on the alternative #16997, especially because of the new patches on that ticket and the related upstream issues.

@jdemeyer
Copy link
Author

comment:7

Replying to @pjbruin:

PARI 2.7.2 was released last week. If you want to finish this straightforward upgrade, I will be happy to review it;

I see no reason to finish this ticket: it doesn't solve any problem we're currently having and I prefer to put efforts towards #16997 instead.

@jdemeyer
Copy link
Author

comment:8

Replying to @pjbruin:

I don't currently want to spend a lot of time on the alternative #16997, especially because of the new patches on that ticket and the related upstream issues.

I agree that is a very annoying situation. However, ignoring that problem isn't going to make it go away. Perhaps a discussion on sage-devel is in order about the "PARI situation".

@sagetrac-tmonteil
Copy link
Mannequin

sagetrac-tmonteil mannequin commented Mar 5, 2015

comment:9

Since the alternative is merged, should we close this ticket ?

@sagetrac-tmonteil sagetrac-tmonteil mannequin removed this from the sage-6.4 milestone Mar 5, 2015
@jdemeyer
Copy link
Author

jdemeyer commented Mar 6, 2015

Reviewer: Jeroen Demeyer

@jdemeyer
Copy link
Author

jdemeyer commented Mar 6, 2015

Changed author from Jeroen Demeyer to none

@jdemeyer jdemeyer changed the title Upgrade PARI to 2.7.2 when it's released Upgrade PARI to 2.7.3 Mar 6, 2015
@vbraun vbraun closed this as completed Mar 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants