hatch-cpp 0.3.3__py3-none-any.whl → 0.3.5__py3-none-any.whl

hatch_cpp/__init__.py

@@ -1,4 +1,4 @@
-__version__ = "0.3.3"
+__version__ = "0.3.5"
 
 from .config import *
 from .hooks import *

hatch_cpp/config.py

@@ -93,7 +93,9 @@ class HatchCppBuildPlan(HatchCppBuildConfig):
 
     def execute(self):
         for command in self.commands:
-            system_call(command)
+            ret = system_call(command)
+            if ret != 0:
+                raise RuntimeError(f"hatch-cpp build command failed with exit code {ret}: {command}")
         return self.commands
 
     def cleanup(self):

hatch_cpp/tests/test_vcpkg_ref.py

@@ -0,0 +1,174 @@
+"""Tests for vcpkg ref/branch checkout support."""
+
+from __future__ import annotations
+
+from pathlib import Path
+
+from hatch_cpp.toolchains.vcpkg import (
+    HatchCppVcpkgConfiguration,
+    _read_vcpkg_ref_from_gitmodules,
+)
+
+
+class TestReadVcpkgRefFromGitmodules:
+    """Tests for the _read_vcpkg_ref_from_gitmodules helper."""
+
+    def test_no_gitmodules_file(self, tmp_path, monkeypatch):
+        monkeypatch.chdir(tmp_path)
+        assert _read_vcpkg_ref_from_gitmodules(Path("vcpkg")) is None
+
+    def test_gitmodules_without_vcpkg_submodule(self, tmp_path, monkeypatch):
+        monkeypatch.chdir(tmp_path)
+        (tmp_path / ".gitmodules").write_text('[submodule "other"]\n\tpath = other\n\turl = https://github.com/example/other.git\n')
+        assert _read_vcpkg_ref_from_gitmodules(Path("vcpkg")) is None
+
+    def test_gitmodules_vcpkg_without_branch(self, tmp_path, monkeypatch):
+        monkeypatch.chdir(tmp_path)
+        (tmp_path / ".gitmodules").write_text('[submodule "vcpkg"]\n\tpath = vcpkg\n\turl = https://github.com/microsoft/vcpkg.git\n')
+        assert _read_vcpkg_ref_from_gitmodules(Path("vcpkg")) is None
+
+    def test_gitmodules_vcpkg_with_branch(self, tmp_path, monkeypatch):
+        monkeypatch.chdir(tmp_path)
+        (tmp_path / ".gitmodules").write_text(
+            '[submodule "vcpkg"]\n\tpath = vcpkg\n\turl = https://github.com/microsoft/vcpkg.git\n\tbranch = 2024.01.12\n'
+        )
+        assert _read_vcpkg_ref_from_gitmodules(Path("vcpkg")) == "2024.01.12"
+
+    def test_gitmodules_vcpkg_with_commit_sha_branch(self, tmp_path, monkeypatch):
+        monkeypatch.chdir(tmp_path)
+        sha = "a1b2c3d4e5f6a1b2c3d4e5f6a1b2c3d4e5f6a1b2"
+        (tmp_path / ".gitmodules").write_text(
+            f'[submodule "vcpkg"]\n\tpath = vcpkg\n\turl = https://github.com/microsoft/vcpkg.git\n\tbranch = {sha}\n'
+        )
+        assert _read_vcpkg_ref_from_gitmodules(Path("vcpkg")) == sha
+
+    def test_gitmodules_custom_vcpkg_root(self, tmp_path, monkeypatch):
+        monkeypatch.chdir(tmp_path)
+        (tmp_path / ".gitmodules").write_text(
+            '[submodule "deps/vcpkg"]\n\tpath = deps/vcpkg\n\turl = https://github.com/microsoft/vcpkg.git\n\tbranch = 2024.06.15\n'
+        )
+        # Default vcpkg root won't match
+        assert _read_vcpkg_ref_from_gitmodules(Path("vcpkg")) is None
+        # Custom root matches
+        assert _read_vcpkg_ref_from_gitmodules(Path("deps/vcpkg")) == "2024.06.15"
+
+    def test_gitmodules_multiple_submodules(self, tmp_path, monkeypatch):
+        monkeypatch.chdir(tmp_path)
+        (tmp_path / ".gitmodules").write_text(
+            '[submodule "other"]\n'
+            "\tpath = other\n"
+            "\turl = https://github.com/example/other.git\n"
+            "\tbranch = main\n"
+            '[submodule "vcpkg"]\n'
+            "\tpath = vcpkg\n"
+            "\turl = https://github.com/microsoft/vcpkg.git\n"
+            "\tbranch = 2024.01.12\n"
+        )
+        assert _read_vcpkg_ref_from_gitmodules(Path("vcpkg")) == "2024.01.12"
+
+
+class TestVcpkgRefConfig:
+    """Tests for vcpkg_ref configuration field."""
+
+    def test_default_vcpkg_ref_is_none(self):
+        cfg = HatchCppVcpkgConfiguration()
+        assert cfg.vcpkg_ref is None
+
+    def test_explicit_vcpkg_ref(self):
+        cfg = HatchCppVcpkgConfiguration(vcpkg_ref="2024.01.12")
+        assert cfg.vcpkg_ref == "2024.01.12"
+
+    def test_explicit_vcpkg_ref_commit_sha(self):
+        sha = "a1b2c3d4e5f6"
+        cfg = HatchCppVcpkgConfiguration(vcpkg_ref=sha)
+        assert cfg.vcpkg_ref == sha
+
+
+class TestResolveVcpkgRef:
+    """Tests for _resolve_vcpkg_ref priority logic."""
+
+    def test_explicit_ref_takes_priority_over_gitmodules(self, tmp_path, monkeypatch):
+        monkeypatch.chdir(tmp_path)
+        (tmp_path / ".gitmodules").write_text(
+            '[submodule "vcpkg"]\n\tpath = vcpkg\n\turl = https://github.com/microsoft/vcpkg.git\n\tbranch = 2024.01.12\n'
+        )
+        cfg = HatchCppVcpkgConfiguration(vcpkg_ref="my-custom-tag")
+        assert cfg._resolve_vcpkg_ref() == "my-custom-tag"
+
+    def test_falls_back_to_gitmodules(self, tmp_path, monkeypatch):
+        monkeypatch.chdir(tmp_path)
+        (tmp_path / ".gitmodules").write_text(
+            '[submodule "vcpkg"]\n\tpath = vcpkg\n\turl = https://github.com/microsoft/vcpkg.git\n\tbranch = 2024.01.12\n'
+        )
+        cfg = HatchCppVcpkgConfiguration()
+        assert cfg._resolve_vcpkg_ref() == "2024.01.12"
+
+    def test_returns_none_when_no_ref(self, tmp_path, monkeypatch):
+        monkeypatch.chdir(tmp_path)
+        cfg = HatchCppVcpkgConfiguration()
+        assert cfg._resolve_vcpkg_ref() is None
+
+
+class TestVcpkgGenerate:
+    """Tests that generate() includes the checkout command when a ref is set."""
+
+    def _make_vcpkg_env(self, tmp_path):
+        """Create a minimal vcpkg.json so generate() produces commands."""
+        (tmp_path / "vcpkg.json").write_text("{}")
+
+    def test_generate_with_explicit_ref(self, tmp_path, monkeypatch):
+        monkeypatch.chdir(tmp_path)
+        self._make_vcpkg_env(tmp_path)
+
+        cfg = HatchCppVcpkgConfiguration(vcpkg_ref="2024.01.12")
+        commands = cfg.generate(None)
+
+        assert any("git clone" in cmd for cmd in commands)
+        assert any("git -C vcpkg checkout 2024.01.12" in cmd for cmd in commands)
+        # checkout must come after clone but before bootstrap
+        clone_idx = next(i for i, c in enumerate(commands) if "git clone" in c)
+        checkout_idx = next(i for i, c in enumerate(commands) if "checkout" in c)
+        bootstrap_idx = next(i for i, c in enumerate(commands) if "bootstrap" in c)
+        assert clone_idx < checkout_idx < bootstrap_idx
+
+    def test_generate_with_gitmodules_ref(self, tmp_path, monkeypatch):
+        monkeypatch.chdir(tmp_path)
+        self._make_vcpkg_env(tmp_path)
+        (tmp_path / ".gitmodules").write_text(
+            '[submodule "vcpkg"]\n\tpath = vcpkg\n\turl = https://github.com/microsoft/vcpkg.git\n\tbranch = 2024.06.15\n'
+        )
+
+        cfg = HatchCppVcpkgConfiguration()
+        commands = cfg.generate(None)
+
+        assert any("git -C vcpkg checkout 2024.06.15" in cmd for cmd in commands)
+
+    def test_generate_without_ref(self, tmp_path, monkeypatch):
+        monkeypatch.chdir(tmp_path)
+        self._make_vcpkg_env(tmp_path)
+
+        cfg = HatchCppVcpkgConfiguration()
+        commands = cfg.generate(None)
+
+        assert not any("checkout" in cmd for cmd in commands)
+        assert any("git clone" in cmd for cmd in commands)
+
+    def test_generate_skips_clone_when_vcpkg_root_exists(self, tmp_path, monkeypatch):
+        monkeypatch.chdir(tmp_path)
+        self._make_vcpkg_env(tmp_path)
+        (tmp_path / "vcpkg").mkdir()
+
+        cfg = HatchCppVcpkgConfiguration(vcpkg_ref="2024.01.12")
+        commands = cfg.generate(None)
+
+        # When vcpkg_root already exists, no clone or checkout happens
+        assert not any("git clone" in cmd for cmd in commands)
+        assert not any("checkout" in cmd for cmd in commands)
+        assert any("vcpkg" in cmd and "install" in cmd for cmd in commands)
+
+    def test_generate_no_vcpkg_json(self, tmp_path, monkeypatch):
+        monkeypatch.chdir(tmp_path)
+        # No vcpkg.json => no commands at all
+        cfg = HatchCppVcpkgConfiguration(vcpkg_ref="2024.01.12")
+        commands = cfg.generate(None)
+        assert commands == []

