[pkg-php-pear] Bug#827483: [pkg-horde] Bug#827483: php-horde-mapi: fix autopkgtest errors
Mathieu Parent
math.parent at gmail.com
Sun Jun 19 19:30:48 UTC 2016
)Control: clone -1 -2
Control: reassign -1 php-horde-mapi 1.0.8-2
Control: tag -1 + patch
Control: tag -2 + patch wontfix
2016-06-19 9:46 GMT+02:00 Nish Aravamudan <nish.aravamudan at canonical.com>:
> On 18.06.2016 [16:50:23 -0400], David Prévot wrote:
[...]
>> I disagree, and stand to what I’ve written in the last changelog entry:
>>
>> Actually fixing the constructors requires to also fix all their calls,
>> both internally and externally. This backward-incompatible change has
>> been achieved in version 2 of phpseclib, packaged in Debian as
>> php-phpseclib to ensure co-installability. (Closes: #819420)
Okay. I agree with you David.
> Right, my original patch in this e-mail was just to quieten the
> deprecated call from this testcase, as it's not really a failure (any
> output on stderr is treated as a failure).
>
>> From http://phpseclib.sourceforge.net/:
>>
>> The 2.0 branch has pretty much the exact same API as the 1.0 branch,
>> save for that it is namespaced, uses PHP5-style constructors (thereby
>> avoiding E_DEPRECATED errors) and requires the use of an autoloader.
[...]
> Agreed, and perhaps something like the attached (which passes
> autopkgtests for me) can be applied to Debian's package (and we can sync
> in Ubuntu) and massaged to be sent upstream? I apologize if this is way
> off-base, I'm not all a PHP developer :)
Thanks Nish. You patch looks good.
Some notes:
- In d/control, use php-phpseclib (>= 2~) as < and > operators are
obsolete (see man:dpkg(1))
- "use phpseclib\Math;" is probably not needed as you use fully
qualified names after (see
http://php.net/manual/en/language.namespaces.importing.php)
- Can you send this upstream (using github.com/horde/horde or bugs.horde.org)?
I'll apply your patch once I have some time to test.
Regards
--
Mathieu Parent
More information about the pkg-php-pear
mailing list