Review Request: yubikey-val - The YubiKey Validation Server

[Jeff Johnson]

tracker

[Dennis (dennis-redhat-bugs)]

Spec URL: http://ausil.us/packages/yubikey-val.spec
SRPM URL: http://ausil.us/packages/yubikey-val-2.7-1.fc14.src.rpm
Description: This is a server that validates Yubikey OTPs. It is written in PHP, for use with web servers such as Apache

[Ricky (ricky-redhat-bugs)]

I think this needs to require php-pdo. The user would then need to install the more specific php-pdo-* package for whatever DB they want to use with the package.

[Dennis (dennis-redhat-bugs)]

Spec URL: http://ausil.us/packages/yubikey-val.spec
SRPM URL: http://ausil.us/packages/yubikey-val-2.7-2.fc15.src.rpm

* Thu Jan 06 2011 Dennis Gilmore <email address hidden> - 2.7-2
- Requires php-pdo
- remove -template from config file name

[Ricky (ricky-redhat-bugs)]

OK - rpmlint clean

yubikey-val-2.7-2.fc14.src.rpm
yubikey-val.src: W: invalid-url Source0: http://yubikey-val-server-php.googlecode.com/files/yubikey-val-2.7.tgz HTTP Error 404: Not Found
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

yubikey-val-2.7-2.fc14.noarch.rpm
yubikey-val.noarch: E: non-readable /etc/ykval/ykval-config.php 0640L
yubikey-val.noarch: W: no-manual-page-for-binary ykval-export
yubikey-val.noarch: W: no-manual-page-for-binary ykval-queue
yubikey-val.noarch: W: no-manual-page-for-binary ykval-import
yubikey-val.noarch: W: no-manual-page-for-binary ykval-checksum-clients
1 packages and 0 specfiles checked; 1 errors, 4 warnings.

Non-readable error and google code warning are false positives.

OK - package meets naming guidelines
OK - spec file name matches base package name
OK - meets packaging guidelines
OK - meets licensing guidelines
OK - license field matches
OK - spec in American English
OK - spec is legible
OK - spec sources match upstream
OK - builds in mock
OK - no bundled libs
OK - owns its directories, does not own files owned by other packages
OK - does not list a file more than once
OK - permissions are OK
OK - consistently uses macros
OK - contains code or permissible content
OK - files in %doc do not affect runtime
OK - filenames valid UTF-8

SHOULD: If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream to
include it.

APPROVED

[Dennis (dennis-redhat-bugs)]

Package Name: yubikey-val
Short Description: The YubiKey Validation Server
Owners: ausil
Branches: el6 f13 f14
InitialCC:

[Dennis (dennis-redhat-bugs)]

New Package SCM Request
=======================
Package Name: yubikey-val
Short Description: The YubiKey Validation Server
Owners: ausil
Branches: el6 f13 f14
InitialCC:

[Dennis (dennis-redhat-bugs)]

Git done (by process-git-requests).

[Fedora (fedora-redhat-bugs)]

yubikey-val-2.7-2.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/yubikey-val-2.7-2.fc14

[Fedora (fedora-redhat-bugs)]

yubikey-val-2.7-2.fc13 has been submitted as an update for Fedora 13.
https://admin.fedoraproject.org/updates/yubikey-val-2.7-2.fc13

[Fedora (fedora-redhat-bugs)]

yubikey-val-2.7-2.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/yubikey-val-2.7-2.el6

[Fedora (fedora-redhat-bugs)]

yubikey-val-2.7-2.el6 has been pushed to the Fedora EPEL 6 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 yubikey-val'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/yubikey-val-2.7-2.el6

[Fedora (fedora-redhat-bugs)]

yubikey-val-2.7-2.el6 has been pushed to the Fedora EPEL 6 stable repository. If problems still persist, please make note of it in this bug report.

[Fedora (fedora-redhat-bugs)]

yubikey-val-2.7-2.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report.

[Fedora (fedora-redhat-bugs)]

yubikey-val-2.7-2.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.