hatch_cpp/toolchains/common.py

@@ -323,7 +323,9 @@ class HatchCppPlatform(BaseModel):
             flags += " " + " ".join(f"/U{macro}" for macro in effective_undef_macros)
             flags += " /EHsc /DWIN32"
             if library.std:
-                flags += f" /std:{library.std}"
+                # MSVC minimum is c++14; clamp older standards
+                std = library.std if library.std not in ("c++11", "c++0x") else "c++14"
+                flags += f" /std:{std}"
         # clean
         while flags.count("  "):
             flags = flags.replace("  ", " ")

hatch_cpp/toolchains/vcpkg.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import configparser
 from pathlib import Path
 from platform import machine as platform_machine
 from sys import platform as sys_platform
@@ -38,14 +39,45 @@ VcpkgPlatformDefaults = {
 }
 
 
+def _read_vcpkg_ref_from_gitmodules(vcpkg_root: Path) -> Optional[str]:
+    """Read the branch/ref for vcpkg from .gitmodules if it exists.
+
+    Looks for a submodule whose path matches ``vcpkg_root`` and returns
+    its ``branch`` value when present.
+    """
+    gitmodules_path = Path(".gitmodules")
+    if not gitmodules_path.exists():
+        return None
+
+    parser = configparser.ConfigParser()
+    parser.read(str(gitmodules_path))
+
+    for section in parser.sections():
+        if parser.get(section, "path", fallback=None) == str(vcpkg_root):
+            return parser.get(section, "branch", fallback=None)
+
+    return None
+
+
 class HatchCppVcpkgConfiguration(BaseModel):
     vcpkg: Optional[str] = Field(default="vcpkg.json")
     vcpkg_root: Optional[Path] = Field(default=Path("vcpkg"))
     vcpkg_repo: Optional[str] = Field(default="https://github.com/microsoft/vcpkg.git")
     vcpkg_triplet: Optional[VcpkgTriplet] = Field(default=None)
