Spec URL: http://tkmame.retrogames.com/fedora-extras/php-pear-Image-Canvas.spec SRPM URL: http://tkmame.retrogames.com/fedora-extras/php-pear-Image-Canvas-0.3.0-1.src.rpm Description: A package providing a common interface to image drawing, making image source code independent on the library used.
Image/Canvas/PDF.php requires php-pecl-pdflib. I provide php-pecl-pdflib and pdflib-lite on my repo (for a long time) but i dont have submit them to review (yet) because i'm not certain that License is OK. With pdflib present, test doesn't work because of Font used ("Courier New" defined as default one in Image/Canvas.php not available). A temporary solution could be to disable Image/PDF.php I will investigate on this...
I have updated the spec to remove this file, however I think it would be better to add a php-pecl-pdflib package to Extras. I think the license should be okay. If you want to put it up for review I can review it for you. Once it's in I can also patch this package to not use Courier New font in the tests. Spec URL: http://tkmame.retrogames.com/fedora-extras/php-pear-Image-Canvas.spec SRPM URL: http://tkmame.retrogames.com/fedora-extras/php-pear-Image-Canvas-0.3.0-2.src.rpm %changelog * Thu Dec 07 2006 Christopher Stone <chris.stone> 0.3.0-2 - Remove PDF.php until php-pecl-pdflib is added to Extras
Tests works. Example doesn't because it's use MS Windows Fonts. REVIEW * source files match upstream: 41dd36fb05436159fb6fccca02cb7aaa /tmp/Image_Canvas-0.3.0.tgz 41dd36fb05436159fb6fccca02cb7aaa ../SOURCES/Image_Canvas-0.3.0.tgz * package meets naming and packaging guidelines. * specfile is properly named, is cleanly written and uses macros consistently. * dist tag is present. * build root is correct. * license field matches the actual license. * license is open source-compatible (LGPL). * latest version is being packaged (0.3.0) * BuildRequires are proper. * %clean is present. * package builds in mock (development). * package installs properly * rpmlint is silent. * final provides are sane: php-pear(Image_Canvas) = 0.3.0 php-pear-Image-Canvas = 0.3.0-2.fc7 * final Requires are sane: php-pear(Image_Color) -> php-gd * %check is not present; * owns the directories it creates => own directories it shouldn't. * no duplicates in %files. * file permissions are appropriate. * scriptlets are OK (pear install) * code, not content. * documentation is small, so no -docs subpackage is necessary. * %docs are not necessary for the proper functioning of the package. MUST : - not own /usr/share/pear/Image which is owned by php-pear-Image-Color (required , remplace with %file /usr/share/pear/Image/Canvas* SHOULD : - also remove /usr/share/pear/test/Image_Canvas/tests/pdf.php - add a information (in description or a README.fedora) about "PDF not available"
> I think it would be better to add a php-pecl-pdflib package to Extras. > I think the license should be okay. php-pecl-pdflib is PHP License It's the "PDFlib Lite License" which is a special License
Ah okay, I didn't realize pdflib-lite was needed for pecl-pdflib. I've discussed the license for pdflib-lite briefly on IRC and initial discussion seemed to indicate it was okay. I'm going to wait a few days to see if there are any objections, and if we can get it in Extras then I'll modify this package to use php-pecl-pdflib and readd the PDF files.
Okay, looks like the pdflib stuff can't go into Extras, I'll go ahead and start working on the items you mentioned in comment #3.
I think perhaps it would be better to leave the PDF files in the RPM. Even though they require a package which cannot be included in Extras, I think we should keep the files there anyway incase someone wants to install the packages from another repository such as Remi's repo. However I agree that adding a README.Fedora file explaining this should be required. Please let me know if you think this is not a good idea. Thanks.
> I think we should keep the files there anyway Ok for me > However I agree that adding a README.Fedora file explaining this should be required Of course, i also think it is a MUST.
Spec URL: http://tkmame.retrogames.com/fedora-extras/php-pear-Image-Canvas.spec SRPM URL: http://tkmame.retrogames.com/fedora-extras/php-pear-Image-Canvas-0.3.0-3.src.rpm %changelog * Wed Jan 03 2007 Christopher Stone <chris.stone> 0.3.0-3 - No longer remove PDF.php file - Add README.Fedora file exlaining requirements for PDF.php
oops this bit got cut off from changelog paste above: - No longer own Image directory
* owns the directories it creates * don't own directories it shouldn't. APPROVED
- Imported into CVS - Added entry to owners.list - Build succeeded for devel - Requested FC-5/6 CVS sync Thanks for the review!
Package Change Request ====================== Package Name: php-pear-Image-Canvas New Branches: EL-5