sage.rings.integer, rational: Remove compile-time dependency on cypari2 and flint

[mkoeppe]

Functions such as set_from_pari_gen should probably be moved to sage.libs.pari.convert_sage.

The compile-time dependency on sage.libs.flint.ulong_extras is for the use of n_factor.

CC: @videlec @kliem @tscrim

Component: refactoring

Author: Matthias Koeppe, Jonathan Kliem

Branch/Commit: b4477da

Reviewer: Jonathan Kliem, Matthias Koeppe

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

[videlec]

comment:1

Moving set_from_pari_gen in sage.libs.pari would make cypari2 depends on sage. Wouldn't it?

[mkoeppe]

comment:2

sage.libs.pari.convert_sage already depends on sage (and on cypari2).

[videlec]

comment:3

I misunderstood the ticket description... I thought you wanted to move that to cypari2. Sorry.

It indeed makes sense to have it in sage.libs where all the low-level interfaces actually are.

[mkoeppe]

Description changed:

--- 
+++ 
@@ -1,4 +1,4 @@
-Functions such as `set_from_pari_gen` should probably be moved to `sage.libs.pari`.
+Functions such as `set_from_pari_gen` should probably be moved to `sage.libs.pari.convert_sage`.
 
 The compile-time dependency on `sage.libs.flint.ulong_extras` is for the use of `n_factor`.  
 

[mkoeppe]

comment:6

Setting new milestone based on a cursory review of ticket status, priority, and last modification date.

[mkoeppe]

Branch: u/mkoeppe/sage_rings_integer__rational__remove_compile_time_dependency_on_cypari2_and_flint

[mkoeppe]

Author: Matthias Koeppe, ...

[mkoeppe]

New commits:

08084b3

sage.libs.pari.convert_sage: Move set_integer_from_gen here from sage.rings.integer (set_from_pari_gen)

[mkoeppe]

Commit: 08084b3

[kliem]

New commits:

9986723	remove compile time dependency of Integer and Rational on cypari2
c08a8b5	remove compile-time dependency on flint from integer

[kliem]

Changed commit from 08084b3 to c08a8b5

[kliem]

Changed branch from u/mkoeppe/sage_rings_integer__rational__remove_compile_time_dependency_on_cypari2_and_flint to u/gh-kliem/sage_rings_integer__rational__remove_compile_time_dependency_on_cypari2_and_flint

[kliem]

Changed author from Matthias Koeppe, ... to Matthias Koeppe, Jonathan Kliem

[kliem]

Reviewer: Jonathan Kliem, ...