Bug 2513838 - Review Request: perl-DBD-ODBC - ODBC Driver for DBI
Summary: Review Request: perl-DBD-ODBC - ODBC Driver for DBI
Keywords:
Status: CLOSED WORKSFORME
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jitka Plesnikova
QA Contact: Fedora Extras Quality Assurance
URL: https://metacpan.org/dist/DBD-ODBC
Whiteboard: Unretirement
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2026-08-11 09:16 UTC by Michal Josef Spacek
Modified: 2026-08-25 09:44 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2026-08-25 09:43:46 UTC
Type: ---
Embargoed:
jplesnik: fedora-review+


Attachments (Terms of Use)

Description Michal Josef Spacek 2026-08-11 09:16:01 UTC
Spec URL: https://skim.cz/tmp/perl-DBD-ODBC/perl-DBD-ODBC.spec
SRPM URL: https://skim.cz/tmp/perl-DBD-ODBC/perl-DBD-ODBC-1.61-13.fc45.src.rpm
Description: 
This module is needed to access ODBC databases from within Perl. The
module uses the unixODBC manager to connect to the database.
Fedora Account System Username: mspacek

Comment 1 Fedora Review Service 2026-08-11 09:17:04 UTC
There seems to be some problem with the following file.
SRPM URL: https://skim.cz/tmp/perl-DBD-ODBC/perl-DBD-ODBC-1.61-13.fc45.src.rpm
Fetching it results in a 404 Not Found error.
Please make sure the URL is correct and publicly available.


---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 2 Michal Josef Spacek 2026-08-20 10:33:07 UTC
Spec URL: https://mspacek.fedorapeople.org/perl-DBD-ODBC/perl-DBD-ODBC.spec
SRPM URL: https://mspacek.fedorapeople.org/perl-DBD-ODBC/perl-DBD-ODBC-1.61-13.fc45.src.rpm
Description: 
This module is needed to access ODBC databases from within Perl. The
module uses the unixODBC manager to connect to the database.
Fedora Account System Username: mspacek

Comment 3 Fedora Review Service 2026-08-20 10:39:54 UTC
Copr build:
https://copr.fedorainfracloud.org/coprs/build/10883613
(succeeded)

Review template:
https://download.copr.fedorainfracloud.org/results/@fedora-review/fedora-review-2513838-perl-dbd-odbc/fedora-rawhide-x86_64/10883613-perl-DBD-ODBC/fedora-review/review.txt

Found issues:

- Explicit dependency on perl-devel is not allowed unless building architecture-specific code which links to libperl.so
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/Perl/
- A package with this name already exists. Please check https://src.fedoraproject.org/rpms/perl-DBD-ODBC
  Read more: https://docs.fedoraproject.org/en-US/packaging-guidelines/Naming/#_conflicting_package_names

Please know that there can be false-positives.

---
This comment was created by the fedora-review-service
https://github.com/FrostyX/fedora-review-service

If you want to trigger a new Copr build, add a comment containing new
Spec and SRPM URLs or [fedora-review-service-build] string.

Comment 4 Jitka Plesnikova 2026-08-21 12:10:56 UTC
Source file is ok
Summary is ok
License is ok
Description is ok
URL and Source0 are ok
All tests passed (database tests skipped because DBI_DSN is undefined,
which is expected without a live ODBC data source)

BuildRequires:
FIX: Add build-require 'coreutils' - it is needed for %{_fixperms}.

$ rpm -qp --requires perl-DBD-ODBC-1.61-13.fc46.x86_64.rpm | sort | uniq -c | grep -v rpmlib
      1 libc.so.6()(64bit)
      1 libc.so.6(GLIBC_2.14)(64bit)
      1 libc.so.6(GLIBC_2.2.5)(64bit)
      1 libc.so.6(GLIBC_2.3.4)(64bit)
      1 libc.so.6(GLIBC_2.3)(64bit)
      1 libc.so.6(GLIBC_2.38)(64bit)
      1 libc.so.6(GLIBC_2.4)(64bit)
      1 libc.so.6(GLIBC_ABI_DT_RELR)(64bit)
      1 libc.so.6(GLIBC_ABI_GNU2_TLS)(64bit)
      1 libodbc.so.2()(64bit)
      1 libperl.so.5.44()(64bit)
      1 perl(constant)
      1 perl(DBI) >= 1.609
      1 perl(DynaLoader)
      1 perl(Exporter)
      1 perl-libs
      1 perl(:MODULE_COMPAT_5.44.0)
      1 perl(strict)
      1 perl(:VERSION) >= 5.8.0
      1 perl(warnings)
      1 rtld(GNU_HASH)
      1 unixODBC
Binary requires are Ok.

$ rpm -qp --provides perl-DBD-ODBC-1.61-13.fc46.x86_64.rpm | sort | uniq -c
      1 perl(DBD::ODBC)
      1 perl-DBD-ODBC = 1.61-13.fc46
      1 perl-DBD-ODBC(x86-64) = 1.61-13.fc46
Binary provides are Ok.

$ rpmlint ./perl-DBD-ODBC*
perl-DBD-ODBC.src: E: spelling-error ('unixODBC', '%description -l en_US')
perl-DBD-ODBC.x86_64: E: spelling-error ('unixODBC', '%description -l en_US')
 2 packages and 1 specfiles checked; 2 errors, 0 warnings, 8 filtered, 2 badness; has taken 0.8 s
Note: The spelling error is a false positive - unixODBC is a product name.

TODO: Explicitly name packaged files in %{_mandir}/man* and %{perl_vendorarch} directories
https://docs.fedoraproject.org/en-US/packaging-guidelines/#_explicit_lists.

TODO: Please consider to add examples to documentation. Don't forget to
      remove execute bit.

Please consider fixing 'FIX' and 'TODO' items.

Resolution:
Approved

Comment 5 Michal Josef Spacek 2026-08-21 12:50:00 UTC
Thanks you.

Comment 6 Michal Josef Spacek 2026-08-25 09:44:31 UTC
Package was unretired: https://forge.fedoraproject.org/releng/fedora-scm-requests/issues/363


Note You need to log in before you can comment on or make changes to this bug.