[Bug 469816] Review Request: perl-App-Cmd - Write command line apps with less suffering

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=469816


Jason Tibbitts <tibbs@xxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+




--- Comment #1 from Jason Tibbitts <tibbs@xxxxxxxxxxx>  2008-11-14 11:13:53 EDT ---
Gah, this still doesn't build on rawhide becaise IO::TieCombine isn't there. 
And it doesn't build on F9 either; the tests fail from lack of Test::More.

I went ahead and added the Test::More build dep and built on F9. 

The inclusion of the test suite makes the doc directory larger than the rest of
the package, but the package is small so I don't see that it makes much
difference in the end.

I'll go ahead and approve this but of course you'll need to fix the
dependencies in order to build it.

* source files match upstream:
  a3a240fb8e680417e9610790112ee80129cfec65a259f97cf77832009f3a06a4
  App-Cmd-0.202.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 included in package.
* latest version is being packaged.
X BuildRequires are proper (needs Test::More)

Note: added BuildRequires: perl(Test::More) to complete this review:

* %clean is present.
* package builds in mock (F9, x86_64) after adding build dep.
* package installs properly.
* rpmlint is silent.
* final provides and requires are sane:
   perl(App::Cmd) = 0.202
   perl(App::Cmd::ArgProcessor) = 0.202
   perl(App::Cmd::Command) = 0.202
   perl(App::Cmd::Command::commands) = 0.202
   perl(App::Cmd::Command::help) = 0.202
   perl(App::Cmd::Plugin)
   perl(App::Cmd::Setup)
   perl(App::Cmd::Simple) = 0.202
   perl(App::Cmd::Subdispatch) = 0.202
   perl(App::Cmd::Subdispatch::DashedStyle) = 0.202
   perl(App::Cmd::Tester)
   perl(App::Cmd::Tester::Result)
   perl-App-Cmd = 0.202-1.fc9
  =
   perl >= 0:5.006
   perl(:MODULE_COMPAT_5.10.0)
   perl(App::Cmd)
   perl(App::Cmd::ArgProcessor)
   perl(App::Cmd::Command)
   perl(App::Cmd::Plugin)
   perl(App::Cmd::Subdispatch)
   perl(Carp)
   perl(Data::OptList)
   perl(File::Basename)
   perl(Module::Pluggable::Object)
   perl(Sub::Exporter)
   perl(Sub::Exporter::Util)
   perl(Sub::Install)
   perl(strict)
   perl(warnings)

* %check is present and all tests pass:
   All tests successful.
   Files=6, Tests=42,  1 wallclock secs ( 0.34 cusr +  0.04 csys =  0.38 CPU)

* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* no generically named files
* code, not content.
* documentation is larger than the library, but still small so no subpackage is 
   needed.
* %docs are not necessary for the proper functioning of the package.

APPROVED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]