spec: http://terjeros.fedorapeople.org/logstalgia/logstalgia.spec srpm: http://koji.fedoraproject.org/koji/getfile?taskID=864378&name=logstalgia-0.9.1-1.fc10.src.rpm koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=864377 description: logstalgia (aka ApachePong) replays or streams a standard website access log (eg access.log) as a retro arcade game-like simulation.
Ready to review, but koji pruned the SRPM, please post to fedorapeople.
srpm: http://terjeros.fedorapeople.org/logstalgia/logstalgia-0.9.1-1.fc9.src.rpm
Thanks for the interest Jon, when I have your attention could you please have a look on # 254059 too. It has been waiting for some love in 14 months now. Thanks for the help so far.
rpmlint is clean all around. License tag should be GPLv3+. Please comment on patch status WRT upstream, and submit upstream if necessary. Would this benefit from a .desktop file and icon? You would either have it run against the example. You could have it run against /var/log/httpd/access_log, but that'd need root and require httpd. Something to consider. Otherwise, full review is good, no BuildRequires missing. And I'll have a peek at 254059. If you wanted to reciprocate, 452107 should be pretty straightforward.
- Upstream patch refs - Fix license spec: http://terjeros.fedorapeople.org/logstalgia/logstalgia.spec srpm: http://koji.fedoraproject.org/koji/getfile?taskID=887396&name=logstalgia-0.9.1-2.fc11.src.rpm koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=887395
Koji build pruned, repost?
srpm: http://terjeros.fedorapeople.org/logstalgia/logstalgia-0.9.1-2.fc9.src.rpm
Ok, all clear. APPROVED.
Thanks again! New Package CVS Request ======================= Package Name: logstalgia Short Description: Web server access log visualizer Owners: terjeros Branches: F-9 InitialCC:
cvs done
Imported and built.
Package Change Request ====================== Package Name: logstalgia New Branches: el6 epel7 Owners: cicku terjeros
Git done (by process-git-requests).