[all] Removing Babel from requirements.txt

Stephen Finucane stephenfin at redhat.com
Fri Apr 17 15:14:33 UTC 2020


On Fri, 2020-04-17 at 15:35 +0200, Dmitry Tantsur wrote:
> Hi folks,
> 
> Many of the projects have [1] Babel in requirements.txt. I'm not sure
> why it was needed initially, but now Babel is only used in the
> translation infrastructure to manage the catalogs. Particularly, it's
> not imported in runtime by anything (oslo.i18n has recently removed
> [2] its usage). The problem is, Babel weighs nearly 30MiB (estimated
> on a venv), making it probably the heaviest dependency we have.
> 
> According to amotoki, we don't need Babel anywhere in the
> requirements, since the translation infrastructure installs it
> explicitly [3][4]. Sphinx has a requirement on Babel, so doc and
> release note builds will be fine as well. I suggest that all projects
> remove an explicit requirement on Babel completely.

Looking at [1] it seems we're not using the setuptools entry point
either. Does that mean we can drop the '[extract_messages]',
'[compile_catalog]' and '[update_catalog]' sections from 'setup.cfg'
also?

Cheers,
Stephen

[1] https://opendev.org/openstack/openstack-zuul-jobs/src/commit/8ab3345e4dde2f5cee511f68b69eb12c3c2ad15e/roles/prepare-zanata-client/files/common_translation_update.sh#L371-L379

> I've started proposing changes [5] to implement it for repositories
> that are used by ironic projects (since it's my immediate area of
> interest). I'd appreciate help with the others.
> 
> Dmitry
> 
> [1] http://codesearch.openstack.org/?q=Babel&i=nope&files=requirements.txt&repos=
> [2] https://review.opendev.org/720550
> [3] https://opendev.org/openstack/project-config/src/branch/master/playbooks/translation/pre.yaml
> [4] https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/ensure-babel/tasks/main.yaml#L35
> [5] https://review.opendev.org/#/q/topic:no-babel




More information about the openstack-discuss mailing list