Red Hat Bugzilla – Bug 183040
Review Request: perl-Kwiki
Last modified: 2007-11-30 17:11:25 EST
Spec Name or Url: http://ftp.kspei.com/pub/steve/rpms/perl-Kwiki/perl-Kwiki.spec
SRPM Name or Url: http://ftp.kspei.com/pub/steve/rpms/perl-Kwiki-0.38-1.src.rpm
A Wiki is a website that allows its users to add pages, and edit any
existing pages. It is one of the most popular forms of web collaboration.
If you are new to wiki, visit http://c2.com/cgi/wiki?WelcomeVisitors
which is possibly the oldest wiki, and has lots of information about how
Kwiki is a Perl wiki implementation based on the Spoon application
architecture and using the Spiffy object orientation model. The major goals
of Kwiki are that it be easy to install, maintain and extend.
All the features of a Kwiki wiki come from plugin modules. The base
installation comes with the bare minimum plugins to make a working Kwiki.
To make a really nice Kwiki installation you need to install additional
plugins. Which plugins you pick is entirely up to you. Another goal of
Kwiki is that every installation will be unique. When there are hundreds of
plugins available, this will hopefully be the case.
* Mon Feb 27 2006 Steven Pritchard <email@example.com> 0.32-2
- Drop explicit BR: perl.
- Filter perl(mixin) dependency.
Everything looks good except the Summary. Since all of the summaries are ending
up as "blah Perl module", I guess this is a product of the specfile generator
you're using. Unfortunately I'll have to carp about all of them.
For this module something like "A Perl wiki implementation" should be fine.
As usual, the spec is clean, source matches upstream, license is correct,
package builds fine in mock (+ dependencies not yet checked in). I wonder if
you need the explicit Requires: perl(Spoon) >= 0.22; extras is guaranteed to
have that version (once you check it in) so RPM should find the requirement on
its own. No blocker, though.
Approved, subject to additional of a reasonable summary.
Oops, I noted that perl-Kwiki requires perl(HTTP::BrowserDetect) which is not in
the repo and not under review. I happened to have it installed from the
RPMForge repo so it wasn't immediately noticeable until I tried to build a
dependent module in mock. So while this still builds, it can't be installed.
Would you like to submit the dependent module?
Regarding comment #2, yes, the "blah Perl module" thing comes from cpanspec. It
tries to guess %description (and gets it wrong most of the time), so I always
verify/correct that by hand, but I usually ignore Summary. Sorry about that.
Regarding comment #3, apparently I had HTTP::BrowserDetect in my local repo from
when I was working on another project. Oops. See bug #183667.