[Bug 1014353] Review Request: php-lightopenid - PHP OpenID library

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

 



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

Patrick Uiterwijk <puiterwijk@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |puiterwijk@xxxxxxxxxx



--- Comment #10 from Patrick Uiterwijk <puiterwijk@xxxxxxxxxx> ---
> Obviously :
> [!]: Package complies to the Packaging Guidelines
> 
> Please read: http://fedoraproject.org/wiki/Packaging:PHP
> 
> Requires:       php >= 5.1.2
> 
> => A PHP library must not have an explicit Requires on php or httpd..
> 
> From phpcompatinfo report should be
> 
> Requires:       php(language) >= 5.1.2 (but this one have no interest)
> Requires:       php-curl
> Requires:       php-pcre
> 
> %{_datadir}/lightopenid
> 
> => Non-PEAR PHP software which provides shared libraries should put its PHP
> source files for such shared libraries in a subfolder of /usr/share/php
Fixed in 0.6-2


> 
> [!]: If the source package does not include license text(s) as a separate
> file
>      from upstream, the packager SHOULD query upstream to include it.
> 
> So ? no link to upstream request or bug report ?
This is in upstream since 2013, I just requested them to make a new release:
http://code.google.com/p/lightopenid/issues/detail?id=75

> 
> [!]: Packages should try to preserve timestamps of original installed files.
> 
> So ? 
> Ok, the package have been fixed, but this is not traced here.
This is already fixed: cp -p preserved timestamps.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





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