You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The reason will be displayed to describe this comment to others. Learn more.
This does NOT work, that's why I changed it to include Paste explicitly. The Zope scripts like makewsgiinstance will not create a startup script that includes Paste and that is why I changed it in f4fd6b6. I will revert this change, it caused yet another complaint at #1154
The reason will be displayed to describe this comment to others. Learn more.
Michael Howitz wrote at 2023-9-14 23:23 -0700:
That's interesting. If `Zope[wsgi]` does not work, maybe we should remove it from `setup.py`.
`Zope[wsgi]` does work for the `virtualenv/pip` based installation
method.
Apparently, it does not fully work for `buildout` based installations
(maybe, due to a `buildout` bug: `buildout` fails to include
the `extra_requires` eggs into its `eggs` definition).
The reason will be displayed to describe this comment to others. Learn more.
We're at a place where buildout is not getting better, it's getting buggier - or the bugs are more obvious. And I don't see much development activity. Buildout is also partly to blame for the issues with Python in universal mode on newer macOS versions. There are days when I wonder if we should standardize on built-in Python tools instead.
The reason will be displayed to describe this comment to others. Learn more.
Maybe the times have come to longer propose using zc.buildout as the default way or even abandon using it. (e. g. We could start to use pip for the tests of the Products.* packages instead of buildout.)
a0cbc57
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This does NOT work, that's why I changed it to include Paste explicitly. The Zope scripts like
makewsgiinstance
will not create a startup script that includesPaste
and that is why I changed it in f4fd6b6. I will revert this change, it caused yet another complaint at #1154a0cbc57
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's interesting. If
Zope[wsgi]
does not work, maybe we should remove it fromsetup.py
.a0cbc57
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a0cbc57
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're at a place where buildout is not getting better, it's getting buggier - or the bugs are more obvious. And I don't see much development activity. Buildout is also partly to blame for the issues with Python in universal mode on newer macOS versions. There are days when I wonder if we should standardize on built-in Python tools instead.
a0cbc57
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe the times have come to longer propose using zc.buildout as the default way or even abandon using it. (e. g. We could start to use
pip
for the tests of the Products.* packages instead of buildout.)a0cbc57
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.