Refactor: Move get_platform_info to mssql_python/platform_utils.py

- Single source of truth for platform detection
- Clean imports in setup.py (no sys.path hacks)
- build_ddbc re-exports for backwards compatibility

Author: bewithgaurav

build_ddbc/__init__.py

@@ -12,7 +12,8 @@
     python -m build                   # Compile + create wheel (automatic)
 """
 
-from .compiler import compile_ddbc, get_platform_info
+from .compiler import compile_ddbc
+from mssql_python.platform_utils import get_platform_info
 
 __all__ = ["compile_ddbc", "get_platform_info"]
 __version__ = "1.2.0"

build_ddbc/compiler.py

@@ -1,70 +1,16 @@
 """
 Core compiler logic for ddbc_bindings.
 
-This module contains the platform detection and build script execution logic.
+This module contains the build script execution logic.
+Platform detection is provided by mssql_python.platform_utils.
 """
 
-import glob
-import os
-import platform
 import sys
 import subprocess
 from pathlib import Path
-from typing import Tuple, Optional
+from typing import Optional
 
-
-def get_platform_info() -> Tuple[str, str]:
-    """
-    Get platform-specific architecture and platform tag information.
-
-    Returns:
-        Tuple of (architecture, platform_tag)
-
-    Raises:
-        OSError: If the platform or architecture is not supported
-    """
-    if sys.platform.startswith("win"):
-        arch = os.environ.get("ARCHITECTURE", "x64")
-        if isinstance(arch, str):
-            arch = arch.strip("\"'")
-
-        if arch in ["x86", "win32"]:
-            return "x86", "win32"
-        elif arch == "arm64":
-            return "arm64", "win_arm64"
-        else:
-            return "x64", "win_amd64"
-
-    elif sys.platform.startswith("darwin"):
-        return "universal2", "macosx_15_0_universal2"
-
-    elif sys.platform.startswith("linux"):
-        target_arch = os.environ.get("targetArch", platform.machine())
-        libc_name, _ = platform.libc_ver()
-
-        if not libc_name:
-            # Fallback: check for musl linker (Alpine Linux)
-            # platform.libc_ver() returns empty string on Alpine
-            is_musl = bool(glob.glob("/lib/ld-musl*"))
-            if not is_musl:
-                print(
-                    "[build_ddbc] Warning: libc detection failed; defaulting to glibc.",
-                    file=sys.stderr,
-                )
-        else:
-            is_musl = "musl" in libc_name.lower()
-
-        if target_arch == "x86_64":
-            return "x86_64", "musllinux_1_2_x86_64" if is_musl else "manylinux_2_28_x86_64"
-        elif target_arch in ["aarch64", "arm64"]:
-            return "aarch64", "musllinux_1_2_aarch64" if is_musl else "manylinux_2_28_aarch64"
-        else:
-            raise OSError(
-                f"Unsupported architecture '{target_arch}' for Linux; "
-                "expected 'x86_64' or 'aarch64'."
-            )
-
-    raise OSError(f"Unsupported platform: {sys.platform}")
+from mssql_python.platform_utils import get_platform_info
 
 
 def find_pybind_dir() -> Path:

mssql_python/platform_utils.py

@@ -0,0 +1,67 @@
+"""
+Platform detection utilities for mssql-python.
+
+This module provides platform and architecture detection used by both
+the build system (setup.py, build_ddbc) and runtime code.
+"""
+
+import glob
+import os
+import platform
+import sys
+from typing import Tuple
+
+
+def get_platform_info() -> Tuple[str, str]:
+    """
+    Get platform-specific architecture and platform tag information.
+
+    Returns:
+        Tuple of (architecture, platform_tag) where platform_tag is a
+        PEP 425 compatible wheel platform tag.
+
+    Raises:
+        OSError: If the platform or architecture is not supported
+    """
+    if sys.platform.startswith("win"):
+        arch = os.environ.get("ARCHITECTURE", "x64")
+        if isinstance(arch, str):
+            arch = arch.strip("\"'")
+
+        if arch in ["x86", "win32"]:
+            return "x86", "win32"
+        elif arch == "arm64":
+            return "arm64", "win_arm64"
+        else:
+            return "x64", "win_amd64"
+
+    elif sys.platform.startswith("darwin"):
+        return "universal2", "macosx_15_0_universal2"
+
+    elif sys.platform.startswith("linux"):
+        target_arch = os.environ.get("targetArch", platform.machine())
+        libc_name, _ = platform.libc_ver()
+
+        if not libc_name:
+            # Fallback: check for musl linker (Alpine Linux)
+            # platform.libc_ver() returns empty string on Alpine
+            is_musl = bool(glob.glob("/lib/ld-musl*"))
+            if not is_musl:
+                print(
+                    "[mssql_python] Warning: libc detection failed; defaulting to glibc.",
+                    file=sys.stderr,
+                )
+        else:
+            is_musl = "musl" in libc_name.lower()
+
+        if target_arch == "x86_64":
+            return "x86_64", "musllinux_1_2_x86_64" if is_musl else "manylinux_2_28_x86_64"
+        elif target_arch in ["aarch64", "arm64"]:
+            return "aarch64", "musllinux_1_2_aarch64" if is_musl else "manylinux_2_28_aarch64"
+        else:
+            raise OSError(
+                f"Unsupported architecture '{target_arch}' for Linux; "
+                "expected 'x86_64' or 'aarch64'."
+            )
+
+    raise OSError(f"Unsupported platform: {sys.platform}")

setup.py

@@ -20,7 +20,7 @@
 from setuptools.dist import Distribution
 from wheel.bdist_wheel import bdist_wheel
 
-from build_ddbc.compiler import get_platform_info
+from mssql_python.platform_utils import get_platform_info
 
 
 # =============================================================================