Bug 445977 - Review Request: ikiwiki - A wiki compiler
Summary: Review Request: ikiwiki - A wiki compiler
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Till Maas
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-05-10 21:41 UTC by Thomas Moschny
Modified: 2014-08-22 19:17 UTC (History)
3 users (show)

Fixed In Version: ikiwiki-3.20140613-2.el6
Clone Of:
Environment:
Last Closed: 2008-08-07 09:29:22 UTC
Type: ---
Embargoed:
opensource: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Thomas Moschny 2008-05-10 21:41:46 UTC
Spec URL: http://thm.fedorapeople.org/ikiwiki.spec
SRPM URL: http://thm.fedorapeople.org/ikiwiki-2.45-1.fc9.src.rpm

Description:
Ikiwiki is a wiki compiler. It converts wiki pages into HTML pages
suitable for publishing on a website. Ikiwiki stores pages and history
in a revision control system such as Subversion or Git. There are many
other features, including support for blogging, as well as a large
array of plugins.

Comment 1 Thomas Moschny 2008-05-14 08:12:20 UTC
Spec URL: http://thm.fedorapeople.org/ikiwiki.spec
SRPM URL: http://thm.fedorapeople.org/ikiwiki-2.46-1.fc9.src.rpm

%changelog
* Tue May 13 2008 Thomas Moschny <thomas.moschny> - 2.46-1
- Update to 2.46.


Comment 2 Thomas Moschny 2008-06-04 08:26:22 UTC
Spec URL: http://thm.fedorapeople.org/ikiwiki.spec
SRPM URL: http://thm.fedorapeople.org/ikiwiki-2.48-1.fc9.src.rpm

%changelog
* Mon Jun  2 2008 Thomas Moschny <thomas.moschny> - 2.48-1
- Update to 2.48.


Comment 3 Till Maas 2008-06-07 21:55:06 UTC
+ rpmlint: ok, warnings are about %{_exclude} sections

ikiwiki.src:100: E: hardcoded-library-path in
%{_prefix}/lib/w3m/cgi-bin/ikiwiki-w3m.cgi
ikiwiki.src:102: E: hardcoded-library-path in %{_prefix}/lib/ikiwiki/plugins

+ Naming: ok
+ License: ok, GPLv2+
+ Source file: ok, matches upstream:
6b293f6e8a08578533d0268b25dae5b3  ikiwiki_2.48.tar.gz
There was a new release just now:
http://git.ikiwiki.info/?p=ikiwiki;a=commitdiff;h=77edb81bee5a2c4966591d7ba1f3c83e31de272f
+ compiles on F8: ok
+ BuildRequires: ok, builds in F9 mock
+ Locale handling: ok, uses %find_lang
+ Libraries: ok, there are none
+ Directory Ownage: ok
+ Permissions: ok
+ %clean: ok
+ %install: ok

- License included: not ok, the license is included in ikiwiki/doc/GPL but not
in %doc
- Perl Requires: not ok

perl(File::MimeInfo) is twice in the requires of the rpm, it is added manually
in the spec, which I guess can be removed.

Also I would probably package ikiwiki-w3m.cgi, because it seems to be useful.

Another isssue is, that the directory %{_prefix}/lib/ikiwiki/plugins should be
probably created, to allow addon packages, that contain ikiwiki modules. Or
would the need the perl(RPC) dependency?

Comment 4 Thomas Moschny 2008-06-16 08:11:34 UTC
Spec URL: http://thm.fedorapeople.org/ikiwiki.spec
SRPM URL: http://thm.fedorapeople.org/ikiwiki-2.50-1.fc9.src.rpm

%changelog
* Sun Jun 15 2008 Thomas Moschny <thomas.moschny> - 2.50-1
- Update to 2.50.
- Move ikiwiki-w3m.cgi into a subpackage.
- Add ikiwiki's own documentation.
- Remove duplicate requirement perl(File::MimeInfo).
- Minor cleanups.

