CLN: clean setup.py
an attempt to clean setup.py
We don’t need to go into the details - just want to make sure this doesn’t cause regressions like the build time slowdown called out before

Sent from my iPhone

> On Nov 11, 2020, at 2:21 PM, Fangchen Li <notifications@github.com> wrote:
> 
> ﻿
> @fangchenli commented on this pull request.
> 
> In setup.py:
> 
> >  if _CYTHON_INSTALLED:
> -    from Cython.Distutils.old_build_ext import old_build_ext as _build_ext
> -
> -    cython = True
> -    from Cython import Tempita as tempita
> +    from Cython import Tempita
> +    from Cython.Distutils.build_ext import build_ext as _build_ext
> Looking at the cython source code. The new build_ext is just
> setuptools.command.build_ext or distutils.command.build_ext. But I'm not sure how it differs from the old_build_ext.
> 
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub, or unsubscribe.
 > We don’t need to go into the details - just want to make sure this doesn’t cause regressions like the build time slowdown called out before
> […](#)
> Sent from my iPhone
> On Nov 11, 2020, at 2:21 PM, Fangchen Li ***@***.***> wrote: ﻿ @fangchenli commented on this pull request. In setup.py: > if _CYTHON_INSTALLED: - from Cython.Distutils.old_build_ext import old_build_ext as _build_ext - - cython = True - from Cython import Tempita as tempita + from Cython import Tempita + from Cython.Distutils.build_ext import build_ext as _build_ext Looking at the cython source code. The new build_ext is just setuptools.command.build_ext or distutils.command.build_ext. But I'm not sure how it differs from the old_build_ext. — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.

Sure, the windows np16 build does seem slower than before. I'll test it more. The runtime for the windows build seems normal now. thanks @fangchenli 