+    vcpkg_ref: Optional[str] = Field(
+        default=None,
+        description="Branch, tag, or commit SHA to checkout after cloning vcpkg. "
+        "If not set, falls back to the branch specified in .gitmodules for the vcpkg submodule.",
+    )
 
     # TODO: overlay
 
+    def _resolve_vcpkg_ref(self) -> Optional[str]:
+        """Return the ref to checkout: explicit config takes priority, then .gitmodules."""
+        if self.vcpkg_ref is not None:
+            return self.vcpkg_ref
+        return _read_vcpkg_ref_from_gitmodules(self.vcpkg_root)
+
     def generate(self, config):
         commands = []
 
@@ -57,9 +89,12 @@ class HatchCppVcpkgConfiguration(BaseModel):
         if self.vcpkg and Path(self.vcpkg).exists():
             if not Path(self.vcpkg_root).exists():
                 commands.append(f"git clone {self.vcpkg_repo} {self.vcpkg_root}")
-                commands.append(
-                    f"./{self.vcpkg_root / 'bootstrap-vcpkg.sh' if sys_platform != 'win32' else self.vcpkg_root / 'sbootstrap-vcpkg.bat'}"
-                )
+
+                ref = self._resolve_vcpkg_ref()
+                if ref is not None:
+                    commands.append(f"git -C {self.vcpkg_root} checkout {ref}")
+
+                commands.append(f"./{self.vcpkg_root / 'bootstrap-vcpkg.sh' if sys_platform != 'win32' else self.vcpkg_root / 'bootstrap-vcpkg.bat'}")
             commands.append(f"./{self.vcpkg_root / 'vcpkg'} install --triplet {self.vcpkg_triplet}")
 
         return commands

