Please post SRPM.

Spec URL: http://www.hardakers.net/FE/perl-Crypt-OpenSSL-DSA.spec
SRPM URL: http://www.hardakers.net/FE/perl-Crypt-OpenSSL-DSA-0.13-1.src.rpm

Spec URL: http://www.hardakers.net/FE/perl-Crypt-OpenSSL-DSA.spec
SRPM URL: http://www.hardakers.net/FE/perl-Crypt-OpenSSL-DSA-0.13-2.src.rpm

- Add BuildRequire openssl-devel
- Don't manually require openssl
- Use vendorarch instead of vendorlib

Source0: URL is incorrect; should be
http://search.cpan.org/CPAN/authors/id/T/TJ/TJMATHER/Crypt-OpenSSL-DSA-%{version}.tar.gz
I find it useful to run spectool -g just to make sure that the Source URLs in
the spec can be fetched without error.

Blah BuildRequires: perl(ExtUtils::MakeMaker) blah.  Annoying, isn't it?  But as
soon as F7 branches, perl will lose the perl-devel dependency things packages
that don't pull in all of their build-time requirements will break, so we're
trying to catch them all in review and prevent pain later.  And again I'm not
sure whether you'll need perl(Test) or not.

You get better test coverage if you have a build-time dependency on
perl(Digest::SHA1).  The package itself doesn't use it, but one of the tests does.

* source files match upstream:
   f786a76441106a6ce3de4d153bddd3fc0a8a90657b2cb6778aa362e4e7d66cef  
   Crypt-OpenSSL-DSA-0.13.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text not included upstream.
* latest version is being packaged.
X BuildRequires are proper.
* compiler flags are appropriate.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly
* debuginfo package looks complete.
* rpmlint is silent.
* final provides and requires are sane:
   DSA.so()(64bit)
   perl(Crypt::OpenSSL::DSA) = 0.13
   perl-Crypt-OpenSSL-DSA = 0.13-2.fc7
  =
   libcrypto.so.6()(64bit)
   libssl.so.6()(64bit)
   perl(:MODULE_COMPAT_5.8.8)
   perl(DynaLoader)
   perl(strict)
   perl(vars)
   perl(warnings)

* %check is present and all tests pass:
   (with a build dependency on Digest::SHA1 added)
   All tests successful.
   Files=3, Tests=115,  1 wallclock secs ( 0.62 cusr +  0.18 csys =  0.80 CPU)

* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no static libraries.
* no libtool .la droppings.

Spec URL: http://www.hardakers.net/FE/perl-Crypt-OpenSSL-DSA.spec
SRPM URL: http://www.hardakers.net/FE/perl-Crypt-OpenSSL-DSA-0.13-3.src.rpm

- BuildRequire perl(Test) perl(ExtUtils::MakeMaker) perl(Digest::SHA1)
  and perl(File::Temp)
- Fixed source code URL

(sorry for the delay on a response; I was out of town at the end of last week)\

There was a number of other modules in some of the packages (including this one)
that the test system requires.  It's unclear what the policy is on these (like
File::Temp).  So...  I've added the other needed module as well.

I was out of town over the weekend as well, so no worries.

Unfortunately it's not really clear just what is going to be split out of the
main Perl package.  It may end up that all core modules are split out; this
would make it a simple matter to update them separately and all packages should
already have all of the runtime dependencies sorted anyway.  But the build-time
dependencies would have to be added manually.

Currently, all I know is that perl(CPAN), perl(ExtUtils::Embed),
perl(ExtUtils::MakeMaker), perl(Test::Harness), perl(Test::Simple) and
perl(Test::More) are split out as of F7, but the base perl package contains a
dependency on them so that all Perl packages don't break.  This will go away
immediately after F7 branches.  I don't know if perl(Test) is being split;
currently it isn't but I suppose it should be given the other modules that are
split.  I don't think anything else is being split at this time, although
certainly adding those dependencies doesn't actually hurt anything and I'm not
going to block because of them.

So we're good at this point; go ahead and apply for sponsorship and I'll take
care of that.  Once that's done, you'll be able to set the fedora-cvs flag and
get your packages checked in and built.

account created on my side, FYI.

FYI, the wiki instructions still say to use "cvsextras"
(http://fedoraproject.org/wiki/PackageMaintainers/Join step 4) which is a bit
confusing post merge.  I assumed that was still the proper thing to do, but
either way I'd change the text of the instructions to say it's still the correct
process even post merge.

Sponsorship should be done now, but it may take an hour or so to propagate.

The group is still cvsextras but will be renamed soon.

New Package CVS Request
=======================
Package Name: perl-Crypt-OpenSSL-DSA
Short Description: Perl OpenSSL bindings for DSA support
Owners: wjhns174
Branches: FC-6 F-7
InitialCC: 

CVS done.

Package Change Request
======================
Package Name: perl-Crypt-OpenSSL-DSA
New Branches: el5 el6
Owners: hardaker

Git done (by process-git-requests).

Package Change Request
======================
Package Name: perl-Crypt-OpenSSL-DSA
New Branches: el7
Owners: hardaker

Git done (by process-git-requests).