sage.rings.fast_arith: Remove compile-time dependency on cypari2

[mkoeppe]

Follow-up on #30022

CC: @kliem @videlec @tscrim

Component: refactoring

Author: Jonathan Kliem

Branch/Commit: 41df930

Reviewer: Matthias Koeppe

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

[kliem]

Author: Jonathan Kliem

[kliem]

Dependencies: 30022

[kliem]

Commit: e0c8466

[kliem]

New commits:

a8a9437	pull in from #30022 to avoid merge conflict
e0c8466	remove pari compile time dependency from rings.fast_arith

[kliem]

Branch: public/32441

[kliem]

Changed dependencies from 30022 to #30022

[sagetrac-git]

Changed commit from e0c8466 to cbfec03

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

cbfec03

fix function signatures

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

cfcf047	add doctests that confirms that cysignals interfaces the PARI signals
d422435	updated checksums

[sagetrac-git]

Changed commit from cbfec03 to d422435

[mkoeppe]

comment:5

wrong ticket?

[sagetrac-git]

Changed commit from d422435 to cbfec03

[sagetrac-git]

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

[kliem]

comment:7

Replying to @mkoeppe:

wrong ticket?

Thanks. I was confused why the checksum for #32431 were incorrect. That explains it.

I thought I had done #32441 from a different computer, but I guess I was in remote shell. Didn't catch on at that time.

[mkoeppe]

comment:8

Also I think commit a8a9437 needs to be dropped

[kliem]

comment:10

Replying to @mkoeppe:

Also I think commit a8a9437 needs to be dropped

This was on purpose. I accidently already commit fast_arith.pyx. But this is mostly to avoid merge conflicts in convert_sage.pyx, because both tickets append to that file. This will not work.

[kliem]

comment:11

Hence the dependency on #30022.

[mkoeppe]

comment:13

I don't get it; can you not just rebase onto the branch of #30022, i.e., b4477da?

[kliem]

Changed branch from public/32441 to public/32441-reb

[kliem]

New commits:

08084b3	sage.libs.pari.convert_sage: Move set_integer_from_gen here from sage.rings.integer (set_from_pari_gen)
9986723	remove compile time dependency of Integer and Rational on cypari2
c08a8b5	remove compile-time dependency on flint from integer
88d91ba	use try ... except ... else
b4477da	fixed indent
6ae2006	Merge branch 'develop' of git://trac.sagemath.org/sage into public/32441-reb
1aa8cd1	pull in from #30022 to avoid merge conflict
05bb421	remove pari compile time dependency from rings.fast_arith

[kliem]

Changed commit from cbfec03 to 05bb421

[kliem]

comment:15

Replying to @mkoeppe:

I don't get it; can you not just rebase onto the branch of #30022, i.e., b4477da?

Indeed, it doesn't work. I tried to pull in only those files, but git couldn't even get it to work and I still had merge conflicts.

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

0087de5

fix function signatures

[sagetrac-git]

Changed commit from 05bb421 to 0087de5

[mkoeppe]

comment:17

9.5.beta1 is out... rebase?

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

1c11285

Merge branch 'public/32441-reb' of git://trac.sagemath.org/sage into public/32441-reb2

[sagetrac-git]

Changed commit from 0087de5 to 1c11285

[sagetrac-git]

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

cbbef63	pull in from #30022 to avoid merge conflict
2c46a20	remove pari compile time dependency from rings.fast_arith
41df930	fix function signatures

[sagetrac-git]

Changed commit from 1c11285 to 41df930

[kliem]

Changed dependencies from #30022 to none

[mkoeppe]

Reviewer: Matthias Koeppe

[kliem]

comment:22

Thank you.

[vbraun]

Changed branch from public/32441-reb to 41df930