Tips (for reviewers) php-xml doesn't provides xml extension
by Remi Collet
During a recent package review a notice this (very old) singularity
pci (from php-pear-PHP-CompatInfo) could reports the need for the xml
extension (often because of utf8_encode / utf8_decode functions)
XML is a buildtin extension, so provided by php-common
$ php -n -m | grep -i xml
libxml
SimpleXML
xml
The php-xml sub-package provides some additional XML related extensions
dom
wddx
xmlreader
xmlwriter
xsl
So, a lot of package (web apps) are probably requiring php-xml when it
is not really needed.
Just a notice,
I don't think we have to fix this.
Remi.
13 years, 1 month
- 1
- 0
PHP and Arch Specific Requires
by Remi Collet
Hi,
I plan to work on PHP spec file for Arch Specific Requires
(the %{?_isa} macro use)
Some changes are quite trivial
Requires: php-common%{?_isa} = %{version}-%{release}
But we have a lot of virtual provides which need to be fixed
Provides: php-bz2%{?_isa}, php-calendar%{?_isa}, php-ctype%{?_isa}...
(keeping the non-arch specific ones for noarch package)
Shouldn't we also use version in this ?
Provides: php-bz2%{?_isa} = %{version}-%{release}
If we also change
Provides: php(api)%{?_isa} = %{apiver}
Provides: php(zend-abi)%{?_isa} = %{zendver}
This will requires a Guidelines change for pecl package to use
Requires: php(zend-abi)%{?_isa} = %{php_zend_api}
Requires: php(api)%{?_isa} = %{php_core_api}
@Joe, do you plan to change httpd ? to be able to
Requires: httpd-mmn%{?_isa} = %{httpd_mmn}
In which branch should we do this change ?
Thanks for your feedbacks
Remi.
13 years, 2 months
- 2
- 5
- ← Newer
- 1
- Older →