sagelib setup.py: Fix dependencies on header files of packages gmp, ntl

[mkoeppe]

These header file locations (for use in Cython dependencies only) are currently hardcoded as living in SAGE_INC, which is no longer true.

This is a step toward #29711, and also preparation for #29847 because it removes use of sage.env.SAGE_INC.

Depends on #29702

CC: @orlitzky @kliem @kiwifb @dimpase @jhpalmieri

Component: build

Author: Matthias Koeppe

Branch/Commit: 18aae7f

Reviewer: François Bissey, Jonathan Kliem

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

[mkoeppe]

Branch: u/mkoeppe/sagelib_setup_py__fix_dependencies_on_header_files_of_packages_gmp__ntl

[mkoeppe]

Commit: 041c9e8

[mkoeppe]

Last 10 new commits:

5db5318	Trac #29345: remove "break" statements from AC_SEARCH_LIBS.
e810ad1	Trac #29345: don't use sage's config.status for the lrcalc build.
93c9921	Trac #29345: replace the function that populates the CVXOPT_* variables.
0e66a0a	Trac #29345: add Dima's SPKG patches for ksh compatibility.
df3f05e	build/make/Makefile.in [SCRIPT_PACKAGE_templ]: cd into the SPKG directory; adjust spkg-install scripts
5372065	Merge branch 't/29793/script_packages_should_cd_into_the_spkg_directory' into t/29411/make_sagelib_a_script_package
c166b97	Merge branch 't/29411/make_sagelib_a_script_package' into t/29702/move_all_code_from_src_setup_py__src_fpickle_setup_py_to_sage_setup
cc30471	build/bin/write-dockerfile.sh: Do not ADD removed file src/Makefile.in
8a41326	Merge branch 't/29411/make_sagelib_a_script_package' into t/29702/move_all_code_from_src_setup_py__src_fpickle_setup_py_to_sage_setup
041c9e8	sage_setup.command.sage_build_cython: Use SAGE_GMP_PREFIX, SAGE_NTL_PREFIX for header dependencies instead of SAGE_INC

[mkoeppe]

comment:3

Branch is on top of #29702; there is only one new commit.

[mkoeppe]

comment:5

Needs review

[kiwifb]

comment:6

Replying to @mkoeppe:

Needs review

This ticket is only the last commit? Or should I look at more commits?

[mkoeppe]

comment:7

That's right, only the last commit.

[kiwifb]

Reviewer: François Bissey

[kiwifb]

comment:8

Hum, you even said so in an earlier. Must be post lunch fatigue :)

LGTM this is much cleaner and much more appropriate.

I really have the feeling that you are breaking sage apart to rebuild it. And I am not complaining.

[mkoeppe]

comment:9

Thanks!

[mkoeppe]

comment:11

Discovered a stupid mistake in the last commit.

[sagetrac-git]

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

18aae7f

sage_setup.command.sage_build_cython: Fix up - add list brackets

[sagetrac-git]

Changed commit from 041c9e8 to 18aae7f