Bug#785281: [pkg-php-pear] Bug#785281: Bug#785281: Split phpseclib?
2015-05-16 22:10 GMT+02:00 David Prévot taf...@debian.org: Hi Mathieu, Hi David, On Sat, May 16, 2015 at 09:22:48PM +0200, Mathieu Parent wrote: 2015-05-16 20:58 GMT+02:00 David Prévot taf...@debian.org: 3./ The php-phpseclib should provide all the libraries it ships. This would be the proper fix for #785281. I prefer the fake provide solution, maybe provides: php-math-biginteger (= 1.0.2+phpseclib) ? Sure, done. I added the other provides (php-seclib, as well as the 19 php-seclib-* packages from the phpseclib channel you pointed earlier) versionless, please shout (or directly fix in the VCS) if you prefer versioned Provides (not that I mind about seeing #761219 fixed first, but if versions are not needed, no need to add them). Upload in progress, you should be able to test it from incoming in a few minutes (I successfully installed php-horde-mapi with php-seclib, but didn’t test if it actually works). Seems to work (but I don't have access to a MAPI server for complete test). I've asked for removal of php-math-biginteger. Reards -- Mathieu -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Bug#785281: [pkg-php-pear] Bug#785281: Bug#785281: Split phpseclib?
Hi Mathieu, On Sat, May 16, 2015 at 09:22:48PM +0200, Mathieu Parent wrote: 2015-05-16 20:58 GMT+02:00 David Prévot taf...@debian.org: 3./ The php-phpseclib should provide all the libraries it ships. This would be the proper fix for #785281. I prefer the fake provide solution, maybe provides: php-math-biginteger (= 1.0.2+phpseclib) ? Sure, done. I added the other provides (php-seclib, as well as the 19 php-seclib-* packages from the phpseclib channel you pointed earlier) versionless, please shout (or directly fix in the VCS) if you prefer versioned Provides (not that I mind about seeing #761219 fixed first, but if versions are not needed, no need to add them). Upload in progress, you should be able to test it from incoming in a few minutes (I successfully installed php-horde-mapi with php-seclib, but didn’t test if it actually works). Regards David signature.asc Description: Digital signature
Bug#785281: [pkg-php-pear] Bug#785281: Bug#785281: Split phpseclib?
2015-05-16 20:58 GMT+02:00 David Prévot taf...@debian.org:
Hi Mathieu,
Hi David,
3./ The php-phpseclib should provide all the libraries it ships.
This would be the proper fix for #785281.
Digging a bit, this would not be the proper fix for #785281.
Here is an extract from the diff:
* Here's an example of how to use this library:
* code
* ?php
- *include('Math/BigInteger.php');
- *
- *$a = new Math_BigInteger(2);
- *$b = new Math_BigInteger(3);
+ *$a = new \phpseclib\Math\BigInteger(2);
+ *$b = new \phpseclib\Math\BigInteger(3);
[...]
+namespace phpseclib\Math;
[...]
Those are two different classes, with different namespaces.
I can’t see such difference between the currently packaged phpseclib and
php-math-biginteger. The namespaces change is supposed to happen in the
next 1. or 2. branch of phpseclib, but the latest upstream version as
available in Debian, is not yet inside this new namespace.
OK.
Ha, that reminds me of the “good old times” and
https://github.com/phpseclib/phpseclib/issues/125…
I see.
It seems They should probably be installed at a different path:
/usr/share/php/Math/BigInteger.php
/usr/share/php/phpseclib/Math/BigInteger.php
Not yet, sorry.
Indeed.
I prefer that the files are moved. But this may be a painfull transition.
One we’ll have to take care anyway, but I don’t intend to start it before
upstream publish a stable release with the new namespase (not even an
alpha has been published yet).
I'd welcome if the phpseclib package Provides php-phpseclib-file-asn1
Sure, I’ll had the others too, but the previous question still stands:
should I
add another fake “provides: php-math-biginteger (= 1.0.2-3)” for example
(i.e., a bit higher than the current php-math-biginteger real package),
or [would] you […] patch php-horde-mapi to drop the versionned dependency
for php-math-biginteger.
?
I prefer the fake provide solution, maybe provides:
php-math-biginteger (= 1.0.2+phpseclib) ?
In parallel, I will remove php-math-integer source package from Debian.
Regards
--
Mathieu
--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
