Spec URL: http://puiterwijk.fedorapeople.org//mediawiki123-openid.spec SRPM URL: http://puiterwijk.fedorapeople.org//mediawiki123-openid-4.4.0-1.fc21.src.rpm Description: Allow users with accounts on other OpenID-enabled sites to log in; if enabled, it also allows to use OpenID identities on other sites
Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=9522614 I'm only interrested in EPEL7.
I'll review this. Look for a full review in a bit...
OK - Package meets naming and packaging guidelines OK - Spec file matches base package name. OK - Spec has consistant macro usage. OK - Meets Packaging Guidelines. OK - License (GPLv2+) OK - 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: ed0eb917b29027ee751215c81cac1d8c OpenID-REL1_23-ece6550.tar.gz OK - BuildRequires correct 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. See below - 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. Changelog is messed up, should be version 4.4.0 (nonblocker) (Please fix on import) 2. Might ask upstream to include a copy of the gplv2. 3. rpmlint says: mediawiki123-openid.noarch: W: incoherent-version-in-changelog 1-1 ['4.4.0-1.el7', '4.4.0-1'] (would be fixed with fixing the changelog) mediawiki123-openid.noarch: E: incorrect-fsf-address /usr/share/mediawiki123/extensions/OpenID/SpecialOpenIDLogin.body.php mediawiki123-openid.noarch: E: incorrect-fsf-address /usr/share/mediawiki123/extensions/OpenID/SpecialOpenIDXRDS.body.php mediawiki123-openid.noarch: E: incorrect-fsf-address /usr/share/mediawiki123/extensions/OpenID/OpenIDProvider.body.php mediawiki123-openid.noarch: E: incorrect-fsf-address /usr/share/mediawiki123/extensions/OpenID/SpecialOpenIDConvert.body.php mediawiki123-openid.noarch: E: incorrect-fsf-address /usr/share/mediawiki123/extensions/OpenID/SpecialOpenID.body.php mediawiki123-openid.noarch: E: incorrect-fsf-address /usr/share/mediawiki123/extensions/OpenID/SpecialOpenIDIdentifier.body.php mediawiki123-openid.noarch: E: incorrect-fsf-address /usr/share/mediawiki123/extensions/OpenID/SpecialOpenIDServer.body.php mediawiki123-openid.noarch: E: incorrect-fsf-address /usr/share/mediawiki123/extensions/OpenID/OpenID.php mediawiki123-openid.noarch: E: incorrect-fsf-address /usr/share/mediawiki123/extensions/OpenID/optionToTable.php (You already mentioned you talked to upstream about this one) 2 packages and 0 specfiles checked; 9 errors, 1 warnings. 4. Shouldn't there be a Requires here on php-pear-Auth-OpenID ?
1. Fixed 2. Have asked upstream, in the same email as #3 4. Yes, there should, and some other libraries. Added them. New Spec URL: http://puiterwijk.fedorapeople.org//mediawiki123-openid.spec New SRPM URL: http://puiterwijk.fedorapeople.org//mediawiki123-openid-4.4.0-1.fc21.src.rpm
Looks good. APPROVED.
mediawiki123-openid-4.4.0-1.el7 has been submitted as an update for Fedora EPEL 7. https://admin.fedoraproject.org/updates/mediawiki123-openid-4.4.0-1.el7
mediawiki123-openid-4.4.0-1.el7 has been pushed to the Fedora EPEL 7 testing repository.