{hatch_cpp-0.3.3.dist-info → hatch_cpp-0.3.5.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: hatch-cpp
-Version: 0.3.3
+Version: 0.3.5
 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.3.dist-info → hatch_cpp-0.3.5.dist-info}/RECORD

@@ -1,5 +1,5 @@
-hatch_cpp/__init__.py,sha256=F_Va8GKZ1FkEZH6EYG9DQRgQpZ3DVRwQDTsMo6Qf9FI,114
-hatch_cpp/config.py,sha256=LlwDBmD1nir08Y5GOl4wa29h2iJBtNRSzYMGHRVFJUo,3698
+hatch_cpp/__init__.py,sha256=oGC85OzqpNweL1rFPBDebJVlVz3i-HmX54zLu5_A8gU,114
+hatch_cpp/config.py,sha256=6oO8J2k91r88DwZwDJpxtxt2F5xZZ-N65IqSyabwOg4,3831
 hatch_cpp/hooks.py,sha256=SQkF5WJIgzw-8rvlTzuQvBqdP6K3fHgnh6CZOZFag50,203
 hatch_cpp/plugin.py,sha256=NcIDi7c9KH-_RkPeeCgWKoH5InIjw2eECswazwvGl_c,4304
 hatch_cpp/utils.py,sha256=lxd3U3aMYsTS6DYMc1y17MR16LzgRzv92f_xh87LSg8,297
@@ -7,6 +7,7 @@ hatch_cpp/tests/test_hatch_build.py,sha256=q2IKTFi3Pq99UsOyL84V-C9VtFUs3ZcA9UlA8
 hatch_cpp/tests/test_platform_specific.py,sha256=8EFHEv86e0CD1bIruHgLb28yf2pSRF1VgvcX2tK5bRo,20124
 hatch_cpp/tests/test_projects.py,sha256=vs-kHdiHM7pCqM3oGQsSryq5blRi0HyK5QFfHxlJDi8,1773
 hatch_cpp/tests/test_structs.py,sha256=cpzbPLneEbEONzXmPS1S9PL9mbhya4udc4eqTpU9w6w,2576
