Bug 820452 - Review Request: mediawiki119 - updated mediawiki for EPEL-6
Summary: Review Request: mediawiki119 - updated mediawiki for EPEL-6
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Kevin Fenzi
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-05-10 00:38 UTC by Stephen John Smoogen
Modified: 2014-01-27 13:00 UTC (History)
7 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2012-08-05 06:40:45 UTC
Type: ---
Embargoed:
kevin: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Stephen John Smoogen 2012-05-10 00:38:46 UTC
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
servers. 

This software is for RHEL-6 EPEL branch.

Comment 1 Kevin Fenzi 2012-05-10 16:45:46 UTC
I'll go ahead and review this. 

Look for a full review in the next few days.

Comment 2 Kevin Fenzi 2012-05-10 21:17:51 UTC
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:
f33dd8be095d6ed3f33b2e422d7b61b6  mediawiki-1.19.0.tar.gz
f33dd8be095d6ed3f33b2e422d7b61b6  mediawiki-1.19.0.tar.gz.orig

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.

SHOULD Items:

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

Issues: 

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:

* skins/common/images/cc-by-nc-sa.png
* skins/common/images/cc-by-sa.png
"

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!:-)"
        From http://www.robertnyman.com/2005/11/07/the-ultimate-getelementsbyclassname/
"

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.

Comment 3 Tom "spot" Callaway 2012-05-15 16:22:28 UTC
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.

Comment 4 Tom "spot" Callaway 2012-05-15 17:24:00 UTC
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.

Comment 5 Kevin Fenzi 2012-05-15 17:31:01 UTC
Excellent. Thanks!

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.

Comment 6 Stephen John Smoogen 2012-05-23 19:38:59 UTC
New Package SCM Request
=======================
Package Name: mediawiki119 
Short Description: A wiki engine
Owners: smooge ianweller
Branches: el6 
InitialCC:

Comment 7 Gwyn Ciesla 2012-05-24 14:47:56 UTC
Git done (by process-git-requests).

Comment 8 Orion Poplawski 2012-07-16 17:15:26 UTC
Looks like this has been built but no update filed.  What's up?

Comment 9 Fedora Update System 2012-07-16 18:23:00 UTC
mediawiki119-1.19.1-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/mediawiki119-1.19.1-1.el6

Comment 10 Fedora Update System 2012-07-19 22:35:31 UTC
Package mediawiki119-1.19.1-1.el6:
* 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:
https://admin.fedoraproject.org/updates/FEDORA-EPEL-2012-6422/mediawiki119-1.19.1-1.el6
then log in and leave karma (feedback).

Comment 11 Fedora Update System 2012-08-05 06:40:45 UTC
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.

Comment 12 Patrick Uiterwijk 2014-01-26 13:06:36 UTC
Package Change Request
======================
Package Name: mediawiki119
New Branches: el5
Owners: puiterwijk
InitialCC:

Comment 13 Gwyn Ciesla 2014-01-27 13:00:25 UTC
Git done (by process-git-requests).


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