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

eclib m4 script incorrectly checks version #38283

Open
2 tasks done
JohnCremona opened this issue Jun 26, 2024 · 0 comments · May be fixed by #38284
Open
2 tasks done

eclib m4 script incorrectly checks version #38283

JohnCremona opened this issue Jun 26, 2024 · 0 comments · May be fixed by #38284
Labels

Comments

@JohnCremona
Copy link
Member

Steps To Reproduce

Run ./configure with a version of eclib installed which is strictly greater than required, and with all of the prerequisites ntl, flint, pari not being built.

Expected Behavior

Configure should conclude that eclib does not need to be built.

Actual Behavior

Configure wrongly decides to build eclib.

Additional Information

In the file spkg-configure.m4 which was changed in #36879 to check for a version recent enough, rather than exactly equal to a specific version, two changes needed to be made but only one was. In line 5 of that file = needs to be >=.

(Thanks to Dima Pasechnik for the diagnosis.)

Environment

- **OS**: ubuntu 22.04
- **Sage Version**: 10.4.rc0

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
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.

1 participant