hatch-cpp 0.3.1__tar.gz → 0.3.2__tar.gz

{hatch_cpp-0.3.1 → hatch_cpp-0.3.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: hatch-cpp
-Version: 0.3.1
+Version: 0.3.2
 Summary: Hatch plugin for C++ builds
 Project-URL: Repository, https://github.com/python-project-templates/hatch-cpp
 Project-URL: Homepage, https://github.com/python-project-templates/hatch-cpp

{hatch_cpp-0.3.1 → hatch_cpp-0.3.2}/hatch_cpp/__init__.py

@@ -1,4 +1,4 @@
-__version__ = "0.3.1"
+__version__ = "0.3.2"
 
 from .config import *
 from .hooks import *

{hatch_cpp-0.3.1 → hatch_cpp-0.3.2}/hatch_cpp/tests/test_platform_specific.py

@@ -454,3 +454,82 @@ class TestPlatformFieldOrdering:
         # Base list should not be modified
         assert library.include_dirs == ["common"]
         assert library.include_dirs_linux == ["linux"]
+
+
+class TestMSVCPythonLibsPath:
+    """Tests for MSVC Python libs path discovery."""
+
+    def test_msvc_link_flags_include_libpath(self):
+        """Test that MSVC link flags include /LIBPATH for Python libs."""
+        library = HatchCppLibrary(
+            name="test",
+            sources=["test.cpp"],
+            binding="generic",  # Skip Python.h include
+        )
+
+        platform = HatchCppPlatform(
+            cc="cl",
+            cxx="cl",
+            ld="link",
+            platform="win32",
+            toolchain="msvc",
+            disable_ccache=True,
+        )
+
+        flags = platform.get_link_flags(library)
+        # Should have /link /DLL flags
+        assert "/link" in flags
+        assert "/DLL" in flags
+        # Should have output file
+        assert "/Fe:" in flags
+
+    def test_msvc_link_flags_with_libraries(self):
+        """Test that MSVC link flags properly format library names."""
+        library = HatchCppLibrary(
+            name="test",
+            sources=["test.cpp"],
+            binding="generic",
+            libraries=["mylib"],
+            library_dirs=["path/to/libs"],
+        )
+
+        platform = HatchCppPlatform(
+            cc="cl",
+            cxx="cl",
+            ld="link",
+            platform="win32",
+            toolchain="msvc",
+            disable_ccache=True,
+        )
+
+        flags = platform.get_link_flags(library)
+        # Libraries should have .lib suffix on Windows
+        assert "mylib.lib" in flags
+        # Library dirs should use /LIBPATH:
+        assert "/LIBPATH:path/to/libs" in flags
+
+    def test_msvc_link_flags_with_platform_specific_libraries(self):
+        """Test that MSVC uses win32-specific libraries."""
+        library = HatchCppLibrary(
+            name="test",
+            sources=["test.cpp"],
+            binding="generic",
+            libraries=["common"],
+            libraries_win32=["kernel32", "user32"],
+            library_dirs_win32=["C:/Windows/System32"],
+        )
+
+        platform = HatchCppPlatform(
+            cc="cl",
+            cxx="cl",
+            ld="link",
+            platform="win32",
+            toolchain="msvc",
+            disable_ccache=True,
+        )
+
+        flags = platform.get_link_flags(library)
+        assert "common.lib" in flags
+        assert "kernel32.lib" in flags
+        assert "user32.lib" in flags
+        assert "/LIBPATH:C:/Windows/System32" in flags

{hatch_cpp-0.3.1 → hatch_cpp-0.3.2}/hatch_cpp/toolchains/common.py

@@ -5,7 +5,7 @@ from pathlib import Path
 from re import match
 from shutil import which
 from sys import executable, platform as sys_platform
-from sysconfig import get_path
+from sysconfig import get_config_var, get_path
 from typing import Any, List, Literal, Optional
 
 from pydantic import AliasChoices, BaseModel, Field, field_validator, model_validator
@@ -368,8 +368,16 @@ class HatchCppPlatform(BaseModel):
             flags += " /LD"
             flags += f" /Fe:{library.get_qualified_name(self.platform)}"
             flags += " /link /DLL"
-            if (Path(executable).parent / "libs").exists():
-                flags += f" /LIBPATH:{str(Path(executable).parent / 'libs')}"
+            # Add Python libs directory - check multiple possible locations
+            python_libs_paths = [
+                Path(executable).parent / "libs",  # Standard Python install
+                Path(executable).parent.parent / "libs",  # Some virtualenv layouts
+                Path(get_config_var("installed_base") or "") / "libs",  # sysconfig approach
+            ]
+            for libs_path in python_libs_paths:
+                if libs_path.exists():
+                    flags += f" /LIBPATH:{str(libs_path)}"
+                    break
             flags += " " + " ".join(f"{lib}.lib" for lib in effective_libraries)
             flags += " " + " ".join(f"/LIBPATH:{lib}" for lib in effective_library_dirs)
         # clean

{hatch_cpp-0.3.1 → hatch_cpp-0.3.2}/pyproject.toml

@@ -8,7 +8,7 @@ authors = [{name = "the hatch-cpp authors", email = "t.paine154@gmail.com"}]
 description = "Hatch plugin for C++ builds"
 readme = "README.md"
 license = { text = "Apache-2.0" }
-version = "0.3.1"
+version = "0.3.2"
 requires-python = ">=3.10"
 keywords = [
     "hatch",
@@ -75,7 +75,7 @@ Repository = "https://github.com/python-project-templates/hatch-cpp"
 Homepage = "https://github.com/python-project-templates/hatch-cpp"
 
 [tool.bumpversion]
-current_version = "0.3.1"
+current_version = "0.3.2"
 commit = true
 tag = true
 commit_args = "-s"