Skip to content

Commit

Permalink
[fc] Repository: Products.CMFPlone
Browse files Browse the repository at this point in the history
Branch: refs/heads/master
Date: 2024-08-29T20:40:18+02:00
Author: Maurits van Rees (mauritsvanrees) <maurits@vanrees.org>
Commit: plone/Products.CMFPlone@68e7d9f

Use five.registerPackage so an editable install with pip works.

Fixes plone/Products.CMFPlone#4002

Files changed:
A news/4002.bugfix
M Products/CMFPlone/configure.zcml
Repository: Products.CMFPlone

Branch: refs/heads/master
Date: 2024-08-31T22:44:49+02:00
Author: Maurits van Rees (mauritsvanrees) <maurits@py76.be>
Commit: plone/Products.CMFPlone@6549387

Merge pull request #4003 from plone/maurits-issue-4002-five-register-package-61

Use five.registerPackage so an editable install with pip works. [6.1]

Files changed:
A news/4002.bugfix
M Products/CMFPlone/configure.zcml
  • Loading branch information
mauritsvanrees committed Aug 31, 2024
1 parent 17a27e2 commit 365ddba
Showing 1 changed file with 18 additions and 18 deletions.
36 changes: 18 additions & 18 deletions last_commit.txt
Original file line number Diff line number Diff line change
@@ -1,36 +1,36 @@
Repository: plone.releaser
Repository: Products.CMFPlone


Branch: refs/heads/master
Date: 2024-08-29T21:10:34+02:00
Date: 2024-08-29T20:40:18+02:00
Author: Maurits van Rees (mauritsvanrees) <maurits@vanrees.org>
Commit: https://github.com/plone/plone.releaser/commit/47c4da206b9f99fd94b371cec52135c90818f336
Commit: https://github.com/plone/Products.CMFPlone/commit/68e7d9f2cd11c7369a8fb6e34f7ab5dec48c8689

Fix checking/adding/removing checkouts: search in mxsources.ini.
Use five.registerPackage so an editable install with pip works.

Fixes https://github.com/plone/Products.CMFPlone/issues/4002

Files changed:
A news/945.bugfix
M plone/releaser/manage.py
M plone/releaser/pip.py
A news/4002.bugfix
M Products/CMFPlone/configure.zcml

