Notes:
1. This is an rt3 dependency.

2. I am intentionally submitting 1.27, because later versions have dependencies
on newer modules from perl core than RH is willing to provide.
c.f. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=166475

I.e. later versions could work on FC > 4, but for FC < 5, we probably will have
to stay with 1.27.

rpmlint returns:
E: perl-DBIx-SearchBuilder script-without-shellbang
/usr/lib/perl5/vendor_perl/5.8.6/DBIx/SearchBuilder/Record.pm
E: perl-DBIx-SearchBuilder script-without-shellbang
/usr/lib/perl5/vendor_perl/5.8.6/DBIx/SearchBuilder/Record/Cachable.pm
E: perl-DBIx-SearchBuilder script-without-shellbang
/usr/lib/perl5/vendor_perl/5.8.6/DBIx/SearchBuilder/Handle.pm
E: perl-DBIx-SearchBuilder script-without-shellbang
/usr/lib/perl5/vendor_perl/5.8.6/DBIx/SearchBuilder.pm
E: perl-DBIx-SearchBuilder script-without-shellbang
/usr/lib/perl5/vendor_perl/5.8.6/DBIx/SearchBuilder/Handle/mysql.pm
E: perl-DBIx-SearchBuilder script-without-shellbang
/usr/lib/perl5/vendor_perl/5.8.6/DBIx/SearchBuilder/Handle/Pg.pm
E: perl-DBIx-SearchBuilder-Oracle script-without-shellbang
/usr/lib/perl5/vendor_perl/5.8.6/DBIx/SearchBuilder/Handle/Oracle.pm

Those files are all +x for some reason.

yum localinstall returns:
Error: Missing Dependency: perl(DBD::Oracle) is needed by package
perl-DBIx-SearchBuilder-Oracle

This one is interesting.  Based on the license text[1] for DBD::Oracle, I don't
expect it to ever show up in Extras.  So we would effectively be placing a
package in Extras that is uninstallable without ignoring dependencies or using
third-party repositories, right?

[1] http://search.cpan.org/dist/DBD-Oracle/Oracle.pm#COPYRIGHT

In addition to the license text, DBD-Oracle needs to be linked with the  
non-OSS Oracle client libs to work. 
 
Personally, I'd just leave the Oracle stuff in the main package here and 
filter out the dependency.  Or not ship it in the first place, either by 
removing the affected files, or leaving them in the subpackage but guarding it 
with a "--with oracle" rpmbuild option (defaulting to off). 

(In reply to comment #2)
>
> yum localinstall returns:
> Error: Missing Dependency: perl(DBD::Oracle) is needed by package
> perl-DBIx-SearchBuilder-Oracle
 
> So we would effectively be placing a
> package in Extras that is uninstallable without ignoring dependencies or using
> third-party repositories, right?
Right, this package has a runtime dependency to a perl-module/rpm outside of FE
and right, this means "pure FE+FC" users won't ever be able to install this rpm.

(In reply to comment #3)
> In addition to the license text, DBD-Oracle needs to be linked with the  
> non-OSS Oracle client libs to work. 
Exactly.

> Personally, I'd just leave the Oracle stuff in the main package here and 
> filter out the dependency.
I'd rather not do this, because this would mean to cheat to users and to
artificially add inconcistencies to the rpmdb.

>  Or not ship it in the first place, either by 
> removing the affected files, or leaving them in the subpackage but guarding it 
> with a "--with oracle" rpmbuild option (defaulting to off). 
This seems more reasonable to me.

Initially, I had considered doing this, but had decided to package the Oracle
bindings into a separate package as convenience to users which might have access
to Oracle DBs :)

I am going to add "--with oracle".

Updated package/spec with comments #2 and #3 reflected:
ftp://packman.iu-bremen.de/fedora/SRPMS/perl-DBIx-SearchBuilder.spec
ftp://packman.iu-bremen.de/fedora/SRPMS/perl-DBIx-SearchBuilder-1.27-2.src.rpm

Good:

- rpmlint clean
- package and spec file names okay
- license is okay, matches upstream
- spec is legible and in am. english
- source matches upstream
- builds cleanly in mock (FC-4 i386)
- no missing BRs, no superfluous BRs
- owns proper directories
- file permissions okay, %files okay

I'm happy with the "--with oracle" solution.  I'll consider this package approved.

(just changing the "depends" on FE-ACCEPT to a "blocks" FE-ACCEPT)