php-twig 3.x in circular build dependency situation with symfony 5

Bug #1929880 reported by Bryce Harrington
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
php-doctrine-bundle (Ubuntu)
Fix Released
Undecided
Unassigned
php-twig (Ubuntu)
Fix Released
Undecided
Unassigned
symfony (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

php-symfony-string is a new package introduced in symfony 5, but not in symfony 4. This package is required by php-twig 3.x for running its testsuite during the build process. Unfortunately, symfony 5 itself depends on three new packages introduced in php-twig 3. Thus, there is a circular dependency between these packages during build.

Since php-twig only requires php-symfony-string for its testsuite, a possible way to sidestep the problem is to temporarily skip running php-twig's testsuite, allowing the php-symfony-string dependency to be temporarily dropped.

Then, once this bootstrapped php-twig is in -proposed, symfony should hopefully complete its build. At that point we should be able to revert the php-twig testsuite disablement and re-upload it to build with the testsuite, and carry on from there.

Revision history for this message
Utkarsh Gupta (utkarsh) wrote :

If only it were simple to build symfony once we landed php-twig, which I did, some moments ago. :)

Anyhow, symfony has a little bit of mess because of .so involved, which can be cleaned up easily but some tests behave oddly:
```
Error in bootstrap script: Error:
Failed opening required 'Cache/TagInterop/autoload.php' (include_path='.:/usr/share/php')
KO src/Symfony/Bundle/DebugBundle
PHP Warning: require_once(Cache/TagInterop/autoload.php): Failed to open stream: No such file or directory in /usr/share/php/Cache/IntegrationTests/autoload.php on line 4
```

This probably happens in Debian as well; me and William (~williamdes)are investigating that on Debian and will try to have that fixed. Anyhow, disabling tests for symfony as well for now. I've got closer to building this. Mostly everything can be reverted once we have settled down the cicular-dependency hell hole. :D

Thus, adding symfony to this bug as well. It further needs dependency bumps which I'll take care of on top of this.

Revision history for this message
Utkarsh Gupta (utkarsh) wrote :

Okay, we've cleared the way through so far, yay! \o/

Now, the interesting bit is that there are circular dependencies for testing as well. php-doctrine-bundle's tests rely on the working symfony & symfony's tests rely on working php-doctrine-bundle :)

But at the moment, php-doctrine-bundle needs a rebuild. Because otherwise, it tries to pull in parts of symfony/cache v4.

So I'd like to disable the tests & do a rebuild & upload. This will help enable symfony's tests and later on we work on enabling tests for php-doctrine-bundle.

Utkarsh Gupta (utkarsh)
Changed in php-twig (Ubuntu):
status: New → Fix Released
Changed in symfony (Ubuntu):
status: New → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package php-doctrine-bundle - 2.2.3-1ubuntu1

---------------
php-doctrine-bundle (2.2.3-1ubuntu1) impish; urgency=medium

  * Rebuild w/ PHP 8.0.
  * d/rules: Temporarily disable testsuite to allow sidestepping
    circular build dependency on php-twig (and its components).
    This change can be reverted once both php-twig and symfony
    have migrated. (LP: #1929880)

 -- Utkarsh Gupta <email address hidden> Fri, 04 Jun 2021 13:40:31 +0530

Changed in php-doctrine-bundle (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.