(In reply to comment #3)
> + Source file: ok, matches upstream:
> 6b293f6e8a08578533d0268b25dae5b3  ikiwiki_2.48.tar.gz
> There was a new release just now:

Updated to 2.50.

> - License included: not ok, the license is included in ikiwiki/doc/GPL but not
> in %doc

Forgot to add ikiwiki's own documentation, fixed (html dir in %doc). This also
includes the GPL file.

> - Perl Requires: not ok
> 
> perl(File::MimeInfo) is twice in the requires of the rpm, it is added manually
> in the spec, which I guess can be removed.

Fixed.

> Also I would probably package ikiwiki-w3m.cgi, because it seems to be useful.

Included in a subpackage now.

> Another isssue is, that the directory %{_prefix}/lib/ikiwiki/plugins should be
> probably created, to allow addon packages, that contain ikiwiki modules. Or
> would the need the perl(RPC) dependency?

Yes it would. This directory is for non-perl plugins ikiwiki talks to using
xmlrpc, as far as I understood. As soon as there is perl(XML), I'm going to
include that dir and probably the demo plugins.

Comment 5 Thomas Moschny 2008-06-20 17:16:14 UTC
perl-XML-RPC is reviewed in bug 435835, btw.

Comment 6 Thomas Moschny 2008-06-27 18:48:45 UTC
Is there something blocking the review?

Comment 7 Till Maas 2008-07-03 16:40:59 UTC
(In reply to comment #6)
> Is there something blocking the review?

I hoped that perl-RPC-XML would be ready, soon. There is one minor issue, the
iconv invocation truncates the file to size zero, you need to put the output in
a temporary file and then mv it to the input file. Please change this before the
first build. This package is APPROVED.

Comment 8 Thomas Moschny 2008-07-03 19:40:07 UTC
Thanks for the review!

New Package CVS Request
=======================
Package Name: ikiwiki
Short Description: A wiki compiler
Owners: thm
Branches: F-8 F-9
InitialCC: none
Cvsextras Commits: yes

Comment 9 Kevin Fenzi 2008-07-04 19:48:58 UTC
cvs done.

Comment 10 Fedora Update System 2008-07-06 16:30:47 UTC
ikiwiki-2.51-1.fc9 has been submitted as an update for Fedora 9

Comment 11 Fedora Update System 2008-07-06 19:02:28 UTC
ikiwiki-2.51-1.fc8 has been submitted as an update for Fedora 8

Comment 12 Fedora Update System 2008-07-09 02:49:07 UTC
ikiwiki-2.51-1.fc8 has been pushed to the Fedora 8 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update ikiwiki'.  You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F8/FEDORA-2008-6214

Comment 13 Fedora Update System 2008-07-17 14:18:18 UTC
ikiwiki-2.51-1.fc9 has been pushed to the Fedora 9 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 14 Fedora Update System 2008-07-17 14:21:00 UTC
ikiwiki-2.51-1.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 15 Thomas Moschny 2008-08-07 09:29:22 UTC
Closing.

Comment 16 Jens Petersen 2012-12-11 03:31:15 UTC
Any chance of building also for EPEL 6?

Comment 17 Thomas Moschny 2012-12-12 13:25:48 UTC
Need to check dependencies.

Also, ikiwiki is (or at least used to be) a package with a fast release pace, which is contradictory to EPEL policies. We would have to carefully pick one version, for example the one in Debian stable. Would that be ok?

Comment 18 Jens Petersen 2012-12-13 01:56:26 UTC
Sure I think so - well I just asked since it is a build dependency
for git-annex's documentation but for now I just disabled it.

Comment 19 Christopher Meng 2014-01-27 04:49:48 UTC
I'd like to see this software in EPEL6/7.

Thomas, any chances for EPEL6/7 now?

Thanks.

Comment 20 Thomas Moschny 2014-07-24 09:09:12 UTC
perl-Text-Markdown and perl-CGI-FormBuilder are still not in EPEL7.

Ok, so let's go for EPEL6 at least now.

Package Change Request
======================
Package Name: ikiwiki
New Branches: el6
Owners: thm

Comment 21 Gwyn Ciesla 2014-07-24 12:19:37 UTC
Git done (by process-git-requests).

Comment 22 Jens Petersen 2014-07-25 01:54:30 UTC
Thanks

For now I turned off ikiwiki doc generation in git-annex/epel7 BTW,
but I guess it would still be nice to have it probably.

Have epel7 branches been requested for
perl-Text-Markdown and perl-CGI-FormBuilder?

Comment 23 Thomas Moschny 2014-07-25 08:46:34 UTC
(In reply to Jens Petersen from comment #22)
> Have epel7 branches been requested for
> perl-Text-Markdown and perl-CGI-FormBuilder?

Sure, see bug 243716 and bug 244262.

I had also sent their respective owners a mail ~1 month ago.

Comment 24 Fedora Update System 2014-07-25 17:19:13 UTC
ikiwiki-3.20140613-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/ikiwiki-3.20140613-1.el6

Comment 25 Fedora Update System 2014-08-22 19:17:54 UTC
ikiwiki-3.20140613-2.el6 has been pushed to the Fedora EPEL 6 stable repository.


Note You need to log in before you can comment on or make changes to this bug.