coxeter3: Add spkg-configure.m4, use coxeter/ prefix for includes, move OptionalExtension options to .pyx file directives

[mkoeppe]

Package exists on Arch and Fedora according to https://repology.org/project/coxeter/versions; they install coxeter in a compatible way, i.e., headers with the prefix coxeter/.

In this ticket we change the includes in src/sage/libs/coxeter3/decl.pxd to use the prefix coxeter/, which allows us to get rid of include_dirs in src/module_list.py:

    OptionalExtension('sage.libs.coxeter3.coxeter',
              sources = ['sage/libs/coxeter3/coxeter.pyx'],
              language="c++",
              include_dirs = [os.path.join(SAGE_INC, 'coxeter')],
              package = 'coxeter3'),

This simplifies the work in #29720.

CC: @tscrim @dimpase @orlitzky @antonio-rojas @isuruf

Component: build: configure

Keywords: sd109

Author: Matthias Koeppe

Branch/Commit: 1497f3f

Reviewer: Travis Scrimshaw

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

[mkoeppe]

Branch: u/mkoeppe/spkg_configure_m4_for_coxeter3

[mkoeppe]

New commits:

ad1b8b7

build/pkgs/coxeter3/distros/fedora.txt: New

[mkoeppe]

Commit: ad1b8b7

[mkoeppe]

Changed keywords from none to sd109

[mkoeppe]

comment:5

Help with this ticket would be very welcome

[mkoeppe]

Author: Matthias Koeppe, ...

[sagetrac-git]

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

3dd2927

src/module_list.py: Move OptionalExtension options for sage.libs.coxeter3.coxeter to distutils directive

[sagetrac-git]

Changed commit from ad1b8b7 to 3dd2927

[sagetrac-git]

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

dbbc985

src/sage/libs/coxeter3/decl.pxd: Add coxeter/ prefix to all header includes; src/module_list.py: remove include_dirs option

[sagetrac-git]

Changed commit from 3dd2927 to dbbc985

[sagetrac-git]

Changed commit from dbbc985 to 1497f3f

[sagetrac-git]

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

6909e23	build/pkgs/coxeter3/distros/fedora.txt: New
2b060ae	src/module_list.py: Move OptionalExtension options for sage.libs.coxeter3.coxeter to distutils directive
4f0d07d	src/sage/libs/coxeter3/decl.pxd: Add coxeter/ prefix to all header includes; src/module_list.py: remove include_dirs option
1497f3f	build/pkgs/coxeter3/spkg-configure.m4: New

[mkoeppe]

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

[mkoeppe]

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

[mkoeppe]

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

[tscrim]

Reviewer: Travis Scrimshaw

[tscrim]

comment:14

coxeter3 builds on my system. So LGTM.

One of these days I will actually get back to converting coxeter3 to full C++ and doing some additional cleanup...

[mkoeppe]

comment:15

Thank you!

[vbraun]

Changed branch from u/mkoeppe/spkg_configure_m4_for_coxeter3 to 1497f3f