+hatch_cpp/tests/test_vcpkg_ref.py,sha256=OEb4u3Ny-OYNYLanjGaG7-dL4WW48EYdbsxeDN0sqwM,7371
 hatch_cpp/tests/test_project_basic/pyproject.toml,sha256=eqM1UVpNmJWDfsuO18ZG_VOV9I4tAWgsM5Dhf49X8Nc,694
 hatch_cpp/tests/test_project_basic/cpp/project/basic.cpp,sha256=gQ2nmdLqIdgaqxSKvkN_vbp6Iv_pAoVIHETXPRnALb0,117
 hatch_cpp/tests/test_project_basic/cpp/project/basic.hpp,sha256=SO5GhPj8k3RzWrfH37lFSDc8w1Vf3yqTUhxmr1hnoko,422
@@ -57,10 +58,10 @@ hatch_cpp/tests/test_project_pybind_vcpkg/cpp/project/basic.hpp,sha256=LZSfCfhLY
 hatch_cpp/tests/test_project_pybind_vcpkg/project/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 hatch_cpp/toolchains/__init__.py,sha256=anza4tXKxauobWMOUrxX3sEO0qs7sp6VdLXhLae3vkY,64
 hatch_cpp/toolchains/cmake.py,sha256=VQYYqiOu0ZEF0IBuep35Xad7jryWy04-dHYYf7c0XgY,3331
-hatch_cpp/toolchains/common.py,sha256=U8EYVEAzp8sQ0S8rslEKD4v-x8al4s5V7TLX80esrXA,18932
-hatch_cpp/toolchains/vcpkg.py,sha256=Gwv3YNXBngpdebvRuYuUcWCzPyDbmDvy8c0K7uV4k38,2146
-hatch_cpp-0.3.3.dist-info/METADATA,sha256=Sg9bdyfspii3R8hnhW-7HKoYBriu2i0ntsh90IqvCoo,9896
-hatch_cpp-0.3.3.dist-info/WHEEL,sha256=QccIxa26bgl1E6uMy58deGWi-0aeIkkangHcxk2kWfw,87
-hatch_cpp-0.3.3.dist-info/entry_points.txt,sha256=RgXfjpD4iwomJQK5n7FnPkXzb5fOnF5v3DI5hbkgVcw,30
-hatch_cpp-0.3.3.dist-info/licenses/LICENSE,sha256=FWyFTpd5xXEz50QpGDtsyIv6dgR2z_dHdoab_Nq_KMw,11452
-hatch_cpp-0.3.3.dist-info/RECORD,,
+hatch_cpp/toolchains/common.py,sha256=o6whV_7W9GGtBsxgxmg-dHQqdFnuMLffkLHzdbqKqME,19075
+hatch_cpp/toolchains/vcpkg.py,sha256=ku5NE4ihdzbxdReNUXsGx03fPG5FYdQC48mJhdpW-oo,3456
+hatch_cpp-0.3.5.dist-info/METADATA,sha256=BNI_Ezt3hDfW5xhETfORdexE3tb4Ul4dY4M__K0hj10,9896
+hatch_cpp-0.3.5.dist-info/WHEEL,sha256=QccIxa26bgl1E6uMy58deGWi-0aeIkkangHcxk2kWfw,87
+hatch_cpp-0.3.5.dist-info/entry_points.txt,sha256=RgXfjpD4iwomJQK5n7FnPkXzb5fOnF5v3DI5hbkgVcw,30
+hatch_cpp-0.3.5.dist-info/licenses/LICENSE,sha256=FWyFTpd5xXEz50QpGDtsyIv6dgR2z_dHdoab_Nq_KMw,11452
+hatch_cpp-0.3.5.dist-info/RECORD,,