Skip to content

Conversation

Xavier-Do
Copy link
Collaborator

@Xavier-Do Xavier-Do commented May 2, 2024

The docutils is pinned to 0.16.0 while the Jammy version should be 0.17

This is the case for a compatibility issue with sphinx-tabs==3.2.0 also official versions in ubuntu Jammy.

The odoo.requirements package contains already a docutils 0.17.0 creating some downgrade when installing requirements, this commit upgrade both dependency to a compatible version.

The proposed solution pins sphinx-tabs to a higher version, compatible with docutils 0.17. Some investigation should still be done to check why the package versions in ubuntu focal are not compatible when installed via pip.

This should avoid installing requirements in all documentation build and hopefully avoid some random errors.

This will also avoid an random error where documentation requirements are installed before odoo ones
https://runbot.odoo.com/runbot/batch/1443963/build/62071320

@robodoo
Copy link
Collaborator

robodoo commented May 2, 2024

@C3POdoo C3POdoo requested a review from a team May 2, 2024 07:43
@Xavier-Do Xavier-Do force-pushed the master-update-requirements-xdo branch from 19ab791 to 0ec81ca Compare May 2, 2024 07:45
@Xavier-Do Xavier-Do changed the title [wip] [FIX] adapt requirements to ubuntu Focal May 2, 2024
@Xavier-Do Xavier-Do changed the title [FIX] adapt requirements to ubuntu Focal [FIX] adapt requirements to ubuntu Jammy May 2, 2024
@Xavier-Do Xavier-Do force-pushed the master-update-requirements-xdo branch 3 times, most recently from 67777a3 to 6c714d1 Compare May 2, 2024 07:58
The docutils is pinned to 0.16.0 while the Jammy version should be 0.17

This is the case for a compatibility issue with sphinx-tabs==3.2.0
also official versions in ubuntu Jammy.

The odoo.requirements package contains already a docutils 0.17.0
creating some downgrade when installing requirements, this commit
upgrade both dependency to a compatible version.

This should avoid installing requirements in all documentation build
and hopefully avoid some random errors.
@Xavier-Do Xavier-Do force-pushed the master-update-requirements-xdo branch from 6c714d1 to 584be77 Compare May 2, 2024 08:07
@Xavier-Do Xavier-Do changed the base branch from master to 17.0 May 2, 2024 08:07
@d-fence
Copy link
Contributor

d-fence commented May 2, 2024

robodoo r+

@Feyensv
Copy link
Collaborator

Feyensv commented May 2, 2024

duplicate of #6205 👀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants