public inbox for git-commits@fedoraproject.org
help / color / mirror / Atom feed
From: Ankur Sinha (Ankur Sinha Gmail) <sanjay.ankur@gmail.com>
To: git-commits@fedoraproject.org
Subject: [rpms/ledger] f43: feat: re-enable python (fixes rh#2405078)
Date: Thu, 16 Jul 2026 12:29:10 GMT	[thread overview]
Message-ID: <178420495096.1.6468480194306021093.rpms-ledger-c63f58b6e1b2@fedoraproject.org> (raw)

A new commit has been pushed.

Repo   : rpms/ledger
Branch : f43
Commit : c63f58b6e1b2bc5ff9fa73acddd534c3bcb3406b
Author : Ankur Sinha (Ankur Sinha Gmail) <sanjay.ankur@gmail.com>
Date   : 2026-07-16T11:08:23+00:00
Stats  : +15/-6 in 1 file(s)
URL    : https://src.fedoraproject.org/rpms/ledger/c/c63f58b6e1b2bc5ff9fa73acddd534c3bcb3406b?branch=f43

Log:
feat: re-enable python (fixes rh#2405078)

---
diff --git a/ledger.spec b/ledger.spec
index 84d12d0..50a097b 100644
--- a/ledger.spec
+++ b/ledger.spec
@@ -18,6 +18,7 @@ BuildRequires:    gmp-devel
 BuildRequires:    libedit-devel
 BuildRequires:    mpfr-devel
 BuildRequires:    utf8cpp-devel
+BuildRequires:    python3-devel
 
 # For building documentation.
 BuildRequires:    doxygen
@@ -37,10 +38,16 @@ their data, there really is no alternative.
 
 %package devel
 Summary: Libraries and header files for %{name} development
-Requires: %{name} = %{version}-%{release}
+Requires: %{name}%{?_isa} = %{version}-%{release}
 %description devel
 Libraries and header files for %{name} development.
 
+%package -n python3-%{name}
+Summary:  Python library for %{name}
+Requires: %{name}%{?_isa} = %{version}-%{release}
+%description -n python3-%{name}
+Python library for %{ledger}.
+
 %prep
 %autosetup -n %{name}-%{version} -p1
 # Avoid texinfo errors on EL7.
@@ -54,7 +61,7 @@ rm -r lib/utfcpp
 %cmake \
        -DCMAKE_INSTALL_PREFIX=%{_prefix} \
        -DCMAKE_SKIP_RPATH:BOOL=ON \
-       -DUSE_PYTHON:BOOL=OFF \
+       -DUSE_PYTHON:BOOL=ONF \
        -DUSE_DOXYGEN:BOOL=ON \
        -DBUILD_WEB_DOCS:BOOL=ON
 
@@ -88,10 +95,9 @@ for i in demo.ledger divzero.dat drewr3.dat drewr.dat sample.dat standard.dat tr
     install -p -m0644 test/input/${i} %{buildroot}%{_pkgdocdir}/samples/${i}
 done
 
-# Tests are disabled for the time being since they seem to require Python 2
-#%%check
-# Tests all fail when removing rpath.
-#LD_LIBRARY_PATH=$PWD %%ctest
+%check
+export LD_LIBRARY_PATH=%{buildroot}%{_libdir}
+%ctest
 
 %files
 %doc README.md doc/GLOSSARY.md NEWS.md
@@ -108,6 +114,9 @@ done
 %config(noreplace) %{_sysconfdir}/bash_completion.d/ledger
 %license LICENSE.md
 
+%files -n python3-%{name}
+%{python3_sitearch}/%{name}.so
+
 %files devel
 %{_includedir}/ledger
 %{_libdir}/libledger.so

                 reply	other threads:[~2026-07-16 12:29 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=178420495096.1.6468480194306021093.rpms-ledger-c63f58b6e1b2@fedoraproject.org \
    --to=sanjay.ankur@gmail.com \
    --cc=git-commits@fedoraproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox