[Bug 454395] Review Request: php-xmpphp - PHP XMPP Library

[bugzilla at redhat.com]

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=454395





--- Comment #5 from Peter Lemenkov <lemenkov at gmail.com>  2008-08-23 07:44:06 EDT ---
Quite easy to review and to package item. I agree with Jason - you'd better to
update package to r50, which is more interesting to deal with. 

I've got only one personal (cosmetic) wish - you shouldn't use macros in
URL-field, because it's more easy not to substitute them manually just for
visiting project's webpage :). E.g. instead of 

URL:            http://code.google.com/p/%{realname}/

I prefer to see something like

URL:            http://code.google.com/p/xmpphp/

Just my opinion, anyway.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.