public inbox for git-commits@fedoraproject.org
help / color / mirror / Atom feed
From: Paul Howarth <paul@city-fan.org>
To: git-commits@fedoraproject.org
Subject: [rpms/perl-DBIx-ContextualFetch] epel10.3: Spec tidy-up
Date: Tue, 01 Sep 2026 18:08:00 GMT	[thread overview]
Message-ID: <178828608066.1.12515695681834917002.rpms-perl-DBIx-ContextualFetch-23f567497f59@fedoraproject.org> (raw)

            A new commit has been pushed.

            Repo   : rpms/perl-DBIx-ContextualFetch
            Branch : epel10.3
            Commit : 23f567497f59012ed6d330a6ae007186e6348cce
            Author : Paul Howarth <paul@city-fan.org>
            Date   : 2020-08-20T21:47:17+01:00
            Stats  : +36/-16 in 1 file(s)
            URL    : https://src.fedoraproject.org/rpms/perl-DBIx-ContextualFetch/c/23f567497f59012ed6d330a6ae007186e6348cce?branch=epel10.3

            Log:
            Spec tidy-up

- Use author-independent source URL
- Specify all build requirements
- Use %{make_build} and %{make_install}
- Don't need to remove empty directories from the buildroot
- Fix permissions verbosely
- Make %files list more explicit

---
diff --git a/perl-DBIx-ContextualFetch.spec b/perl-DBIx-ContextualFetch.spec
index 0a42dc5..d89fb44 100644
--- a/perl-DBIx-ContextualFetch.spec
+++ b/perl-DBIx-ContextualFetch.spec
@@ -1,17 +1,31 @@
 Name:           perl-DBIx-ContextualFetch
 Version:        1.03
-Release:        39%{?dist}
+Release:        40%{?dist}
 Summary:        Add contextual fetches to DBI
 License:        GPL+ or Artistic
 URL:            https://metacpan.org/release/DBIx-ContextualFetch
-Source0:        https://cpan.metacpan.org/authors/id/T/TM/TMTM/DBIx-ContextualFetch-%{version}.tar.gz
-
+Source0:        https://cpan.metacpan.org/modules/by-module/DBIx/DBIx-ContextualFetch-%{version}.tar.gz
 BuildArch:      noarch
+# Build
+BuildRequires:  coreutils
+BuildRequires:  make
 BuildRequires:  perl-generators
-BuildRequires:  perl(DBI), perl(ExtUtils::MakeMaker)
-BuildRequires:  perl(DBD::SQLite), perl(Test::Pod::Coverage), perl(Test::More)
-Requires:  perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version))
-Requires:	perl(DBI)
+BuildRequires:  perl-interpreter
+BuildRequires:  perl(ExtUtils::MakeMaker) >= 6.76
+# Module
+BuildRequires:  perl(base)
+BuildRequires:  perl(DBI)
+BuildRequires:  perl(strict)
+BuildRequires:  perl(warnings)
+# Test Suite
+BuildRequires:  perl(File::Temp)
+BuildRequires:  perl(Test::More)
+# Optional Tests
+BuildRequires:  perl(DBD::SQLite)
+BuildRequires:  perl(Test::Pod) >= 1.00
+BuildRequires:  perl(Test::Pod::Coverage) >= 1.00
+# Dependencies
+Requires:       perl(:MODULE_COMPAT_%(eval "`perl -V:version`"; echo $version))
 
 %{?perl_default_filter}
 %global __requires_exclude %{?__requires_exclude:%__requires_exclude|}^perl\\(DBI::(st|db)\\)
@@ -23,25 +37,31 @@ Requires:	perl(DBI)
 %setup -q -n DBIx-ContextualFetch-%{version}
 
 %build
-%{__perl} Makefile.PL INSTALLDIRS=vendor
-make %{?_smp_mflags}
+perl Makefile.PL INSTALLDIRS=vendor NO_PACKLIST=1 NO_PERLLOCAL=1
+%{make_build}
 
 %install
-make pure_install PERL_INSTALL_ROOT=$RPM_BUILD_ROOT
-find $RPM_BUILD_ROOT -type f -name .packlist -exec rm -f {} ';'
-find $RPM_BUILD_ROOT -type d -depth -exec rmdir {} 2>/dev/null ';'
-chmod -R u+w $RPM_BUILD_ROOT/*
+%{make_install}
+%{_fixperms} -c %{buildroot}
 
 %check
 make test
 
 %files
 %doc Changes
-%{perl_vendorlib}/DBIx
-%{_mandir}/man3/*.3*
-
+%{perl_vendorlib}/DBIx/
+%{_mandir}/man3/DBIx::ContextualFetch.3*
 
 %changelog
+* Thu Aug 20 2020 Paul Howarth <paul@city-fan.org> - 1.03-40
+- Spec tidy-up
+  - Use author-independent source URL
+  - Specify all build requirements
+  - Use %%{make_build} and %%{make_install}
+  - Don't need to remove empty directories from the buildroot
+  - Fix permissions verbosely
+  - Make %%files list more explicit
+
 * Tue Jul 28 2020 Fedora Release Engineering <releng@fedoraproject.org> - 1.03-39
 - Rebuilt for https://fedoraproject.org/wiki/Fedora_33_Mass_Rebuild
 

                 reply	other threads:[~2026-09-01 18:08 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=178828608066.1.12515695681834917002.rpms-perl-DBIx-ContextualFetch-23f567497f59@fedoraproject.org \
    --to=paul@city-fan.org \
    --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