Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: yubikey-val - The YubiKey Validation Server
https://bugzilla.redhat.com/show_bug.cgi?id=637213
Summary: Review Request: yubikey-val - The YubiKey Validation Server Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: dennis@ausil.us QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=637213
Ricky Zhou rzhou@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |rzhou@redhat.com
--- Comment #1 from Ricky Zhou rzhou@redhat.com 2010-09-27 17:40:40 EDT --- 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.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=637213
--- Comment #2 from Dennis Gilmore dennis@ausil.us 2011-01-06 21:44:05 EST --- 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 dennis@ausil.us - 2.7-2 - Requires php-pdo - remove -template from config file name
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=637213
Ricky Zhou rzhou@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|nobody@fedoraproject.org |rzhou@redhat.com Flag| |fedora-review+
--- Comment #3 from Ricky Zhou rzhou@redhat.com 2011-01-07 20:52:17 EST --- 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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=637213
Dennis Gilmore dennis@ausil.us changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Flag| |fedora-cvs?
--- Comment #4 from Dennis Gilmore dennis@ausil.us 2011-01-25 21:03:01 EST --- Package Name: yubikey-val Short Description: The YubiKey Validation Server Owners: ausil Branches: el6 f13 f14 InitialCC:
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=637213
--- Comment #5 from Dennis Gilmore dennis@ausil.us 2011-01-25 21:09:04 EST --- New Package SCM Request ======================= Package Name: yubikey-val Short Description: The YubiKey Validation Server Owners: ausil Branches: el6 f13 f14 InitialCC:
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=637213
--- Comment #6 from Dennis Gilmore dennis@ausil.us 2011-01-25 21:12:37 EST --- Git done (by process-git-requests).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=637213
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=637213
--- Comment #7 from Fedora Update System updates@fedoraproject.org 2011-01-25 22:49:44 EST --- 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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=637213
--- Comment #8 from Fedora Update System updates@fedoraproject.org 2011-01-25 22:50:37 EST --- 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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=637213
--- Comment #9 from Fedora Update System updates@fedoraproject.org 2011-01-25 22:51:10 EST --- 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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=637213
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #10 from Fedora Update System updates@fedoraproject.org 2011-01-26 14:03:28 EST --- 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
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=637213
--- Comment #11 from Fedora Update System updates@fedoraproject.org 2011-02-12 19:20:06 EST --- 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.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=637213
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Fixed In Version| |yubikey-val-2.7-2.el6 Resolution| |ERRATA Last Closed| |2011-02-12 19:20:12
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=637213
--- Comment #12 from Fedora Update System updates@fedoraproject.org 2011-02-13 03:49:44 EST --- 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.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=637213
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|yubikey-val-2.7-2.el6 |yubikey-val-2.7-2.fc14
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=637213
--- Comment #13 from Fedora Update System updates@fedoraproject.org 2011-02-13 03:54:17 EST --- 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.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=637213
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|yubikey-val-2.7-2.fc14 |yubikey-val-2.7-2.fc13
package-review@lists.fedoraproject.org