b'diff --git a/news/945.bugfix b/news/945.bugfix\nnew file mode 100644\nindex 0000000..f237dd6\n--- /dev/null\n+++ b/news/945.bugfix\n@@ -0,0 +1,2 @@\n+Fix checking/adding/removing checkouts: search in `mxsources.ini`.\n+[maurits]\ndiff --git a/plone/releaser/manage.py b/plone/releaser/manage.py\nindex b235d28..25b036f 100644\n--- a/plone/releaser/manage.py\n+++ b/plone/releaser/manage.py\n@@ -137,12 +137,12 @@ def _get_checkouts(path=None):\n """Get the parsed checkouts file at the given path.\n \n If no path is given, we use several paths:\n- both checkouts.cfg and mxdev.ini.\n+ both checkouts.cfg and mxcheckouts.ini.\n """\n if path:\n paths = [path]\n else:\n- paths = glob.glob("mxdev.ini") + glob.glob("checkouts.cfg")\n+ paths = glob.glob("mxcheckouts.ini") + glob.glob("checkouts.cfg")\n for path in paths:\n if path.endswith(".ini"):\n checkouts = MxCheckoutsFile(path)\n@@ -155,7 +155,7 @@ def check_checkout(package_name, *, path=None):\n """Check if package is in the checkouts.\n \n If no path is given, we try several paths:\n- both checkouts.cfg and mxdev.ini.\n+ both checkouts.cfg and mxcheckouts.ini.\n """\n for checkouts in _get_checkouts(path=path):\n loc = checkouts.file_location\n@@ -169,7 +169,7 @@ def remove_checkout(package_name, *, path=None):\n """Remove package from auto checkouts.\n \n If no path is given, we try several paths:\n- both checkouts.cfg and mxdev.ini.\n+ both checkouts.cfg and mxcheckouts.ini.\n """\n for checkouts in _get_checkouts(path=path):\n checkouts.remove(package_name)\n@@ -179,7 +179,7 @@ def add_checkout(package_name, *, path=None):\n """Add package to auto checkouts.\n \n If no path is given, we try several paths:\n- both checkouts.cfg and mxdev.ini.\n+ both checkouts.cfg and mxcheckouts.ini.\n """\n for checkouts in _get_checkouts(path=path):\n checkouts.add(package_name)\ndiff --git a/plone/releaser/pip.py b/plone/releaser/pip.py\nindex d791fa9..fb1ded9 100644\n--- a/plone/releaser/pip.py\n+++ b/plone/releaser/pip.py\n@@ -254,6 +254,7 @@ def append_package(self, package_name, enabled=True):\n use = "true" if enabled else "false"\n contents += f"\\n[{package_name}]\\nuse = {use}\\n"\n self.path.write_text(contents)\n+ print(f"{self.file_location}: {package_name} added to checkouts.")\n \n def __setitem__(self, package_name, enabled=True):\n """Enable or disable a checkout.\n'
b'diff --git a/Products/CMFPlone/configure.zcml b/Products/CMFPlone/configure.zcml\nindex c1a3ba041d..b44a78d4de 100644\n--- a/Products/CMFPlone/configure.zcml\n+++ b/Products/CMFPlone/configure.zcml\n@@ -2,11 +2,13 @@\n xmlns="http://namespaces.zope.org/zope"\n xmlns:browser="http://namespaces.zope.org/browser"\n xmlns:cmf="http://namespaces.zope.org/cmf"\n+ xmlns:five="http://namespaces.zope.org/five"\n xmlns:i18n="http://namespaces.zope.org/i18n"\n xmlns:zcml="http://namespaces.zope.org/zcml"\n i18n_domain="plone"\n >\n \n+ <five:registerPackage package="." initialize=".initialize" />\n <!-- basic zope/cmf -->\n <include package="zope.app.locales" />\n <include package="Products.CMFCore" />\ndiff --git a/news/4002.bugfix b/news/4002.bugfix\nnew file mode 100644\nindex 0000000000..8dc1c4c0e3\n--- /dev/null\n+++ b/news/4002.bugfix\n@@ -0,0 +1,2 @@\n+Use `five.registerPackage` so an editable install with `pip` works.\n+[maurits]\n'

Repository: plone.releaser
Repository: Products.CMFPlone


Branch: refs/heads/master
Date: 2024-08-31T22:44:05+02:00
Author: Maurits van Rees (mauritsvanrees) <maurits@vanrees.org>
Commit: https://github.com/plone/plone.releaser/commit/b5ae17e8f336fd06bfe907baa62f97d01bd313e8
Date: 2024-08-31T22:44:49+02:00
Author: Maurits van Rees (mauritsvanrees) <maurits@py76.be>
Commit: https://github.com/plone/Products.CMFPlone/commit/6549387f9302933bbd1202fdd24db9399f38f9b7

Merge pull request #74 from plone/maurits-mxcheckouts-ini
Merge pull request #4003 from plone/maurits-issue-4002-five-register-package-61

Fix checking/adding/removing checkouts: search in mxsources.ini.
Use five.registerPackage so an editable install with pip works. [6.1]

Files changed:
A news/945.bugfix
M plone/releaser/manage.py
M plone/releaser/pip.py
A news/4002.bugfix
M Products/CMFPlone/configure.zcml

