Skip to content

Commit

Permalink
Try to remove all cgal to see if it works with header only, showing n…
Browse files Browse the repository at this point in the history
…othign happening
  • Loading branch information
matthewfeickert committed Sep 17, 2024
1 parent e07a65a commit ddaedee
Showing 1 changed file with 0 additions and 15 deletions.
15 changes: 0 additions & 15 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -15,14 +15,10 @@
import subprocess
import sys
import sysconfig
import urllib.request
import zipfile

import setuptools.command.build_ext
import setuptools.command.install

CGAL_ZIP = "https://github.com/CGAL/cgal/releases/download/v5.6/CGAL-5.6-library.zip"

DIR = pathlib.Path(__file__).parent.resolve()
FASTJET = DIR / "extern" / "fastjet-core"
FASTJET_CONTRIB = DIR / "extern" / "fastjet-contrib"
Expand Down Expand Up @@ -56,15 +52,6 @@ def get_version() -> str:
class FastJetBuild(setuptools.command.build_ext.build_ext):
def build_extensions(self):
if not OUTPUT.exists():
zip_filename = DIR / pathlib.Path(CGAL_ZIP).parts[-1]

with urllib.request.urlopen(CGAL_ZIP) as http_obj:
with open(zip_filename, "wb") as file_obj:
shutil.copyfileobj(http_obj, file_obj)

with zipfile.ZipFile(zip_filename) as zip_obj:
cgal_dir = DIR / zip_obj.namelist()[0]
zip_obj.extractall(DIR)

# Patch for segfault of LimitedWarning
# For more info see https://github.com/scikit-hep/fastjet/pull/131
Expand All @@ -91,8 +78,6 @@ def build_extensions(self):
"--disable-auto-ptr",
"--enable-allcxxplugins",
"--enable-cgal-header-only",
"--enable-cgal",
f"--with-cgaldir={cgal_dir}",
"--enable-swig",
"--enable-pyext",
f'LDFLAGS={env["LDFLAGS"]}',
Expand Down

0 comments on commit ddaedee

Please sign in to comment.