Project sanitization

.github/workflows/deploy-wheels.yaml → .github/workflows/pypi.yml

@@ -2,11 +2,6 @@ name: Build
 
 on: 
   push:
-    branches:      
-      - deploy-pypi-test
-      - deploy-pypi
-      # Release branches
-      #- "[0-9]+.[0-9]+.X"
 
   # Manual run
   workflow_dispatch:
@@ -83,8 +78,7 @@ jobs:
       url: https://test.pypi.org/project/delphivcl           
     permissions:
       id-token: write
-    # upload to PyPI test only for pushes to 'deploy-pypi-test' 
-    if: github.event_name == 'push' && github.ref == 'refs/heads/deploy-pypi-test'
+    if: github.ref == 'refs/heads/main'
     steps:                       
       - uses: actions/download-artifact@v2
         with:
@@ -105,8 +99,7 @@ jobs:
       url: https://pypi.org/project/delphivcl/         
     permissions:
       id-token: write
-    # upload to PyPI only for pushes to 'deploy-pypi' 
-    if: github.event_name == 'push' && github.ref == 'refs/heads/deploy-pypi'
+    if: startsWith(github.ref, 'refs/tags/v')
     steps:                       
       - uses: actions/download-artifact@v2
         with:

.github/workflows/tests.yaml → .github/workflows/tests.yml

@@ -3,10 +3,7 @@ name: Tests
 on: 
   push:
     branches:   
-      - main   
-      - test            
-      # Release branches
-      #- "[0-9]+.[0-9]+.X"
+      - main               
 
   # Manual run
   workflow_dispatch:

MANIFEST.in

@@ -0,0 +1,2 @@
+recursive-include delphivcl *.pyd
+recursive-include delphivcl docs.xml

build.py

@@ -0,0 +1,133 @@
+import sys
+import os
+import sys
+import shutil
+import time
+import platform
+import distutils.dir_util
+from wheel.bdist_wheel import bdist_wheel
+
+
+'''
+BDistWheel forces python and abi wheel tags for binary distributions
+'''
+
+
+class BDistWheel(bdist_wheel):
+
+    def finalize_options(self):
+        bdist_wheel.finalize_options(self)
+        self.root_is_pure = ("--universal" in sys.argv)
+
+
+'''
+Sort out the platform library for binary distribution and add to the package directory.
+Place all libraries onto the package directory for source distribution.
+Place the XML doc file onto the package directory.
+'''
+
+
+class PackageDataBuilder():
+
+    def __init__(self):
+        self.pkg_dir = os.path.join(os.curdir, "delphivcl")
+        self.plat_name = None
+        for arg in sys.argv:
+            if arg.startswith("--plat-name=") and (len(arg.split("=")) == 2):
+                self.plat_name = arg.split("=")[1]
+
+    '''
+    Must run "python setup.py clean --all" between builds.
+    '''
+
+    def clean_up(self):
+        lib_dirs = [os.path.join(self.pkg_dir, "Win32"),
+                    os.path.join(self.pkg_dir, "Win64")]
+
+        for lib_dir in lib_dirs:
+            if os.path.isdir(lib_dir):
+                shutil.rmtree(lib_dir)
+
+        # Wait until the OS remove all dirs
+        for lib_dir in lib_dirs:
+            for attempts in range(3):
+                if not os.path.isdir(lib_dir):
+                    break
+                else:
+                    time.sleep(1)
+
+    '''
+    Cross-compiling wheel. 
+    This generates a wheel following the cross platform set on --plat-name.
+    See: https://docs.python.org/3/distutils/builtdist.html#cross-compiling-on-windows
+    '''
+
+    def __check_cross_compiling(self):
+        is_cross_compiling = False
+        lib_dir = None
+
+        if self.plat_name:
+            is_cross_compiling = True
+            if self.plat_name == "win32":
+                lib_dir = "Win32"
+            elif self.plat_name == "win_amd64":
+                lib_dir = "Win64"
+            else:
+                is_cross_compiling = False
+
+        return (is_cross_compiling, lib_dir)
+
+    '''
+    Copy the VCL extension module(s) to the package data folder.
+    Source distributions and Universal binary distributions will deliver 
+    all library versions. The right one will be loaded by __init__.py.
+    Platform distributions will deliver only the library that matches the runner.
+    '''
+
+    def __pick_and_copy_libraries(self):
+        if ("sdist" in sys.argv) or (("bdist_wheel" in sys.argv) and ("--universal" in sys.argv)):
+            # sdist/bdist[--universal] deploy all extension modules
+            distutils.dir_util.copy_tree("lib", self.pkg_dir)
+        else:
+            # Deploys the current platform extension module only
+            is_cross_compiling, lib_dir = self.__check_cross_compiling()
+            if not is_cross_compiling:
+                plat_sys = platform.system()
+                if plat_sys == "Windows":
+                    if (sys.maxsize > 2**32):
+                        # Win x64
+                        lib_dir = "Win64"
+                    else:
+                        # Win x86
+                        lib_dir = "Win32"
+
+            if lib_dir:
+                distutils.dir_util.copy_tree(os.path.join(
+                    "lib", lib_dir), os.path.join(self.pkg_dir, lib_dir))
+            else:
+                raise ValueError("Unsupported platform.")
+
+    '''
+    Copy the XML doc file to the package data folder.
+    The docs.xml file is the result of the compilation of all xml doc files.
+    '''
+
+    def __pick_and_copy_docs(self):
+        # Copy the doc files to the package folder into the doc subfolder
+        docs_file = os.path.join("docs", "xml", "docs.xml")
+        if os.path.exists(docs_file):
+            pkg_doc_dir = os.path.join(self.pkg_dir, "doc")
+            if not os.path.exists(pkg_doc_dir):
+                os.mkdir(pkg_doc_dir)
+            distutils.file_util.copy_file(
+                docs_file, os.path.join(pkg_doc_dir, "docs.xml"))
+
+    def build_package_data(self):
+        self.__pick_and_copy_libraries()
+        self.__pick_and_copy_docs()
+
+
+def setup():
+    builder = PackageDataBuilder()
+    builder.clean_up()
+    builder.build_package_data()