b'diff --git a/news/945.bugfix b/news/945.bugfix\nnew file mode 100644\nindex 0000000..f237dd6\n--- /dev/null\n+++ b/news/945.bugfix\n@@ -0,0 +1,2 @@\n+Fix checking/adding/removing checkouts: search in `mxsources.ini`.\n+[maurits]\ndiff --git a/plone/releaser/manage.py b/plone/releaser/manage.py\nindex b235d28..25b036f 100644\n--- a/plone/releaser/manage.py\n+++ b/plone/releaser/manage.py\n@@ -137,12 +137,12 @@ def _get_checkouts(path=None):\n """Get the parsed checkouts file at the given path.\n \n If no path is given, we use several paths:\n- both checkouts.cfg and mxdev.ini.\n+ both checkouts.cfg and mxcheckouts.ini.\n """\n if path:\n paths = [path]\n else:\n- paths = glob.glob("mxdev.ini") + glob.glob("checkouts.cfg")\n+ paths = glob.glob("mxcheckouts.ini") + glob.glob("checkouts.cfg")\n for path in paths:\n if path.endswith(".ini"):\n checkouts = MxCheckoutsFile(path)\n@@ -155,7 +155,7 @@ def check_checkout(package_name, *, path=None):\n """Check if package is in the checkouts.\n \n If no path is given, we try several paths:\n- both checkouts.cfg and mxdev.ini.\n+ both checkouts.cfg and mxcheckouts.ini.\n """\n for checkouts in _get_checkouts(path=path):\n loc = checkouts.file_location\n@@ -169,7 +169,7 @@ def remove_checkout(package_name, *, path=None):\n """Remove package from auto checkouts.\n \n If no path is given, we try several paths:\n- both checkouts.cfg and mxdev.ini.\n+ both checkouts.cfg and mxcheckouts.ini.\n """\n for checkouts in _get_checkouts(path=path):\n checkouts.remove(package_name)\n@@ -179,7 +179,7 @@ def add_checkout(package_name, *, path=None):\n """Add package to auto checkouts.\n \n If no path is given, we try several paths:\n- both checkouts.cfg and mxdev.ini.\n+ both checkouts.cfg and mxcheckouts.ini.\n """\n for checkouts in _get_checkouts(path=path):\n checkouts.add(package_name)\ndiff --git a/plone/releaser/pip.py b/plone/releaser/pip.py\nindex d791fa9..fb1ded9 100644\n--- a/plone/releaser/pip.py\n+++ b/plone/releaser/pip.py\n@@ -254,6 +254,7 @@ def append_package(self, package_name, enabled=True):\n use = "true" if enabled else "false"\n contents += f"\\n[{package_name}]\\nuse = {use}\\n"\n self.path.write_text(contents)\n+ print(f"{self.file_location}: {package_name} added to checkouts.")\n \n def __setitem__(self, package_name, enabled=True):\n """Enable or disable a checkout.\n'
b'diff --git a/Products/CMFPlone/configure.zcml b/Products/CMFPlone/configure.zcml\nindex c1a3ba041d..b44a78d4de 100644\n--- a/Products/CMFPlone/configure.zcml\n+++ b/Products/CMFPlone/configure.zcml\n@@ -2,11 +2,13 @@\n xmlns="http://namespaces.zope.org/zope"\n xmlns:browser="http://namespaces.zope.org/browser"\n xmlns:cmf="http://namespaces.zope.org/cmf"\n+ xmlns:five="http://namespaces.zope.org/five"\n xmlns:i18n="http://namespaces.zope.org/i18n"\n xmlns:zcml="http://namespaces.zope.org/zcml"\n i18n_domain="plone"\n >\n \n+ <five:registerPackage package="." initialize=".initialize" />\n <!-- basic zope/cmf -->\n <include package="zope.app.locales" />\n <include package="Products.CMFCore" />\ndiff --git a/news/4002.bugfix b/news/4002.bugfix\nnew file mode 100644\nindex 0000000000..8dc1c4c0e3\n--- /dev/null\n+++ b/news/4002.bugfix\n@@ -0,0 +1,2 @@\n+Use `five.registerPackage` so an editable install with `pip` works.\n+[maurits]\n'

0 comments on commit 365ddba

Please sign in to comment.