Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

plone.app.upgrade must not be a hard dependency. #142

Open
jensens opened this issue Apr 2, 2024 · 3 comments
Open

plone.app.upgrade must not be a hard dependency. #142

jensens opened this issue Apr 2, 2024 · 3 comments
Labels
01 type: bug something does not work 12 prio: high should be fixed soon 21 status: confirmed issue was read and got confirmed

Comments

@jensens
Copy link
Sponsor Member

jensens commented Apr 2, 2024

zope.configuration.xmlconfig.ZopeXMLConfigurationError: File "/home/jensens/ws/sandbox/check342/backend/.venv/lib/python3.12/site-packages/plone/volto/patches.zcml", line 10.2-15.8
    File "/home/jensens/ws/sandbox/check342/backend/instance/etc/site.zcml", line 12.2-12.23
    File "/home/jensens/ws/sandbox/check342/backend/.venv/lib/python3.12/site-packages/Products/CMFPlone/configure.zcml", line 128.2-132.8
    File "/home/jensens/ws/sandbox/check342/backend/.venv/lib/python3.12/site-packages/plone/volto/configure.zcml", line 20.2-20.33
    ModuleNotFoundError: No module named 'plone.app.upgrade'

With pip install -c https://dist.plone.org/release/6.1-dev/constraints.txt plone.volto

The real bug is here: plone.app.upgrade should always be a soft-dependency, never a hard dependency, since some of us do not want to have it on production instances.

@jensens jensens added 01 type: bug something does not work 12 prio: high should be fixed soon 21 status: confirmed issue was read and got confirmed labels Apr 2, 2024
@davisagli
Copy link
Sponsor Member

@jensens https://github.com/plone/plone.volto/blob/main/src/plone/volto/patches.py is using alias_module from plone.app.upgrade to provide backward compatibility with old content type class locations from collective.folderishtypes.

Should we move alias_module to plone.base maybe?

@jensens
Copy link
Sponsor Member Author

jensens commented Apr 4, 2024

Should we move alias_module to plone.base maybe?

Yes. And since this are only 18 LOC it's something which could be copied to here as well - if it otherwise would break with older versions (https://github.com/plone/plone.volto/blob/fdb85bb5305da27ea9f1747844a43aea3bb5bbbb/setup.py#L36C37-L37C1)

@davisagli
Copy link
Sponsor Member

it otherwise would break with older versions

unless we create a module alias for it....oh wait....

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
01 type: bug something does not work 12 prio: high should be fixed soon 21 status: confirmed issue was read and got confirmed
Projects
None yet
Development

No branches or pull requests

2 participants