rpms/libdnf
7
0
Fork 1
Code Issues Pull Requests Releases Activity

swig: Add missing %template for std::vector<TransactionPersistence>

Browse Source 
Resolves: RHEL-154734
This commit is contained in:
Evan Goode 2026-05-20 14:40:48 -04:00
parent 26b7225ec0
commit f43314b51a
2 changed files with 40 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

35
0041-swig-Add-missing-template-for-std-vector-Transaction.patch Normal file
View File

@ -0,0 +1,35 @@
From e5a7516c4c865f2b8e186e957d03abf965057382 Mon Sep 17 00:00:00 2001
From: Evan Goode <mail@evangoo.de>
Date: Mon, 9 Mar 2026 22:23:45 -0400
Subject: [PATCH] swig: Add missing %template for
std::vector<TransactionPersistence>
MergedTransaction::listPersistences() returns std::vector<TransactionPersistence>,
but no SWIG %template was declared for this type. Without it, SWIG wraps the
return value as an opaque pointer instead of a Python list, causing memory leak
warnings and preventing Python code from iterating over the values.
Add the missing template, matching the existing TransactionStateVector pattern.
For: https://github.com/rpm-software-management/libdnf/issues/1744
Assisted-by: Claude Opus 4.6
---
bindings/swig/transaction.i | 1 +
1 file changed, 1 insertion(+)
diff --git a/bindings/swig/transaction.i b/bindings/swig/transaction.i
index c5fe04d8e..e1299b05f 100644
--- a/bindings/swig/transaction.i
+++ b/bindings/swig/transaction.i
@@ -80,6 +80,7 @@ typedef libdnf::CompsPackageType CompsPackageType;
%template() std::vector<std::shared_ptr<libdnf::CompsGroupPackage> >;
%template() std::vector<std::shared_ptr<libdnf::CompsEnvironmentGroup> >;
%template(TransactionStateVector) std::vector<libdnf::TransactionState>;
+%template(TransactionPersistenceVector) std::vector<libdnf::TransactionPersistence>;
%template() std::vector<uint32_t>;
%template() std::vector<int64_t>;
--
2.54.0

6
libdnf.spec
View File

@ -58,7 +58,7 @@
Name: libdnf
Version: %{libdnf_major_version}.%{libdnf_minor_version}.%{libdnf_micro_version}
Release: 18%{?dist}
Release: 19%{?dist}
Summary: Library providing simplified C and Python API to libsolv
License: LGPLv2+
URL: https://github.com/rpm-software-management/libdnf
@ -103,6 +103,7 @@ Patch37: 0037-Add-filterUnneededExtraUserinstalled-and-Python-vers.patch
Patch38: 0038-Describe-all-problems-even-when-there-are-protected-.patch
Patch39: 0039-Clearer-error-for-protected-package-broken-dependenc.patch
Patch40: 0040-Goal-set-protected-as-userinstalled-only-for-the-tem.patch
Patch41: 0041-swig-Add-missing-template-for-std-vector-Transaction.patch
BuildRequires: cmake
@ -352,6 +353,9 @@ popd
%endif
%changelog
* Wed May 20 2026 Evan Goode <egoode@redhat.com> - 0.69.0-19
- swig: Add missing %template for std::vector<TransactionPersistence> (RHEL-154734)
* Mon Feb 09 2026 Ales Matej <amatej@redhat.com> - 0.69.0-18
- Fix a crash when parsing multiple key files fails (RHEL-145618)
- Mark all protected packages as user installed for all transactions (RHEL-76112)