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

external Pari 2.17 does not work (yet) #38769

Open
2 tasks done
dimpase opened this issue Oct 4, 2024 · 3 comments · May be fixed by #38772
Open
2 tasks done

external Pari 2.17 does not work (yet) #38769

dimpase opened this issue Oct 4, 2024 · 3 comments · May be fixed by #38772
Labels

Comments

@dimpase
Copy link
Member

dimpase commented Oct 4, 2024

Steps To Reproduce

If Pari 2.17 is installed system-wide, it could be picked up, but doesn't work.

Expected Behavior

Sage should not use Pari 2.17

Actual Behavior

it does

Additional Information

it's a 1-line fix in pari's spkg-configure.m4
Ban versions $\geq 135424$

Environment

  • OS: macOS 15.0.1 + Homebrew
  • Sage Version: all

Checklist

  • I have searched the existing issues for a bug report that matches the one I want to file, without success.
  • I have read the documentation and troubleshoot guide
@dimpase dimpase added the t: bug label Oct 4, 2024
@jhpalmieri
Copy link
Member

jhpalmieri commented Oct 5, 2024

Related: #38749 (upgrade pari to 2.17). It's not clear to me how close that is to being ready, though.

@antonio-rojas
Copy link
Contributor

Related: #38749 (upgrade pari to 2.17). It's not clear to me how close that is to being ready, though.

The sagelib part is mostly ready (I just need to push some fixes for test failures caused by different ideal and unit group generators). However, that's just a small part of the story: most of the breakage is in cypari, which would need a new release including sagemath/cypari2#165 and sagemath/cypari2#166. And, to make things worse, 2.17.0 ships with a bug that breaks cypari build. So 2.17.0 should probably be unconditionally disallowed, and once 2.17.1 is released we should upgrade pari and cypari in sage.

@jhpalmieri jhpalmieri linked a pull request Oct 5, 2024 that will close this issue
5 tasks
@jhpalmieri
Copy link
Member

it's a 1-line fix in pari's spkg-configure.m4 Ban versions ≥ 135424

Not surprisingly, this works well, fixes the problem for me. I've opened a PR that makes this change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants