Fix build recursion issue

Windows had an issue where it was trying to build the CFFI module after
it had already imported the CFFI module. Which caused permissions
errors during builds.

This fix does several things to make the Windows build work properly and
improve the Linux build too:

* The CFFI module is only build when needed, not as part of an sdist
  package
* Version numbering spilt out into separate file so __init__.py import
  is not required
* Merged _build_ffi.py and _build_wolfssl.py into one file
* Made CFFI only build when called as an executable (which happens
  during binary build time)
* Make tox use bdist-wheel instead of sdist

Author: LinuxJedi

setup.py

@@ -26,8 +26,21 @@
 from setuptools import setup
 from setuptools.command.build_ext import build_ext
 
-import wolfcrypt
-from wolfcrypt._build_wolfssl import build_wolfssl
+import re
+VERSIONFILE = "wolfcrypt/_version.py"
+verstrline = open(VERSIONFILE, "rt").read()
+VSRE = r"^__version__ = ['\"]([^'\"]*)['\"]"
+mo = re.search(VSRE, verstrline, re.M)
+if mo:
+    verstr = mo.group(1)
+else:
+    raise RuntimeError("Unable to find version string in %s." % (VERSIONFILE,))
+VSRE = r"^__wolfssl_version__ = ['\"]([^'\"]*)['\"]"
+mo = re.search(VSRE, verstrline, re.M)
+if mo:
+    wolfverstr = mo.group(1)
+else:
+    raise RuntimeError("Unable to find version string in %s." % (VERSIONFILE,))
 
 
 # long_description
@@ -38,30 +51,15 @@
     long_description = long_description.replace(".. include:: LICENSING.rst\n",
                                                 licensing_file.read())
 
-
-class cffiBuilder(build_ext, object):
-
-    def build_extension(self, ext):
-        """ Compile manually the wolfcrypt-py extension, bypass setuptools
-        """
-
-        # if USE_LOCAL_WOLFSSL environment variable has been defined,
-        # do not clone and compile wolfSSL from GitHub
-        if os.environ.get("USE_LOCAL_WOLFSSL") is None:
-            build_wolfssl(wolfcrypt.__wolfssl_version__)
-
-        super(cffiBuilder, self).build_extension(ext)
-
-
 setup(
-    name=wolfcrypt.__title__,
-    version=wolfcrypt.__version__,
-    description=wolfcrypt.__summary__,
+    name="wolfcrypt",
+    version=verstr,
+    description="Python module that encapsulates wolfSSL's crypto engine API.",
     long_description=long_description,
-    author=wolfcrypt.__author__,
-    author_email=wolfcrypt.__email__,
-    url=wolfcrypt.__uri__,
-    license=wolfcrypt.__license__,
+    author="wolfSSL Inc.",
+    author_email="[email protected]",
+    url="https://github.com/wolfssl/wolfcrypt-py",
+    license="GPLv2 or Commercial License",
 
     packages=["wolfcrypt"],
 
@@ -83,6 +81,5 @@ def build_extension(self, ext):
     ],
 
     setup_requires=["cffi"],
-    install_requires=["cffi"],
-    cmdclass={"build_ext" : cffiBuilder}
+    install_requires=["cffi"]
 )

tox.ini

@@ -2,6 +2,7 @@
 envlist = py3
 
 [testenv]
+wheel = true
 setenv =
     PYTHONPATH = {toxinidir}:{toxinidir}/wolfcrypt-py
 

wolfcrypt/__init__.py

@@ -18,22 +18,12 @@
 # along with this program; if not, write to the Free Software
 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA
 
+from wolfcrypt._version import __version__, __wolfssl_version__
+
 __title__ = "wolfcrypt"
 __summary__ = "Python module that encapsulates wolfSSL's crypto engine API."
 __uri__ = "https://github.com/wolfssl/wolfcrypt-py"
 
-# When bumping the C library version, reset the POST count to 0
-
-__wolfssl_version__ = "v5.1.1-stable"
-
-# We're using implicit post releases [PEP 440] to bump package version
-# while maintaining the C library version intact for better reference.
-# https://www.python.org/dev/peps/pep-0440/#implicit-post-releases
-#
-# MAJOR.MINOR.BUILD-POST
-
-__version__ = "5.1.1-0"
-
 __author__ = "wolfSSL Inc."
 __email__ = "[email protected]"