Skip to content

Commit 049af7c

Browse files
committed
Remove cython option from healpy- always use distributed sources.
1 parent 4f23954 commit 049af7c

2 files changed

Lines changed: 18 additions & 32 deletions

File tree

configure.ac

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ dnl
55
dnl +------------------------
66
dnl | Initialize package info
77
dnl +------------------------
8-
AC_INIT([HEALPix], [3.30], [work@theodorekisner.com], [healpix-autotools], [https://github.com/tskisner/healpix])
8+
AC_INIT([HEALPix], [3.30.0], [work@theodorekisner.com], [healpix-autotools], [https://github.com/tskisner/healpix])
99
AC_CONFIG_SRCDIR([Makefile.am])
1010
AM_INIT_AUTOMAKE([foreign tar-ustar])
1111
AC_CONFIG_HEADERS(config.h)
@@ -191,6 +191,12 @@ fi
191191
AC_LANG_POP([C])
192192
AM_CONDITIONAL([HAVE_AM_PYTHON], [test $healpix_python = yes])
193193

194+
dnl +------------------------------------------------
195+
dnl | Fix libtool trying to be too smart
196+
dnl +------------------------------------------------
197+
postdeps_FC=$(echo $postdeps_FC | sed 's/-l //g')
198+
echo "postdeps_FC=$postdeps_FC"
199+
194200
dnl +------------------------------------------------
195201
dnl | Makefiles to generate
196202
dnl +------------------------------------------------

src/healpy/setup.py.in

Lines changed: 11 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -39,39 +39,19 @@ if get_config_var('MACOSX_DEPLOYMENT_TARGET') and not 'MACOSX_DEPLOYMENT_TARGET'
3939
# For ReadTheDocs, do not build the extensions, only install .py files
4040
on_rtd = os.environ.get('READTHEDOCS', None) == 'True'
4141

42-
cython_require = 'Cython >= 0.16'
43-
try:
44-
if ('--help' in sys.argv[1:] or
45-
sys.argv[1] in ('--help-commands', 'egg_info', 'clean', '--version')):
46-
from distutils.command.build_ext import build_ext
47-
ext = "c"
48-
extcpp = "cpp"
49-
else:
50-
pkg_resources.require(cython_require)
51-
from Cython.Distutils import build_ext
52-
ext = "pyx"
53-
extcpp = "pyx"
54-
except:
55-
# User does not have a sufficiently new version of Cython.
56-
if os.path.exists('healpy/src/_query_disc.cpp'):
57-
# This source package already contains the Cython-generated sources,
58-
# so we can just use them.
59-
from distutils.command.build_ext import build_ext
60-
ext = "c"
61-
extcpp = "cpp"
62-
else:
63-
# This source does not contain the Cython-generated sources, so fail.
64-
raise DistutilsExecError('''
65-
66-
It looks like you are attempting to build from the Healpy development
67-
sources, i.e., from GitHub. You need {0} to build Healpy from
68-
development sources.
42+
if os.path.exists('healpy/src/_query_disc.cpp'):
43+
# This source package already contains the Cython-generated sources,
44+
# so we can just use them.
45+
from distutils.command.build_ext import build_ext
46+
ext = "c"
47+
extcpp = "cpp"
48+
else:
49+
# This source does not contain the Cython-generated sources, so fail.
50+
raise DistutilsExecError('''
6951
70-
Either install Healpy from an official stable release from:
71-
https://pypi.python.org/pypi/healpy
52+
This version of healpy should always contain the Cython-generated sources.
7253
73-
OR, to build from development sources, first get {0} from:
74-
https://pypi.python.org/pypi/Cython'''.format(cython_require))
54+
Something is badly broken with your source tree!''')
7555

7656
if on_rtd:
7757
numpy_inc = ''

0 commit comments

Comments
 (0)