[Bug 510979] Review Request: phpSmug - PHP wrapper for the SmugMug API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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





--- Comment #6 from David Nalley <david@xxxxxxx>  2009-07-17 16:42:43 EDT ---
My reasoning for approving the name is this: 

http://fedoraproject.org/wiki/Packaging/NamingGuidelines#General_Naming
Says that: 
When naming a package, the name should match the upstream tarball or project
name from which this software came.

The php guidelines not withstanding, php-phpSmug struck me as unnecessary
duplication and the same with php-phpFlickr. I suppose one could have chosen
php-Smug, however that seems to run afoul of the same guideline, where it is
supposed to be php-$packagename. In this case packagename is phpSmug. There is
precedence for this other places. phplogcon, phpldapadmin (albeit they are
standalone applications). Even other languages have similar precedence. PyXML
and pywbem being the ones that immediately jumps to mind (though in their
defense, Python libraries have no such naming guidance). That said there is
also precedent in the other direction - such as: php-pear-PHPUnit and
php-xmpphp 

Smarty was originally a php.net project, and doesn't contain php in the name,
so php-Smarty makes sense. 

My decision could be wrong, so feel free to say so. 
On the other hand if this makes sense, perhaps I need to send a change draft to
FPC for clarification.

-- 
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.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]