Red Hat Bugzilla – Bug 820452
Review Request: mediawiki119 - updated mediawiki for EPEL-6
Last modified: 2014-01-27 08:00:25 EST
Spec URL: http://smooge.fedorapeople.org/Downloads/mediawiki119.spec
SRPM URL: http://smooge.fedorapeople.org/Downloads/mediawiki119-1.19.0-2.el6.src.rpm
Description: MediaWiki is the software used for Wikipedia and the other Wikimedia Foundation websites. Compared to other wikis, it has an excellent
range of features and support for high-traffic websites using multiple
This software is for RHEL-6 EPEL branch.
I'll go ahead and review this.
Look for a full review in the next few days.
OK - Package meets naming and packaging guidelines
OK - Spec file matches base package name.
OK - Spec has consistant macro usage.
OK - Meets Packaging Guidelines.
See below - License
See below - License field in spec matches
See below - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
OK - BuildRequires correct
OK - Package has %defattr and permissions on files is good.
OK - Package has a correct %clean section.
OK - Package has correct buildroot
OK - Package is code or permissible content.
OK - Packages %doc files don't affect runtime.
OK - Package has rm -rf RPM_BUILD_ROOT at top of %install
OK - Package compiles and builds on at least one arch.
OK - Package has no duplicate files in %files.
OK - Package doesn't own any directories other packages own.
OK - Package owns all the directories it creates.
OK - Package obey's FHS standard (except for 2 exceptions)
See below - No rpmlint output.
OK - final provides and requires are sane.
OK - Should build in mock.
OK - Should build on all supported archs
OK - Should function as described.
OK - Should have sane scriptlets.
OK - Should have dist tag
OK - Should package latest version
OK - Should not use file requires outside of /etc, /bin, /sbin, /usr/bin, or /usr/sbin
1. From the README, it looks like the License here should be:
GPLv2+ and public domain and CC-BY-SA
and looking at:
"MediaWiki use Creative Commons license marks to points to their online
licenses. This content is trademarked and used under a specific license
available at http://creativecommons.org/policies#trademark
The restricted content is:
I think we need to remove those two files since the link says:
"You are authorized to use our trademarks on the terms and conditions below,
and only on the further condition that you download the trademarks directly from our website."
Looking at the actual files, I see:
maintenance/cssjanus/ => ASL 2.0
resources/jquery.effects/ => MIT or GPLv2+
resources/jquery/jquery.appear.js => MIT
resources/jquery.ui/ => MIT or GPLv2+
resources/jquery/ => MIT or GPLv2+
resources/jquery.tipsy => MIT
resources/jquery.ui => MIT or GPLv2+
skins/vector/csshover.htc => LGPLv2+
skins/common/wikibits.js => says:
" Written by Jonathan Snook, http://www.snook.ca/jonathan
Add-ons by Robert Nyman, http://www.robertnyman.com
Author says "The credit comment is all it takes, no license. Go crazy with it!:-)"
So, provided we can figure out if the wikibits.js is ok, I think it should be:
GPLv2+ and MIT and public domain and CC-BY-SA and ASL2.0
2. rpmlint says:
Might rm -f these:
mediawiki119.noarch: W: hidden-file-or-dir /usr/share/mediawiki119/extensions/Gadgets/.gitreview
mediawiki119.noarch: W: hidden-file-or-dir /usr/share/mediawiki119/extensions/ParserFunctions/.gitreview
mediawiki119.noarch: W: hidden-file-or-dir /usr/share/mediawiki119/extensions/Vector/.gitreview
mediawiki119.noarch: E: version-control-internal-file /usr/share/mediawiki119/extensions/.gitignore
mediawiki119.noarch: W: hidden-file-or-dir /usr/share/mediawiki119/extensions/WikiEditor/.gitreview
mediawiki119.noarch: W: hidden-file-or-dir /usr/share/mediawiki119/extensions/Renameuser/.gitreview
mediawiki119.noarch: E: version-control-internal-file /usr/share/doc/mediawiki119-1.19.0/docs/html/.gitignore
mediawiki119.noarch: W: hidden-file-or-dir /usr/share/mediawiki119/extensions/Nuke/.gitreview
mediawiki119.noarch: W: hidden-file-or-dir /usr/share/mediawiki119/extensions/ConfirmEdit/.gitreview
Might chmod 755 these:
mediawiki119.noarch: E: non-executable-script /usr/share/mediawiki119/includes/normal/UtfNormalTest2.php 0644L /usr/bin/php
mediawiki119.noarch: E: non-executable-script /usr/share/mediawiki119/extensions/ConfirmEdit/captcha.py 0644L /usr/bin/python
can be ignored:
mediawiki119.noarch: W: install-file-in-docs /usr/share/doc/mediawiki119-1.19.0/INSTALL
I don't see any further blockers aside from the fixes for rpmlint and the legal question.
I'll add legal here for comment.
I've confirmed with Jonathan Snook and Robert Nyman via email that we have permission to distribute their code (the part of wikibits.js which is problematic) under CC-BY-SA 2.5.
I've reached out to Creative Commons to try to come up with some sort of resolution to that issue, please stay tuned.
Diane Peters, General Counsel for Creative Commons has agreed to except the distribution made by Fedora of the unmodified Creative Commons trademarked logo files as distributed in Mediawiki.
This resolves the issue with the trademark terms to my satisfaction, so I'm lifting FE-Legal.
So, the only outstanding things I see are the rpmlint fixes, which are pretty minor and updating the License field.
Please do those before checking in, but otherwise this package is APPROVED.
New Package SCM Request
Package Name: mediawiki119
Short Description: A wiki engine
Owners: smooge ianweller
Git done (by process-git-requests).
Looks like this has been built but no update filed. What's up?
mediawiki119-1.19.1-1.el6 has been submitted as an update for Fedora EPEL 6.
* should fix your issue,
* was pushed to the Fedora EPEL 6 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=epel-testing mediawiki119-1.19.1-1.el6'
as soon as you are able to.
Please go to the following url:
then log in and leave karma (feedback).
mediawiki119-1.19.1-1.el6 has been pushed to the Fedora EPEL 6 stable repository. If problems still persist, please make note of it in this bug report.
Package Change Request
Package Name: mediawiki119
New Branches: el5