Bug 428780

This one build OK and rpmlint is silent.

I find it odd that there's no %check section even though there are tests.  Did
you remove it, or did cpanspec not include it for some reason?  I added
  %check
  ./Build test verbose=1
and dependencies on perl(File::Find::Rule) and perl(Test::Pod) and the tests failed:

Failed Test  Stat Wstat Total Fail  Failed  List of Failed
-------------------------------------------------------------------------------
t/01-basic.t    1   256    12    1   8.33%  11
t/03-multi.t    1   256     7    1  14.29%  4
Failed 2/3 test scripts, 33.33% okay. 2/21 subtests failed, 90.48% okay.

You should investigate why these tests are failing and try to understand the
impact of the failures.  I looked at the CPAN testing report and see that other
testers have seen the same test failures.

Checklist:
* source files match upstream:
   30b0e6acb71ec790a8adbbec065f96093a7537233fb8c401491ac2e86df4484c  
   MasonX-Request-WithApacheSession-0.30.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* 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.
* BuildRequires are proper.
* %clean is present.
* package builds in mock (rawhide, x86_64).
* package installs properly
* rpmlint is silent.
* final provides and requires are sane:
   perl(MasonX::Request::WithApacheSession) = 0.30
   perl(MasonX::Request::WithMultiSession)
   perl-MasonX-Request-WithApacheSession = 0.30-1.fc9
  =
   perl >= 0:5.005
   perl(:MODULE_COMPAT_5.8.8)
   perl(Apache::Session::Wrapper) >= 0.13
   perl(Digest::SHA1)
   perl(HTML::Mason) >= 1.16
   perl(HTML::Mason::Exceptions)
   perl(HTML::Mason::Request)
   perl(Params::Validate)
   perl(Time::HiRes)
   perl(base)
   perl(strict)
   perl(vars)

X %check is not present even though a test suite is in the tarball.  You need to 
   run upstream test suites whenever possible.
* 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.

Yes, you are correct about the %check section.  When I was packaging it, I had
problems with the %check section.  Just to see if it would build - I temporarily
removed that section and just forgot to add it back.  I will add it back and
correct whatever is causing the make test to fail.  Thanks.

I have added the %check back into the spec file.  In order to be able to build
the package without the tests failing, I had to apply a patch that I found on a
Mason mailing list (http://permalink.gmane.org/gmane.comp.web.mason.devel/1648).

The new urls are: 

Spec:
http://niscripts.googlecode.com/svn/trunk/rpms/SPECS/perl-MasonX-Request-WithApacheSession.spec
SRPM:
http://niscripts.googlecode.com/svn/trunk/rpms/SRPMS/perl-MasonX-Request-WithApacheSession-0.30-2.fc8.src.rpm

Hmm, that failed to build for me:

Patch #0 (MasonX-Request-WithApacheSession-0.30_subrequests.patch):
+ patch -s
Reversed (or previously applied) patch detected!  Assume -R? [n]
Apply anyway? [n]
1 out of 1 hunk ignored -- saving rejects to file
lib/MasonX/Request/WithApacheSession.pm.rej
error: Bad exit status from /var/tmp/rpm-tmp.48599 (%prep)

Crap - I am sorry.  It was pretty late last night when I was working on this. I
had the patch file messed up.  Try the below files.  It seems to build ok now
and rpmlint reports no errors.

SPEC:
http://niscripts.googlecode.com/svn/trunk/rpms/SPECS/perl-MasonX-Request-WithApacheSession.spec
SRPM:
http://niscripts.googlecode.com/svn/trunk/rpms/SRPMS/perl-MasonX-Request-WithApacheSession-0.30-3.fc8.src.rpm

OK, that builds and the tests pass.

If you look at the test output, you'll see that some tests are skipped because
perl(File::Find::Rule) and perl(Test::Pod) are not installed.  We try to get
complete test coverage whenever possible, even if it means additional build-time
dependencies.

I'll go ahead and approve this package, and you can add build dependencies on
those two modules when you check in.

APPROVED

This should conclude the review process; you should now apply for cvsextras
access in the account system and I'll sponsor you.  Let me know if you need
assistance with the rest of the process.

Thanks - I will add the dependencies that you mention above. 

So, does this mean that all three of my packages are approved?  I have not heard
back from the developers regarding the SNMP-Info MIB issue...   Also, you said
you will sponsor me?  That is great news!!  I appreciate your patience and
assistance in reviewing my packages.

If I have questions that come up during the rest of the process I may shoot you
an e-mail (if you do not mind).  Thanks again.

Walter  

Yes, they're all approved, and you should proceed with getting things set up in
the account system.  We can work out the MIB issue after the fact.

Feel free to ping me here, via email or in IRC if you need assistance.

New Package CVS Request
=======================
Package Name: perl-MasonX-Request-WithApacheSession
Short Description: Integrates Apache::Session into Mason
Owners: gouldwp
Branches: F-8 EL-5
InitialCC: 
Cvsextras Commits: yes

cvs done.

perl-MasonX-Request-WithApacheSession-0.30-4.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.

perl-MasonX-Request-WithApacheSession-0.30-6.el5 has been submitted as an update for Fedora EPEL 5.
http://admin.fedoraproject.org/updates/perl-MasonX-Request-WithApacheSession-0.30-6.el5

perl-MasonX-Request-WithApacheSession-0.30-6.el5 has been pushed to the Fedora EPEL 5 stable repository.  If problems still persist, please make note of it in this bug report.