Skip to content

Commit

Permalink
Perform a little cleanup on setup.py
Browse files Browse the repository at this point in the history
We don't need the `CONDA_BUILD` flags anymore.

Signed-off-by: Patrick Avery <[email protected]>
  • Loading branch information
psavery committed Oct 18, 2023
1 parent b60dc94 commit f47aa1e
Showing 1 changed file with 3 additions and 19 deletions.
22 changes: 3 additions & 19 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@
np_include_dir = numpy.get_include()

install_reqs = [
'appdirs',
'fabio>=0.11',
'fast-histogram',
'h5py',
Expand All @@ -18,6 +19,7 @@
'psutil',
'pycifrw',
'pyyaml',
'scikit-image',
'scikit-learn',
'scipy',
'tbb',
Expand All @@ -34,15 +36,6 @@
else:
compiler_optimize_flags = []

# This a hack to get around the fact that scikit-image on conda-forge doesn't install
# dist info so setuptools can't find it, even though its there, which results in
# pkg_resources.DistributionNotFound, even though the package is available. So we
# only added it if we aren't building with conda.
# appdirs has the same issue.
if os.environ.get('CONDA_BUILD') != '1':
install_reqs.append('scikit-image')
install_reqs.append('appdirs')


# extension for convolution from astropy
def get_convolution_extensions():
Expand Down Expand Up @@ -103,7 +96,7 @@ def get_extension_modules():
# use entry_points, not scripts:
entry_points = {
'console_scripts': ["hexrd = hexrd.cli.main:main"]
}
}

setup(
name='hexrd',
Expand Down Expand Up @@ -135,12 +128,3 @@ def get_extension_modules():
python_requires='>=3.8',
install_requires=install_reqs
)

# ext_modules = get_extension_modulesf()
# setupF(
# name='hexrd',
# url='https://github.com/cryos/hexrd',
# license='BSD',
# ext_modules=ext_modules,
# packages=find_packages(),
# )

0 comments on commit f47aa1e

Please sign in to comment.