Skip to content

Conversation

@tornaria
Copy link
Contributor

ecl 23.9.9 has two minor changes that affect sagemath:

  • api for bignums changed, so the macro ecl_mpz_from_bignum has to be (conditionally) changed
  • printing of floating point numbers changed 1.e40 to 1.0e40, I changed the example to 1.234e40 which prints the same in 23.9.9 as in previous versions.

With this changes sagemath should work ok with either 21.2.1 or 23.9.9.

@github-actions
Copy link

Documentation preview for this PR (built with commit 26f1b72; changes) is ready! 🎉

@tornaria tornaria requested a review from jhpalmieri September 11, 2023 16:07
@jhpalmieri
Copy link
Member

Should there by any tests in spkg-configure.m4 to make sure the version is compatible?

@tornaria
Copy link
Contributor Author

Should there by any tests in spkg-configure.m4 to make sure the version is compatible?

How would we know a future release of ecl will break compatibility? Are you suggesting that only currently known versions are allowed? That doesn't seem very useful since most new releases of most software are reasonably backwards compatible.

In this case, the incompatibility was detected soon enough and this PR seems to fix it. Can you confirm that it works for you?

@jhpalmieri
Copy link
Member

This works for me so far. I want to try at least one more thing, though.

@jhpalmieri
Copy link
Member

This works with ecl installed by homebrew (23.9.9) and also Sage's ecl (21.2.1).

@jhpalmieri
Copy link
Member

Blocker since it is necessary when using the latest homebrew version of ecl.

@orlitzky
Copy link
Contributor

Works on Gentoo too, thanks.

vbraun pushed a commit to vbraun/sage that referenced this pull request Sep 14, 2023
sagemathgh-36235: support for ecl 23.9.9
    
ecl 23.9.9 has two minor changes that affect sagemath:
- api for bignums changed, so the macro `ecl_mpz_from_bignum` has to be
(conditionally) changed
- printing of floating point numbers changed `1.e40` to `1.0e40`, I
changed the example to `1.234e40` which prints the same in 23.9.9 as in
previous versions.

With this changes sagemath should work ok with either 21.2.1 or 23.9.9.
    
URL: sagemath#36235
Reported by: Gonzalo Tornaría
Reviewer(s):
@vbraun vbraun merged commit b6b50a8 into sagemath:develop Sep 16, 2023
@tornaria tornaria deleted the ecl-23 branch September 16, 2023 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants