Spec URL: http://rpms.damian.net/SPECS/php-phpmd-PHP-PMD.spec SRPM URL: http://rpms.damian.net/SRPMS/php-phpmd-PHP-PMD-0.2.0-1.fc12.src.rpm Description: <description here>
forgot description: Spec URL: http://rpms.damian.net/SPECS/php-phpmd-PHP-PMD.spec SRPM URL: http://rpms.damian.net/SRPMS/php-phpmd-PHP-PMD-0.2.0-1.fc12.src.rpm Description: PHPMD - PHP Mess Detector
new upstream release: Spec URL: http://rpms.damian.net/SPECS/php-phpmd-PHP-PMD.spec SRPM URL: http://rpms.damian.net/SRPMS/php-phpmd-PHP-PMD-0.2.1-1.fc12.src.rpm Description: PHPMD - PHP Mess Detector
First/quick notes - Replace %define by %global (I have miss this on previous reviews...) - use %{channel} in %postun - use %{pear_datadir} instead of %{pear_phpdir}/data All docs should be in /usr/share/doc/php-phpmd-PHP-PMD-0.2.1 Most of pear package use (what is generated by pear make-rpm-spec): after install: mv $RPM_BUILD_ROOT%{pear_docdir}/* docdir and in %file %doc %{pear_name}-%{version}/docdir/%{pear_name}/* In fact, Guidelines need to be clearer about this, so Waiting for a reply to http://lists.fedoraproject.org/pipermail/packaging/2010-January/006845.html
According to package.xml this requires dom extension According to pci this requires libxml and SimpleXML So, replace Requires: php-common >= 5.2.0 With Requires: php-xml >= 5.2.0
(In reply to comment #3) > First/quick notes > > - Replace %define by %global (I have miss this on previous reviews...) > - use %{channel} in %postun > - use %{pear_datadir} instead of %{pear_phpdir}/data all done. I will replace the define while doing other updates on the other packages. > > All docs should be in /usr/share/doc/php-phpmd-PHP-PMD-0.2.1 > > Most of pear package use (what is generated by pear make-rpm-spec): > > after install: > mv $RPM_BUILD_ROOT%{pear_docdir}/* docdir > > and in %file > %doc %{pear_name}-%{version}/docdir/%{pear_name}/* That makes sense. (In reply to comment #4) > According to package.xml this requires dom extension > According to pci this requires libxml and SimpleXML > > So, replace > Requires: php-common >= 5.2.0 > With > Requires: php-xml >= 5.2.0 I added the php-xml. I prefer to have the php-common requirement with the php version number and the separate requirement for php-xml, because it makes the spec file more readable. Is that ok too? Or is there a guideline to reduce the requires to a minimum? I also upgraded to the new upstream bugfix release: Spec URL: http://rpms.damian.net/SPECS/php-phpmd-PHP-PMD.spec SRPM URL: http://rpms.damian.net/SRPMS/php-phpmd-PHP-PMD-0.2.2-1.fc12.src.rpm
> Or is there a guideline to reduce the requires to a minimum? Well... I don't find anything. I think the use of minimal Requires is a good practice (which also reduce the size of the repo metadata) > because it makes the spec file more readable. Ok. Seems you have forget to upload the new files...
(In reply to comment #6) > Seems you have forget to upload the new files... Oops, wrong server. I just moved my hosting.
REVIEW: + rpmlint is ok php-phpmd-PHP-PMD.noarch: I: checking php-phpmd-PHP-PMD.src: I: checking 2 packages and 1 specfiles checked; 0 errors, 0 warnings. + package name ok + spec file name ok + package meet the PHP Guidelines + License ok : BSD + License is upstream + spec in english and legible + provided license file + sources match the upstream sources 9c76d49df7137b52332267b76bb11f64 PHP_PMD-0.2.2.tgz + Source URL ok + build on F12.x86_64 + BuildRequires (php-pear >= 1:1.6.0, php-channel(pear.phpmd.org)) ok + no locale + no .so + own all directories that it creates + no duplicate file + %defattr ok + %clean section + use macros consistently + contain code + small documentation not required to run + no devel + no pkgconfig + no sub-package + no GUI + don't own files or directories already owned by other packages + %install start with rm -rf + valid UTF-8 + test suite ok (can't be run during build) + scriptlets ok + Final Requires ok /usr/bin/pear /usr/bin/php php-channel(pear.phpmd.org) php-common >= 5.2.0 php-pear(pear.pdepend.org/PHP_Depend) >= 0.9.7 php-xml + Final Provides ok php-pear(pear.phpmd.org/PHP_PMD) = 0.2.2 php-phpmd-PHP-PMD = 0.2.2-1.fc8 + Build in koji http://koji.fedoraproject.org/koji/taskinfo?taskID=1954558 + run ok in %file, should use %{pear_datadir}/PHP_PMD instead of %{pear_phpdir}/data/PHP_PMD ===== APPROVED =====
(In reply to comment #8) > > in %file, should use %{pear_datadir}/PHP_PMD instead of > %{pear_phpdir}/data/PHP_PMD I will change that before the import > ===== APPROVED ===== Thank you for the review. New Package CVS Request ======================= Package Name: php-phpmd-PHP-PMD Short Description: PHPMD - PHP Mess Detector Owners: cdamian Branches: F-11 F-12 InitialCC:
CVS done (by process-cvs-requests.py).
php-phpmd-PHP-PMD-0.2.2-2.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/php-phpmd-PHP-PMD-0.2.2-2.fc11
php-phpmd-PHP-PMD-0.2.2-2.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/php-phpmd-PHP-PMD-0.2.2-2.fc12
php-phpmd-PHP-PMD-0.2.2-2.fc11 has been pushed to the Fedora 11 stable repository. If problems still persist, please make note of it in this bug report.
php-phpmd-PHP-PMD-0.2.2-2.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report.
Package Change Request ====================== Package Name: php-phpmd-PHP-PMD New Branches: EL-6 Owners: cdamian
Package Change Request ====================== Package Name: php-phpmd-PHP-PMD New Branches: EL-7 Owners: llaumgui
Git done (by process-git-requests).
Can you please check, it seems epel7 branch doesn't exist.
Package Change Request ====================== Package Name: php-phpmd-PHP-PMD New Branches: epel7 Owners: llaumgui