delphivcl/__init__.py

@@ -1,47 +1,58 @@
-import sys, os, sys, platform
-from os import environ
-import importlib, importlib.machinery, importlib.util
+import sys
+import os
+import sys
+import platform
+import importlib
+import importlib.machinery
+import importlib.util
 
-class PyVerNotSupported(Exception):
-  pass
 
-def findmodule():
-  pyver = f"{sys.version_info.major}.{sys.version_info.minor}"
-  ossys = platform.system()
-  libdir = None
+def find_extension_module():
+    py_ver = f"{sys.version_info.major}.{sys.version_info.minor}"
+    plat_sys = platform.system()
+    lib_dir = None
 
-  if not (pyver in ["3.6", "3.7", "3.8", "3.9", "3.10", "3.11"]): 
-    raise PyVerNotSupported(f"DelphiVCL doesn't support Python{pyver}.")
+    if not (py_ver in ["3.6", "3.7", "3.8", "3.9", "3.10", "3.11"]):
+        raise ValueError(f"DelphiVCL doesn't support Python{py_ver}.")
 
-  if ossys == "Windows":
-    if (sys.maxsize > 2**32):
-      #Win x64
-      libdir = "Win64"
+    if plat_sys == "Windows":
+        if (sys.maxsize > 2**32):
+            # Win x64
+            lib_dir = "Win64"
+        else:
+            # Win x86
+            lib_dir = "Win32"
+
+    if lib_dir:
+        lib_dir = os.path.join(os.path.dirname(
+            os.path.abspath(__file__)), lib_dir)
+        if not os.path.exists(lib_dir):
+            raise ValueError(
+                "DelphiVCL module not found. \
+                Try to reinstall the delphivcl package or check for support compatibility.")
+
+        for file_name in os.listdir(lib_dir):
+            if 'DelphiVCL' in file_name:
+                return os.path.join(lib_dir, os.path.basename(file_name))
+        raise ValueError(
+            "DelphiVCL module not found. Try to reinstall the delphivcl package.")
     else:
-      #Win x86
-      libdir = "Win32"
-
-  if libdir:
-    sdir = os.path.join(os.path.dirname(os.path.abspath(__file__)), libdir)
-    if not os.path.exists(sdir):
-      raise ValueError("DelphiVCL module not found. Try to reinstall the delphivcl package or check for support compatibility.")
-    for fname in os.listdir(sdir):
-      if 'DelphiVCL' in fname:
-        return os.path.join(sdir, os.path.basename(fname))
-    raise ValueError("DelphiVCL module not found. Try to reinstall the delphivcl package.")
-  else:
-    raise ValueError("Unsupported platform.")
+        raise ValueError("Unsupported platform.")
+
 
 def new_import():
