[Bug 421241] Review Request: php-ZendFramework - Leading open-source PHP framework

bugzilla at redhat.com bugzilla at redhat.com
Tue Jun 3 20:15:04 UTC 2008


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: php-ZendFramework - Leading open-source PHP framework


https://bugzilla.redhat.com/show_bug.cgi?id=421241





------- Additional Comments From akahl at iconmobile.com  2008-06-03 16:15 EST -------
Hi Jess,

this is my status:

Current packages
  (base)
  demos
  tests
  Cache-Backend-Memcached
  Cache-Backend-Sqlite
  Db-Adapter-Mysqli
  Db-Adapter-Db2
  Db-Adapter-Firebird
  Db-Adapter-Oracle
  Feed
  Gdata (why doesn't this exist as a service component?)
  Pdf
  Search-Lucene
  Services

There are some issues here:
Cache-Backend-Sqlite requires php-sqlite but this is not available in Fedora's
PHP build, it was disabled Mon Nov 8 2004 by Joe Orton, most probably in favor
of pdo-sqlite. Can the back end handle pdo-sqlite instead of sqlite? If not we
probably have to exclude it.

Db-Adapter-Db2, Db-Adapter-Firebird and Db-Adapter-Oracle require extensions
that must be compiled in while having the respective database software
installed, in case of DB2 and Oracle this is impossible for Fedora since they
are proprietary software, I don't know about InterBase/Firebird however it's
also missing. We can of course offer Zend's adapters but the user is required to
compile PHP and enable the affected dependencies himself.

Feed has been separated because it requires mbstring; same goes for Pdf because 
of GD, Search-Lucene because of bitset; Locale-Math has _not_ been separated as
planned since Date uses bcmath as well and cannot be separated as a major
dependency for other base components.

About mime_magic: I've checked our PHP build, the extension was disabled Wed Mar
21 2007, indeed in favor of Fileinfo. I'd really appreciate Fileinfo support
from your side! A wrapper functions sounds completely reasonable.

The total list of extensions required for the base framework package as of now:
bcmath, ctype, curl, dom, hash, iconv, json, pcre, posix, reflection, session,
simplexml, spl, zlib.
Only two of them, bcmath and dom, require additional extensions to be pulled
in which is pretty acceptable.

About the getTmpDir() issue: Please keep in mind you're targeting a multi-user
platform, setting a fixed value in a configuration file can still create
clashes, especially if no cleanup is performed afterwards. Given both choices
only, I'd rather go with the environment variable; if you want a much more     
                                                                               
                                                  
flexible solution involving the configuration file, I suggest append a hash,
timestamp or the uid to the base path given. Actually this is what I do in my
tests :)
Alternatively, you could go with world writable modes for all files and
directories created during the tests.


@Gianluca
Thanks in advance for volunteering!

-- 
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, or are watching someone who is.




More information about the Fedora-package-review mailing list