Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: wordpress - database driven blogging software https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=193478 jwilson@xxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|bugzilla-sink@xxxxxxxxxxxxx |jwilson@xxxxxxxxxx OtherBugsDependingO|163776 |163778 nThis| | ------- Additional Comments From jwilson@xxxxxxxxxx 2006-06-19 15:50 EST ------- I'll go ahead and take on this review, since I'm familiar with the software. My findings thus far: 1) There's a new version (2.0.3) out now (obsoletes the patch), need to update 2) Web apps should be installed in /usr/share/<name>, not /var/www, per fedora extras guidelines 3) rpmlint complains about a bunch of empty files, they should be removed 4) config files should be broken out into /etc/ somewhere 5) %doc lines aren't quite correct, you should be removing them from the installed packages and then sucking them in w/the %doc line, referring to them by their relative path within the source 6) the Source0: url doesn't appear to actually work, upstream only post a 'latest.tar.gz' (rather annoying of them...) I'm rather partial to proposing fixes for issues I come up with in review in the form of an updated spec file, so you can see exactly what I'm suggesting: http://wilsonet.com/packages/wordpress/ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug, or are watching the QA contact. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review