-    modulefullpath = findmodule()
-    loader = importlib.machinery.ExtensionFileLoader("DelphiVCL", modulefullpath)
-    spec = importlib.util.spec_from_file_location("DelphiVCL", modulefullpath,
-        loader=loader, submodule_search_locations=None)
-    ld = loader.create_module(spec)
+    lib_path = find_extension_module()
+    loader = importlib.machinery.ExtensionFileLoader("DelphiVCL", lib_path)
+    spec = importlib.util.spec_from_file_location("DelphiVCL",
+                                                  lib_path,
+                                                  loader=loader,
+                                                  submodule_search_locations=None)
+    loader.create_module(spec)
     package = importlib.util.module_from_spec(spec)
     sys.modules["delphivcl"] = package
     spec.loader.exec_module(package)
     return package
 
-#Import the shared lib
-package = new_import()
+
+# Import the extension module
+package = new_import()

pyproject.toml

@@ -1,27 +1,14 @@
 [build-system]
-requires = ["setuptools>=40.9.0", "wheel"]
-
-[tool.cibuildwheel]
-build = ["cp36-win*", "cp37-win*", "cp38-win*", "cp39-win*", "cp310-win*"]
-skip = "pp*"
-#archs = ["auto"]
-#repair-wheel-command = ""
-
-[tool.cibuildwheel.windows]
-archs = ["x86", "AMD64"]
-
-[tool.isort]
-profile = "black"
-multi_line_output = 3
-
-[tool.poetry]
-name = "delphivcl"
-version = "0.1.18"
-description = ""
-authors = ["Jim McKeth", "Lucas Belo<[email protected]>", "Lucio Montero<[email protected]>"]
-
-[tool.poetry.dependencies]
-python = ">=3.6<=3.10"
-
-[tool.poetry.dev-dependencies]
-pytest = "^6.2.5"
+requires = [
+    "setuptools >= 54",
+    "setuptools_scm[toml] >= 4, <6",
+    "setuptools_scm_git_archive",
+    "wheel >= 0.29.0",
+]
+build-backend = 'setuptools.build_meta'
+
+[tool.setuptools_scm]
+version_scheme = "post-release"
+local_scheme = "no-local-version"
+write_to = "delphivcl/__version__.py"
+git_describe_command = "git describe --dirty --tags --long --match v* --first-parent"

setup.cfg

@@ -0,0 +1,54 @@
+[egg_info]
+tag_build = 
+tag_date = 0
+tag_svn_revision = 0
+
+[bdist_wheel]
+universal = 0
+
+[build_sphinx]
+source-dir = docs/source
+build-dir = _build
+all-files = True
+
+[metadata]
+name = delphivcl
+version = 1.0.5
+description = Delphi VCL for Python
+long_description = file: README.md
+long_description_content_type = text/markdown; charset=UTF-8
+url = https://github.com/Embarcadero/DelphiVCL4Python
+author = Lucas Belo, Jim McKeeth
+author_email = [email protected]
+license = BSD
+license_files = ['LICENSE.md']
+classifiers =
+    Intended Audience :: Developers
+    Topic :: Software Development
+    License :: Other/Proprietary License
+    Programming Language :: Python
+    Programming Language :: Python :: 3
+    Programming Language :: Python :: 3.6
+    Programming Language :: Python :: 3.7    
+    Programming Language :: Python :: 3.8
+    Programming Language :: Python :: 3.9
+    Programming Language :: Python :: 3.10
+    Programming Language :: Python :: 3.11
+    Programming Language :: Python :: 3 :: Only
+    Operating System :: Microsoft :: Windows    
+project_urls = 
+    Documentation = https://embarcadero.github.io/DelphiVCL4Python
+    Source = https://github.com/Embarcadero/DelphiVCL4Python
+    Tracker = https://github.com/Embarcadero/DelphiVCL4Python/issues
+
+[options]
+zip_safe = False
+packages = find:
+platforms = any
+include_package_data = True
+install_requires =
+python_requires = >=3
+setup_requires =
+    setuptools_scm
+cmdclass =
+    bdist_wheel = build.BDistWheel