Bug#1007236: php-symfony-polyfill-mbstring shouldn't provide php-mbstring

[Guilhem Moulin]

Package: php-symfony-polyfill-mbstring
Version: 1.25.0-1
Severity: normal

Dear Maintainer,

$ apt show php-symfony-polyfill-mbstring
Package: php-symfony-polyfill-mbstring
Version: 1.25.0-1
[…]
Provides: php-mbstring
[…]
Description: Symfony polyfill for the Mbstring extension
This component provides a partial, native PHP implementation for the
Mbstring extension.
[…]

Given it's not a drop in replacement for php-mbstring I suppose it
shouldn't provide php-mbstring no?

That Provides: causes roundcube 1.6~beta+dfsg-1 to FTBFS on the buildds [0].
roundcube has ‘Build-Depends: php-mbstring’, in my local build environment
that pulls php8.1-mbstring and the build succeeds, but on the buildds for
some reason the dependency resolves to ‘php-symfony-polyfill-mbstring’ and
the build fails with

PHP Fatal error: Uncaught Error: Call to undefined function mb_internal_encoding() in /<<PKGBUILDDIR>>/program/lib/Roundcube/bootstrap.php:86
Stack trace:
#0 /<<PKGBUILDDIR>>/program/include/iniset.php(78): require_once()
#1 /<<PKGBUILDDIR>>/program/include/clisetup.php(25): require_once('...')
#2 /<<PKGBUILDDIR>>/bin/updatecss.sh(22): require_once('...')
#3 {main}
thrown in /<<PKGBUILDDIR>>/program/lib/Roundcube/bootstrap.php on line 86

I suppose removing the Provides: would fix the FTBFS but I'm open to other
suggestions to solve the FTBFS :-) (I note that php-symfony-polyfill-
{ctype,iconv,uuid} have matching Provides: but haven't checked whether they
are affected by similar issues.)

Cheers
--
Guilhem.

[0] https://buildd.debian.org/status/fetch.php?pkg=roundcube&arch=all&ver=1.6%7Ebeta%2Bdfsg-1&stamp=1647250601&raw=1

[Guilhem Moulin]

On Mon, 14 Mar 2022 at 12:05:20 +0100, David Prévot wrote:
>> That Provides: causes roundcube 1.6~beta+dfsg-1 to FTBFS on the buildds [0].
>

> I had the same problem with two other packages (in experimental only). I’m
> pretty busy currently, but feel free to push a fix via a team upload or an
> NMU.

OK, thanks taffit for the quick feedback!

--
Guilhem.

[David Prévot]

Hi Guilhem,

Le 14/03/2022 à 11:52, Guilhem Moulin a écrit :
> Package: php-symfony-polyfill-mbstring
> Version: 1.25.0-1

> Given it's not a drop in replacement for php-mbstring I suppose it
> shouldn't provide php-mbstring no?

Agreed.

> That Provides: causes roundcube 1.6~beta+dfsg-1 to FTBFS on the buildds [0].

I had the same problem with two other packages (in experimental only).
I’m pretty busy currently, but feel free to push a fix via a team upload
or an NMU.

Regards

David