plone / plone.recipe.zope2instance

zc.buildout recipe to setup and configure a Zope 2 instance.
https://pypi.org/project/plone.recipe.zope2instance
7 stars 23 forks source link

Fix zdaemon socket location #79

Closed tschorr closed 5 years ago

tschorr commented 5 years ago

zdaemon sets a default name 'zdsock', which is respected by ctl.py (instead of using it's own per instance default). As a result, you cannot start two instances/ZEO clients from the same directory because they both try to use the zdsock in cwd.

This PR restores the ctl.py default (zopectlsock in clienthome).

mister-roboto commented 5 years ago

@tschorr thanks for creating this Pull Request and help improve Plone!

To ensure that these changes do not break other parts of Plone, the Plone test suite matrix needs to pass.

Whenever you feel that the pull request is ready to be tested, either start all jenkins jobs pull requests by yourself, or simply add a comment in this pull request stating:

@jenkins-plone-org please run jobs

With this simple comment all the jobs will be started automatically.

Happy hacking!

tschorr commented 5 years ago

@jenkins-plone-org please run jobs

tschorr commented 5 years ago

@jenkins-plone-org please run jobs

tschorr commented 5 years ago

The one failing Jenkins robot test looks unrelated.