new style package for qhull

[videlec]

The branch turns qhull into a new style package (and updates it to the most recent version).

Upstream tarball at

http://www.qhull.org/download/qhull-2015-src-7.2.0.tgz

see also: #19220

Upstream: Reported upstream. Developers acknowledge bug.

Component: packages: optional

Author: Vincent Delecroix

Branch/Commit: 6fc873d

Reviewer: Matthias Koeppe

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

[videlec]

New commits:

20af0fc

21143: new style package for qhull

[videlec]

Branch: u/vdelecroix/21143

[videlec]

Commit: 20af0fc

[sagetrac-git]

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

fcc68c5

21143: new style package for qhull

[sagetrac-git]

Changed commit from 20af0fc to fcc68c5

[videlec]

comment:4

(I reported upstream with respect to the patch)

[videlec]

Upstream: Reported upstream. No feedback yet.

[mkoeppe]

comment:5

It builds fine for me on Mac OS X.

An answer to an Ask SageMath question here claims that qhull is included with scipy: http://ask.sagemath.org/question/10254/how-does-one-use-the-qhull-optional-package/

Also there is this ticket: #12352

[videlec]

comment:6

Replying to @mkoeppe:

It builds fine for me on Mac OS X.

An answer to an Ask SageMath question here claims that qhull is included with scipy: http://ask.sagemath.org/question/10254/how-does-one-use-the-qhull-optional-package/

Indeed

• http://docs.scipy.org/doc/scipy/reference/generated/scipy.spatial.ConvexHull.html
• http://docs.scipy.org/doc/scipy/reference/generated/scipy.spatial.Delaunay.html
• http://docs.scipy.org/doc/scipy/reference/generated/scipy.spatial.Voronoi.html

But scipy does not seem to ship the binaries... But however their interface is pretty good and that's actually what we should use!

Also there is this ticket: #12352

which is rather disjoint.

[mkoeppe]

comment:7

Replying to @videlec:

Replying to @mkoeppe:

It builds fine for me on Mac OS X.

An answer to an Ask SageMath question here claims that qhull is included with scipy: http://ask.sagemath.org/question/10254/how-does-one-use-the-qhull-optional-package/

Indeed

• http://docs.scipy.org/doc/scipy/reference/generated/scipy.spatial.ConvexHull.html
• http://docs.scipy.org/doc/scipy/reference/generated/scipy.spatial.Delaunay.html
• http://docs.scipy.org/doc/scipy/reference/generated/scipy.spatial.Voronoi.html

But scipy does not seem to ship the binaries... But however their interface is pretty good and that's actually what we should use!

Right, for polymake (#20892) we do need the binaries.
Could you update the SPKG.txt to include info about scipy?

[sagetrac-git]

Changed commit from fcc68c5 to 6fc873d

[sagetrac-git]

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

6fc873d

21143: mention scipy and Pyhull in SPKG.txt

[videlec]

comment:9

done

[mkoeppe]

Reviewer: Matthias Koeppe

[videlec]

comment:11

Thanks!

[videlec]

Changed upstream from Reported upstream. No feedback yet. to Reported upstream. Developers acknowledge bug.

[videlec]

comment:13

for info: got a reply from upstream (Brad Barber). The next version of the Makefile will contain the target dependencies and there will be a configurable make install. However, it seems hard to get a configure to work on all platforms...

[vbraun]

Changed branch from u/vdelecroix/21143 to 6fc873d