Bug 226267 - (perl-LDAP) Merge Review: perl-LDAP
Merge Review: perl-LDAP
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Robin Norwood
Fedora Package Reviews List
Depends On:
  Show dependency treegraph
Reported: 2007-01-31 15:36 EST by Nobody's working on this, feel free to take it
Modified: 2012-11-26 08:07 EST (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2007-06-21 16:42:03 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---
jochen: fedora‑review+

Attachments (Terms of Use)

  None (edit)
Description Nobody's working on this, feel free to take it 2007-01-31 15:36:31 EST
Fedora Merge Review: perl-LDAP

Initial Owner: rnorwood@redhat.com
Comment 1 Jochen Schmitt 2007-04-03 12:26:05 EDT
+ Tar ball matches with upstream.
+ Rpmlint ok for source package.
+ Local build works fine.

- Rpmlint complaint binary rpm:
rpmlint perl-LDAP-0.34-1.noarch.rpm
E: perl-LDAP useless-explicit-provides perl(Net::LDAP::Filter)

Comment 2 Robin Norwood 2007-04-05 13:03:54 EDT
Ok - filtering out the provides for perl(Net::LDAP::Filter) in the usual way.

Comment 3 Robin Norwood 2007-04-10 10:17:18 EDT
oops - turns out when I actually looked at it that the problem with the provides
was that there is a versioned "Provides: perl(Net::LDAP::Filter) = 15", and and
unversioned "Provides: perl(Net::LDAP::Filter)".  So, the filter had to be
changed to only filter out the 'unversioned' one.
Comment 4 Jochen Schmitt 2007-06-21 14:13:57 EDT
Ping morwood@redhat.com
Comment 5 Jochen Schmitt 2007-06-21 14:23:55 EDT
+ Rpmlint quite on binary package.
+ Filelist seems ok.
+ Local install and uninstall works fine.
+ Mock build works fine.
+ License seems ok.

- Package should not used Epoch Tag. (No blocker, becouse changing it may breaks
EVR path)

*** APPROVED ***

Comment 6 Robin Norwood 2007-06-21 15:23:23 EDT
Thanks for the review.

Comment 7 Jochen Schmitt 2007-06-21 16:42:03 EDT
I think, is ok, when I close this bug.
Comment 8 Petr Pisar 2012-11-26 04:30:52 EST
Package Change Request
Package Name: perl-LDAP
Branches: f16 f17 f18
InitialCC: perl-sig

Please add pseudo-user `perl-sig' with watch* permission only to all Fedora branches.
Comment 9 Jon Ciesla 2012-11-26 08:03:58 EST

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