Bug 231758 (perl-Workflow)

Note the rpmlint warning of:

[cweyl@zeus noarch]$ rpmlint perl-Workflow-0.26-1.fc6.noarch.rpm 
W: perl-Workflow doc-file-dependency
/usr/share/doc/perl-Workflow-0.26/eg/ticket/App/Web.pm perl(XML::Simple)

...is expected.  perl(XML::Simple) is an actual requirement.

*** Bug 232734 has been marked as a duplicate of this bug. ***

Your package is requiring perl(Data::UUID). This seems to be problematic which
is why I filtered the requirement from my version.
The license.txt for Data::UUID states:

"The license under which this code was originally released is unknown, although
its release has been interpreted as its release under an open source license.
Attempts to contact the original author are underway. -- rjbs, 2006-11-14"

I'd suggest you do filter that requirement as it is optional according to the
readme. Alternatively, review my package. :D Less work for you.

About the docs, I solved that problem by just running tar and gzip on the
example file and packaging that tar.gz file.

(In reply to comment #3)
> Your package is requiring perl(Data::UUID). This seems to be problematic which
> is why I filtered the requirement from my version.
> The license.txt for Data::UUID states:
> 
> "The license under which this code was originally released is unknown, although
> its release has been interpreted as its release under an open source license.
> Attempts to contact the original author are underway. -- rjbs, 2006-11-14"
> 
> I'd suggest you do filter that requirement as it is optional according to the
> readme. Alternatively, review my package. :D Less work for you.

Data::UUID I didn't worry about too much -- it's in extras, so presumptively the
license is OK.  If a package license is under question, then a bug should be
filed, an audit conducted, etc; otherwise I'm going to trust, in good faith, the
reviewer, packager, and "rjbs" :) got it right.

> About the docs, I solved that problem by just running tar and gzip on the
> example file and packaging that tar.gz file.

Yeah, I've never actually been a fan of that...  if the docs are too large to
reasonably keep with the base package, then a -docs subpackage is in order.  The
point is well taken though -- looking back at the package, it's ~400k installed,
~220k of which is %doc.  So, that's a large percentage of the package taken up
by %doc, but still a fairly small amount of disk space overall.  I could easily
be talked into splitting them out into -docs.

I think you should add perl(Data::UUID) and SPOPS as make test showed me
t/persister_spops..................skipped
        all skipped: SPOPS not installed
t/persister_uuid...................skipped
        all skipped: Data::UUID not installed

Added perl(Data::UUID) and perl(SPOPS) as BRs; however, perl(SPOPS) is commented
out as it isn't in Fedora yet, and depends on perl(Class::Fields) whose tests
fail.  If Class-Fields cleans itself up and perl(SPOPS) makes it into Fedora,
I'll uncomment it...

SRPM URL: http://home.comcast.net/~ckweyl/perl-Workflow-0.26-2.fc6.src.rpm
SPEC URL: http://home.comcast.net/~ckweyl/perl-Workflow.spec

Is perl-Class-Fields and perl(SPOPS) is in Fedora or its submitted for review?

(In reply to comment #7)
> Is perl-Class-Fields and perl(SPOPS) is in Fedora or its submitted for review?

Neither are.  perl-SPOPS would depend on perl-Class-Fields, and
perl-Class-Fields fails its own tests currently.

ping?

Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
e9c25c357b0e2a62fa91493b818a538f  Workflow-0.26.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test
t/action...........................ok
t/action_field.....................ok
t/action_null......................ok
t/base.............................ok
t/condition........................ok
t/condition_evaluate...............ok
t/config...........................Use of uninitialized value in numeric eq (==)
at t/config.t line 41.
ok
t/context..........................ok
t/exception........................ok
t/factory..........................ok
t/factory_subclass.................ok
t/history..........................ok
t/persister_dbi....................Use of uninitialized value in concatenation
(.) or string at /usr/lib/perl5/vendor_perl/5.8.8/DBD/Mock.pm line 531.
ok
t/persister_dbi_extra_data.........ok
t/persister_file...................ok
t/persister_random_id..............ok
t/persister_spops..................skipped
        all skipped: SPOPS not installed
t/persister_uuid...................skipped
        all skipped: Data::UUID not installed
t/state............................ok
t/validator........................ok
t/validator_has_required_field.....ok
t/validator_in_enumerated_type.....ok
t/validator_matches_date_format....ok
t/workflow.........................Use of uninitialized value in concatenation
(.) or string at /usr/lib/perl5/vendor_perl/5.8.8/DBD/Mock.pm line 531.
ok
All tests successful, 2 tests skipped.
Files=24, Tests=192, 10 wallclock secs ( 7.04 cusr +  0.79 csys =  7.83 CPU)

+ Provides: perl(Workflow) perl(Workflow::Action)
perl(Workflow::Action::InputField) perl(Workflow::Action::Null)
perl(Workflow::Base) perl(Workflow::Condition)
perl(Workflow::Condition::Evaluate) perl(Workflow::Condition::HasUser)
perl(Workflow::Config) perl(Workflow::Config::Perl) perl(Workflow::Config::XML)
perl(Workflow::Context) perl(Workflow::Exception) perl(Workflow::Factory)
perl(Workflow::History) perl(Workflow::Persister) perl(Workflow::Persister::DBI)
perl(Workflow::Persister::DBI::AutoGeneratedId)
perl(Workflow::Persister::DBI::ExtraData)
perl(Workflow::Persister::DBI::SequenceId) perl(Workflow::Persister::File)
perl(Workflow::Persister::RandomId) perl(Workflow::Persister::SPOPS)
perl(Workflow::Persister::UUID) perl(Workflow::State) perl(Workflow::Validator)
perl(Workflow::Validator::HasRequiredField)
perl(Workflow::Validator::InEnumeratedType)
perl(Workflow::Validator::MatchesDateFormat)
+ Requires: perl(:MODULE_COMPAT_5.8.8) perl(DBI) perl(Data::Dumper)
perl(Data::UUID) perl(DateTime) perl(DateTime::Format::Strptime)
perl(Exception::Class) perl(File::Spec::Functions) perl(Log::Log4perl)
perl(Safe) perl(Workflow) perl(Workflow::Action)
perl(Workflow::Action::InputField) perl(Workflow::Condition)
perl(Workflow::Condition::Evaluate) perl(Workflow::Config)
perl(Workflow::Context) perl(Workflow::Exception) perl(Workflow::Factory)
perl(Workflow::History) perl(Workflow::Persister)
perl(Workflow::Persister::DBI::AutoGeneratedId)
perl(Workflow::Persister::DBI::SequenceId) perl(Workflow::Persister::RandomId)
perl(Workflow::State) perl(Workflow::Validator)
perl(Workflow::Validator::HasRequiredField) perl(XML::Simple) perl(base)
perl(constant) perl(strict)

APPROVED.

New Package CVS Request
=======================
Package Name: perl-Workflow
Short Description: Simple, flexible system to implement workflows
Owners: cweyl.edu
Branches: FC-5, FC-6, devel
InitialCC: fedora-perl-devel-list

Imported, built, and all that goodness.  Thanks for the review! :)