[Bug 201502] Review Request: php-pear-PhpDocumentor

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

Summary: Review Request: php-pear-PhpDocumentor
Alias: phpDocumentor

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





------- Additional Comments From cweyl@xxxxxxxxxxxxxxx  2006-08-24 02:09 EST -------
The release tag is off by a little bit, the %{?dist} tag should always be at
the end of the tag unless it's a sub-release bump.

    0.2%{?dist}.%{rcrel} -> 0.2.%{rcrel}%{?dist}

According to Packaging/PHP, the package should also "Requires: php".

The latest version is 1.3.0 GA, but RC6 is being packaged.  To be fair, GA was
released the day after I assigned the review to myself, but... :)  Please
update the package to 1.3.0.

For the rpmlint warnings, the second is spurious and can be ignored.  The
first, however, could be fixed, but it appears part of the pear install
process is to check the md5sums of each file.  If the line ending can be fixed
w/o too much pain and mucking around with the pear-based install process, I'd
like to see it addressed.

You have php.ini both listed as source1, and created in %prep.  One or the
other works nicely, but both is redundant.  (Especially given nothing is ever
done with %{SOURCE0}...)

Also, while using php.ini to override the default memory and execution limits
works during the rpm build, the install fails on %post with the defaults on
the system:

 [root@zeus mock]# rpm -ivh php-pear-PhpDocumentor-1.3.0-0.2.fc5.RC6.noarch.rpm
 Preparing...                ########################################### [100%]
   1:php-pear-PhpDocumentor ########################################### [100%]
   PHP Fatal error:  Allowed memory size of 8388608 bytes exhausted (tried to
   allocate 229 bytes) in /usr/share/pear/PEAR/Installer.php on line 283
   Allowed memory size of 8388608 bytes exhausted (tried to allocate 72 bytes)

Installing the created php.ini as an additional .ini file in /etc/php.d
enables the install to suceed, e.g. in %install:

 mkdir -p $RPM_BUILD_ROOT%{_sysconfdir}/php.d
 cp -p php.ini $RPM_BUILD_ROOT%{_sysconfdir}/php.d/phpDocumentor.ini

I don't get a good feeling from overriding core php.ini values in a package's
own modular ini file, but there doesn't seem to be another clean way to do
this.  (ideas/comments from those reading are solicited :) )  Perhaps it would
be a good idea to open a bug asking for the core memory limits be updated?
The response at least from the php maintainer would be valuable.

There's nearly a meg of tutorials and examples, which is great, but that's a
bit large to not be put in a -docs subpackage.

SO, to recap:
 * release tag
 * check on fixing rpmlint warning
 * update version
 * decide how php.ini is going to be included in srpm
 * deal with memory limits (a la a /etc/php.d/foo.ini file), open bug
 * split %_docdir/Documentation and %_docdir/tutorials off into a -docs
   package

X package meets naming and packaging guidelines. (release tag)
+ specfile is properly named, is cleanly written and uses macros consistently.
X dist tag is present (but in the wrong place)
+ build root is correct.
+ license field matches the actual license.
+ license is open source-compatible (LGPL). License text included in package.
+ source files match upstream:
 e6f31c313b0b06c09acaf7047e6a5b23  PhpDocumentor-1.3.0RC6.tgz
 e6f31c313b0b06c09acaf7047e6a5b23  PhpDocumentor-1.3.0RC6.tgz.srpm
X latest version is being packaged.
+ BuildRequires are proper.
X package has required requires/provides statements.
+ package builds in mock (devel/x86_64).
O rpmlint is silent. (see below)
O final provides and requires are sane (except as noted above):
 ** phpdoc-1.3.0-0.2.fc5.RC6.noarch.rpm
 == rpmlint
 == provides
 phpdoc = 1.3.0-0.2.fc5.RC6
 == requires
 /usr/bin/php
 php-cli >= 5
 php-pear(PhpDocumentor) = 1.3.0
 ** php-pear-PhpDocumentor-1.3.0-0.2.fc5.RC6.noarch.rpm
 == rpmlint
 W: php-pear-PhpDocumentor wrong-file-end-of-line-encoding
 /usr/share/doc/php-pear-PhpDocumentor-1.3.0/Documentation/Release-old/Release-0.3.0
 W: php-pear-PhpDocumentor dangerous-command-in-%post install
 == provides
 php-pear(PhpDocumentor) = 1.3.0
 php-pear-PhpDocumentor = 1.3.0-0.2.fc5.RC6
 == requires
 /bin/sh
 /usr/bin/pear
 php-pear(PEAR) >= 1.4.9
+ no shared libraries are present.
+ package is not relocatable.
+ owns the directories it creates.
+ doesn't own any directories it shouldn't.
+ no duplicates in %files.
+ file permissions are appropriate.
+ %clean is present.
O %check is not present; no automated test suite
+ sane scriptlets present.
+ code, not content.
X documentation is large (~876k), so -docs subpackage is necessary.
+ %docs are not necessary for the proper functioning of the package.
+ no headers.
+ no pkgconfig files.
+ no libtool .la droppings.
+ not a GUI app.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
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]