pygit2 1.19.0__tar.gz → 1.19.1__tar.gz

{pygit2-1.19.0 → pygit2-1.19.1}/CHANGELOG.md

@@ -1,3 +1,17 @@
+# 1.19.1 (2025-12-29)
+
+- Update wheels to libgit2 1.9.2 and OpenSSL 3.5
+
+- Fix: now diff's getitem/iter returns `None` for unchanged or binary files
+  [#1412](https://github.com/libgit2/pygit2/pull/1412)
+
+- CI (macOS): arm, intel and pypy wheels (instead of universal)
+  [#1441](https://github.com/libgit2/pygit2/pull/1441)
+
+- CI (pypy): fix tests
+  [#1437](https://github.com/libgit2/pygit2/pull/1437)
+
+
 # 1.19.0 (2025-10-23)
 
 - Add support for Python 3.14 and drop 3.10

pygit2-1.19.1/Makefile

@@ -0,0 +1,7 @@
+.PHONY: build html
+
+build:
+	OPENSSL_VERSION=3.5.4 LIBSSH2_VERSION=1.11.1 LIBGIT2_VERSION=1.9.2 sh build.sh
+
+html: build
+	make -C docs html

{pygit2-1.19.0 → pygit2-1.19.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pygit2
-Version: 1.19.0
+Version: 1.19.1
 Summary: Python bindings for libgit2.
 Home-page: https://github.com/libgit2/pygit2
 Maintainer: J. David Ibáñez

{pygit2-1.19.0 → pygit2-1.19.1}/build.sh

@@ -22,14 +22,14 @@
 #
 #   sh build.sh
 #
-# Build libgit2 1.9.0 (will use libssh2 if available), then build pygit2
+# Build libgit2 1.9.2 (will use libssh2 if available), then build pygit2
 # inplace:
 #
-#   LIBGIT2_VERSION=1.9.0 sh build.sh
+#   LIBGIT2_VERSION=1.9.2 sh build.sh
 #
-# Build libssh2 1.11.1 and libgit2 1.9.0, then build pygit2 inplace:
+# Build libssh2 1.11.1 and libgit2 1.9.2, then build pygit2 inplace:
 #
-#   LIBSSH2_VERSION=1.11.1 LIBGIT2_VERSION=1.9.0 sh build.sh
+#   LIBSSH2_VERSION=1.11.1 LIBGIT2_VERSION=1.9.2 sh build.sh
 #
 # Build inplace and run the tests:
 #
@@ -62,6 +62,8 @@ if [ "$CIBUILDWHEEL" = "1" ]; then
         apt-get install wget -y
         if [ -z "$OPENSSL_VERSION" ]; then
             apt-get install libssl-dev -y
+        else
+            apt-get install libtime-piece-perl -y
         fi
     elif [ -f /usr/bin/yum ]; then
         yum install wget zlib-devel -y
@@ -70,11 +72,14 @@ if [ "$CIBUILDWHEEL" = "1" ]; then
         else
             yum install perl-IPC-Cmd -y
             yum install perl-Pod-Html -y
+            yum install perl-Time-Piece -y
         fi
     elif [ -f /sbin/apk ]; then
         apk add wget
         if [ -z "$OPENSSL_VERSION" ]; then
-            apk add openssl-dev
+            apk add --no-cache openssl-dev
+        else
+            apk add --no-cache perl
         fi
     fi
     rm -rf ci
@@ -268,7 +273,7 @@ if [ "$1" = "mypy" ]; then
     if [ -n "$WHEELDIR" ]; then
         $PREFIX/bin/pip install $WHEELDIR/pygit2*-$PYTHON_TAG-*.whl
     fi
-    $PREFIX/bin/pip install -r requirements-test.txt
+    $PREFIX/bin/pip install -r requirements-test.txt -r requirements-typing.txt
     $PREFIX/bin/mypy pygit2 test
 fi
 

{pygit2-1.19.0 → pygit2-1.19.1}/pygit2/_build.py

@@ -34,7 +34,7 @@ from pathlib import Path
 #
 # The version number of pygit2
 #
-__version__ = '1.19.0'
+__version__ = '1.19.1'
 
 
 #

{pygit2-1.19.0 → pygit2-1.19.1}/pygit2/_pygit2.pyi

@@ -432,8 +432,8 @@ class Diff:
     def from_c(diff, repo) -> Diff: ...
     @staticmethod
     def parse_diff(git_diff: str | bytes) -> Diff: ...
-    def __getitem__(self, index: int) -> Patch: ...  # Diff_getitem
-    def __iter__(self) -> Iterator[Patch]: ...  # -> DiffIter
+    def __getitem__(self, index: int) -> Patch | None: ...  # Diff_getitem
+    def __iter__(self) -> Iterator[Patch | None]: ...  # -> DiffIter
     def __len__(self) -> int: ...
 
 class DiffDelta:

{pygit2-1.19.0 → pygit2-1.19.1}/pyproject.toml

@@ -8,14 +8,16 @@ skip = "*musllinux_ppc64le"
 archs = ["native"]
 build-frontend = "default"
 dependency-versions = "pinned"
-environment = {LIBGIT2_VERSION="1.9.1", LIBSSH2_VERSION="1.11.1", OPENSSL_VERSION="3.3.3", LIBGIT2="/project/ci"}
+environment = {LIBGIT2_VERSION="1.9.2", LIBSSH2_VERSION="1.11.1", OPENSSL_VERSION="3.5.4", LIBGIT2="/project/ci"}
 
 before-all = "sh build.sh"
 test-command = "pytest"
 test-sources = ["test", "pytest.ini"]
 before-test = "pip install -r {project}/requirements-test.txt"
 # Will avoid testing on emulated architectures (specifically ppc64le)
-test-skip = "*-*linux_ppc64le"
+# Also, skip testing pypy on macOS arm64 due issue with bootstrapping git config paths
+# see https://github.com/libgit2/pygit2/issues/1442
+test-skip = "*-*linux_ppc64le pp*-macosx_arm64"
 
 [tool.cibuildwheel.linux]
 repair-wheel-command = "LD_LIBRARY_PATH=/project/ci/lib64 auditwheel repair -w {dest_dir} {wheel}"
@@ -25,13 +27,12 @@ select = "*-musllinux*"
 repair-wheel-command = "LD_LIBRARY_PATH=/project/ci/lib auditwheel repair -w {dest_dir} {wheel}"
 
 [tool.cibuildwheel.macos]
-archs = ["universal2"]
-environment = {LIBGIT2_VERSION="1.9.1", LIBSSH2_VERSION="1.11.1", OPENSSL_VERSION="3.3.3", LIBGIT2="/Users/runner/work/pygit2/pygit2/ci"}
+environment = {LIBGIT2_VERSION="1.9.2", LIBSSH2_VERSION="1.11.1", OPENSSL_VERSION="3.5.4", LIBGIT2="/Users/runner/work/pygit2/pygit2/ci"}
 repair-wheel-command = "DYLD_LIBRARY_PATH=/Users/runner/work/pygit2/pygit2/ci/lib delocate-wheel --require-archs {delocate_archs} -w {dest_dir} {wheel}"
 
 [tool.cibuildwheel.windows]
 environment.LIBGIT2_SRC = "build/libgit2_src"
-environment.LIBGIT2_VERSION = "1.9.1"
+environment.LIBGIT2_VERSION = "1.9.2"
 before-all = "powershell -File build.ps1"
 
 [[tool.cibuildwheel.overrides]]

{pygit2-1.19.0 → pygit2-1.19.1}/requirements-test.txt

@@ -1,4 +1,2 @@
 pytest
 pytest-cov
-mypy
-types-cffi

pygit2-1.19.1/requirements-typing.txt

@@ -0,0 +1,2 @@
+mypy
+types-cffi

pygit2-1.19.1/requirements-wheel.txt

@@ -0,0 +1 @@
+cibuildwheel ~= 3.3

{pygit2-1.19.0 → pygit2-1.19.1}/src/diff.c

@@ -533,6 +533,11 @@ diff_get_patch_byindex(git_diff *diff, size_t idx)
     if (err < 0)
         return Error_set(err);
 
+    /* libgit2 may decide not to create a patch if the file is
+       "unchanged or binary", but this isn't an error case */
+    if (patch == NULL)
+        Py_RETURN_NONE;
+
     return (PyObject*) wrap_patch(patch, NULL, NULL);
 }
 

{pygit2-1.19.0 → pygit2-1.19.1}/test/test_diff.py

@@ -28,6 +28,7 @@
 import textwrap
 from collections.abc import Iterator
 from itertools import chain
+from pathlib import Path
 
 import pytest
 
@@ -35,6 +36,8 @@ import pygit2
 from pygit2 import Diff, Repository
 from pygit2.enums import DeltaStatus, DiffFlag, DiffOption, DiffStatsFormat, FileMode
 
+from .utils import diff_safeiter
+
 COMMIT_SHA1_1 = '5fe808e8953c12735680c257f56600cb0de44b10'
 COMMIT_SHA1_2 = 'c2792cfa289ae6321ecf2cd5806c2194b0fd070c'
 COMMIT_SHA1_3 = '2cdae28389c059815e951d0bb9eed6533f61a46b'
@@ -176,11 +179,11 @@ def test_diff_empty_index(dirtyrepo: Repository) -> None:
     head = repo[repo.lookup_reference('HEAD').resolve().target]
 
     diff = head.tree.diff_to_index(repo.index)
-    files = [patch.delta.new_file.path for patch in diff]
+    files = [patch.delta.new_file.path for patch in diff_safeiter(diff)]
     assert DIFF_HEAD_TO_INDEX_EXPECTED == files
 
     diff = repo.diff('HEAD', cached=True)
-    files = [patch.delta.new_file.path for patch in diff]
+    files = [patch.delta.new_file.path for patch in diff_safeiter(diff)]
     assert DIFF_HEAD_TO_INDEX_EXPECTED == files
 
 
@@ -189,17 +192,17 @@ def test_workdir_to_tree(dirtyrepo: Repository) -> None:
     head = repo[repo.lookup_reference('HEAD').resolve().target]
 
     diff = head.tree.diff_to_workdir()
-    files = [patch.delta.new_file.path for patch in diff]
+    files = [patch.delta.new_file.path for patch in diff_safeiter(diff)]
     assert DIFF_HEAD_TO_WORKDIR_EXPECTED == files
 
     diff = repo.diff('HEAD')
-    files = [patch.delta.new_file.path for patch in diff]
+    files = [patch.delta.new_file.path for patch in diff_safeiter(diff)]
     assert DIFF_HEAD_TO_WORKDIR_EXPECTED == files
 
 
 def test_index_to_workdir(dirtyrepo: Repository) -> None:
     diff = dirtyrepo.diff()
-    files = [patch.delta.new_file.path for patch in diff]
+    files = [patch.delta.new_file.path for patch in diff_safeiter(diff)]
     assert DIFF_INDEX_TO_WORK_EXPECTED == files
 
 
@@ -217,15 +220,15 @@ def test_diff_empty_index_bare(barerepo: Repository) -> None:
     head = repo[repo.lookup_reference('HEAD').resolve().target]
 
     diff = barerepo.index.diff_to_tree(head.tree)
-    files = [patch.delta.new_file.path.split('/')[0] for patch in diff]
+    files = [patch.delta.new_file.path.split('/')[0] for patch in diff_safeiter(diff)]
     assert [x.name for x in head.tree] == files
 
     diff = head.tree.diff_to_index(repo.index)
-    files = [patch.delta.new_file.path.split('/')[0] for patch in diff]
+    files = [patch.delta.new_file.path.split('/')[0] for patch in diff_safeiter(diff)]
     assert [x.name for x in head.tree] == files
 
     diff = repo.diff('HEAD', cached=True)
-    files = [patch.delta.new_file.path.split('/')[0] for patch in diff]
+    files = [patch.delta.new_file.path.split('/')[0] for patch in diff_safeiter(diff)]
     assert [x.name for x in head.tree] == files
 
 
@@ -235,9 +238,10 @@ def test_diff_tree(barerepo: Repository) -> None:
 
     def _test(diff: Diff) -> None:
         assert diff is not None
-        assert 2 == sum(map(lambda x: len(x.hunks), diff))
+        assert 2 == sum(map(lambda x: len(x.hunks), diff_safeiter(diff)))
 
         patch = diff[0]
+        assert patch is not None
         hunk = patch.hunks[0]
         assert hunk.old_start == 1
         assert hunk.old_lines == 1
@@ -267,16 +271,16 @@ def test_diff_empty_tree(barerepo: Repository) -> None:
     diff = commit_a.tree.diff_to_tree()
 
     def get_context_for_lines(diff: Diff) -> Iterator[str]:
-        hunks = chain.from_iterable(map(lambda x: x.hunks, diff))
+        hunks = chain.from_iterable(map(lambda x: x.hunks, diff_safeiter(diff)))
         lines = chain.from_iterable(map(lambda x: x.lines, hunks))
         return map(lambda x: x.origin, lines)
 
-    entries = [p.delta.new_file.path for p in diff]
+    entries = [p.delta.new_file.path for p in diff_safeiter(diff)]
     assert all(commit_a.tree[x] for x in entries)
     assert all('-' == x for x in get_context_for_lines(diff))
 
     diff_swaped = commit_a.tree.diff_to_tree(swap=True)
-    entries = [p.delta.new_file.path for p in diff_swaped]
+    entries = [p.delta.new_file.path for p in diff_safeiter(diff_swaped)]
     assert all(commit_a.tree[x] for x in entries)
     assert all('+' == x for x in get_context_for_lines(diff_swaped))
 
@@ -293,11 +297,15 @@ def test_diff_tree_opts(barerepo: Repository) -> None:
     for flag in [DiffOption.IGNORE_WHITESPACE, DiffOption.IGNORE_WHITESPACE_EOL]:
         diff = commit_c.tree.diff_to_tree(commit_d.tree, flag)
         assert diff is not None
-        assert 0 == len(diff[0].hunks)
+        patch = diff[0]
+        assert patch is not None
+        assert 0 == len(patch.hunks)
 
     diff = commit_c.tree.diff_to_tree(commit_d.tree)
     assert diff is not None
-    assert 1 == len(diff[0].hunks)
+    patch = diff[0]
+    assert patch is not None
+    assert 1 == len(patch.hunks)
 
 
 def test_diff_merge(barerepo: Repository) -> None:
@@ -310,13 +318,14 @@ def test_diff_merge(barerepo: Repository) -> None:
 
     diff_c = commit_b.tree.diff_to_tree(commit_c.tree)
     assert diff_c is not None
-    assert 'b' not in [patch.delta.new_file.path for patch in diff_b]
-    assert 'b' in [patch.delta.new_file.path for patch in diff_c]
+    assert 'b' not in [patch.delta.new_file.path for patch in diff_safeiter(diff_b)]
+    assert 'b' in [patch.delta.new_file.path for patch in diff_safeiter(diff_c)]
 
     diff_b.merge(diff_c)
-    assert 'b' in [patch.delta.new_file.path for patch in diff_b]
+    assert 'b' in [patch.delta.new_file.path for patch in diff_safeiter(diff_b)]
 
     patch = diff_b[0]
+    assert patch is not None
     hunk = patch.hunks[0]
     assert hunk.old_start == 1
     assert hunk.old_lines == 1
@@ -340,6 +349,7 @@ def test_diff_ids(barerepo: Repository) -> None:
     commit_a = barerepo[COMMIT_SHA1_1]
     commit_b = barerepo[COMMIT_SHA1_2]
     patch = commit_a.tree.diff_to_tree(commit_b.tree)[0]
+    assert patch is not None
     delta = patch.delta
     assert delta.old_file.id == '7f129fd57e31e935c6d60a0c794efe4e6927664b'
     assert delta.new_file.id == 'af431f20fc541ed6d5afede3e2dc7160f6f01f16'
@@ -357,6 +367,7 @@ def test_hunk_content(barerepo: Repository) -> None:
     commit_a = barerepo[COMMIT_SHA1_1]
     commit_b = barerepo[COMMIT_SHA1_2]
     patch = commit_a.tree.diff_to_tree(commit_b.tree)[0]
+    assert patch is not None
     hunk = patch.hunks[0]
     lines = (f'{x.origin} {x.content}' for x in hunk.lines)
     assert HUNK_EXPECTED == ''.join(lines)
@@ -371,11 +382,15 @@ def test_find_similar(barerepo: Repository) -> None:
     # ~ Must pass INCLUDE_UNMODIFIED if you expect to emulate
     # ~ --find-copies-harder during rename transformion...
     diff = commit_a.tree.diff_to_tree(commit_b.tree, DiffOption.INCLUDE_UNMODIFIED)
-    assert all(x.delta.status != DeltaStatus.RENAMED for x in diff)
-    assert all(x.delta.status_char() != 'R' for x in diff)
+    assert all(
+        patch.delta.status != DeltaStatus.RENAMED for patch in diff_safeiter(diff)
+    )
+    assert all(patch.delta.status_char() != 'R' for patch in diff_safeiter(diff))
     diff.find_similar()
-    assert any(x.delta.status == DeltaStatus.RENAMED for x in diff)
-    assert any(x.delta.status_char() == 'R' for x in diff)
+    assert any(
+        patch.delta.status == DeltaStatus.RENAMED for patch in diff_safeiter(diff)
+    )
+    assert any(patch.delta.status_char() == 'R' for patch in diff_safeiter(diff))
 
 
 def test_diff_stats(barerepo: Repository) -> None:
@@ -398,7 +413,7 @@ def test_deltas(barerepo: Repository) -> None:
     commit_b = barerepo[COMMIT_SHA1_2]
     diff = commit_a.tree.diff_to_tree(commit_b.tree)
     deltas = list(diff.deltas)
-    patches = list(diff)
+    patches = list(diff_safeiter(diff))
     assert len(deltas) == len(patches)
     for i, delta in enumerate(deltas):
         patch_delta = patches[i].delta
@@ -459,3 +474,33 @@ def test_diff_blobs(emptyrepo: Repository) -> None:
     assert diff_one_context_line.text == PATCH_BLOBS_ONE_CONTEXT_LINE
     diff_all_together = repo.diff(blob1, blob2, context_lines=1, interhunk_lines=1)
     assert diff_all_together.text == PATCH_BLOBS_DEFAULT
+
+
+def test_diff_unchanged_file_no_patch(testrepo: Repository) -> None:
+    repo = testrepo
+
+    # Convert hello.txt line endings to CRLF
+    path = Path(repo.workdir) / 'hello.txt'
+    data = path.read_bytes()
+    data = data.replace(b'\n', b'\r\n')
+    path.write_bytes(data)
+
+    # Enable CRLF filter
+    repo.config['core.autocrlf'] = 'input'
+
+    diff = repo.diff()
+    assert len(diff) == 1
+
+    # Get patch #0 in the same diff several times.
+    # git_patch_from_diff eventually decides that the file is "unchanged";
+    # it returns a NULL patch in this case.
+    # https://libgit2.org/docs/reference/main/patch/git_patch_from_diff
+    for i in range(10):  # loop typically exits in the third iteration
+        patch = diff[0]
+        if patch is None:  # libgit2 decides the file is unchanged
+            break
+        assert patch.delta.new_file.path == path.name
+        assert patch.text == ''  # no content change (just line endings)
+    else:
+        # Didn't find the edge case that this test is supposed to exercise.
+        assert False, 'libgit2 rebuilt a new patch every time'

{pygit2-1.19.0 → pygit2-1.19.1}/test/test_repository.py

@@ -456,7 +456,9 @@ def test_stash_partial(testrepo: Repository) -> None:
         )
         stash_commit = testrepo[stash_id].peel(pygit2.Commit)
         stash_diff = testrepo.diff(stash_commit.parents[0], stash_commit)
-        stash_files = set(patch.delta.new_file.path for patch in stash_diff)
+        stash_files = set(
+            patch.delta.new_file.path for patch in utils.diff_safeiter(stash_diff)
+        )
         return stash_files == set(paths)
 
     # Stash a modified file

{pygit2-1.19.0 → pygit2-1.19.1}/test/utils.py

@@ -30,7 +30,7 @@ import socket
 import stat
 import sys
 import zipfile
-from collections.abc import Callable
+from collections.abc import Callable, Iterator
 from pathlib import Path
 from types import TracebackType
 from typing import Any, Optional, ParamSpec, TypeVar
@@ -103,6 +103,18 @@ def rmtree(path: str | Path) -> None:
         shutil.rmtree(path, onerror=force_rm_handle)
 
 
+def diff_safeiter(diff: pygit2.Diff) -> Iterator[pygit2.Patch]:
+    """
+    In rare cases, Diff.__iter__ may yield None (see diff_get_patch_byindex).
+    To make mypy happy, use this iterator instead of Diff.__iter__ to ensure
+    that all patches in a Diff are valid Patch objects, not None.
+    """
+    for patch in diff:
+        if patch is None:
+            raise TypeError('patch is None')
+        yield patch
+
+
 class TemporaryRepository:
     def __init__(self, name: str, tmp_path: Path) -> None:
         self.name = name

pygit2-1.19.0/Makefile

@@ -1,7 +0,0 @@
-.PHONY: build html
-
-build:
-	OPENSSL_VERSION=3.3.3 LIBSSH2_VERSION=1.11.1 LIBGIT2_VERSION=1.9.1 sh build.sh
-
-html: build
-	make -C docs html

{pygit2-1.19.0 → pygit2-1.19.1}/AUTHORS.md

@@ -13,9 +13,9 @@ Authors:
     Dave Borowitz
     Brandon Milton
     Daniel Rodríguez Troitiño
+    Brendan Doherty
     Peter Rowlands
     Richo Healey
-    Brendan Doherty
     Christian Boos
     Julien Miotte
     Nick Hynes