Commit 8d86fe3a authored by Reinout van Rees's avatar Reinout van Rees

Removed buildout.dumppickedversions from the warning.

I *have* retained buildout.dumppickedversions in the doctest with
the qualification 'only part of it'.

Removing the warning *does* make it less obvious for the people
that use buildout.dumppickedversions in the way I thought it
was used most, but apparently the default behaviour of dumppickedversions
was otherwise.
parent 8c13789c
...@@ -953,14 +953,13 @@ class Buildout(DictMixin): ...@@ -953,14 +953,13 @@ class Buildout(DictMixin):
def _load_extensions(self): def _load_extensions(self):
__doing__ = 'Loading extensions.' __doing__ = 'Loading extensions.'
specs = self['buildout'].get('extensions', '').split() specs = self['buildout'].get('extensions', '').split()
for superceded_extension in ['buildout-versions', if 'buildout-versions' in specs:
'buildout.dumppickedversions']: raise zc.buildout.UserError(
if superceded_extension in specs: "The extension 'buildout-versions' is now included in "
msg = ("The extension %s is now included in buildout itself.\n" "buildout itself.\n"
"Remove the extension from your configuration and " "Remove the extension from your configuration and "
"look at the `show-picked-versions`\n" "look at the `show-picked-versions`\n"
"option in buildout's documentation.") "option in buildout's documentation.")
raise zc.buildout.UserError(msg % superceded_extension)
if specs: if specs:
path = [self['buildout']['develop-eggs-directory']] path = [self['buildout']['develop-eggs-directory']]
if self.offline: if self.offline:
......
...@@ -408,7 +408,7 @@ The versions file contains the extra pin: ...@@ -408,7 +408,7 @@ The versions file contains the extra pin:
>>> 'spam = 2' in open('my_versions.cfg').read() >>> 'spam = 2' in open('my_versions.cfg').read()
True True
Because buildout now includes buildout-versions' (and the older Because buildout now includes buildout-versions' (and part of the older
buildout.dumppickedversions') functionality, it warns if these extensions are buildout.dumppickedversions') functionality, it warns if these extensions are
configured. configured.
...@@ -425,6 +425,6 @@ configured. ...@@ -425,6 +425,6 @@ configured.
While: While:
Installing. Installing.
Loading extensions. Loading extensions.
Error: The extension buildout-versions is now included in buildout itself. Error: The extension 'buildout-versions' is now included in buildout itself.
Remove the extension from your configuration and look at the `show-picked-versions` Remove the extension from your configuration and look at the `show-picked-versions`
option in buildout's documentation. option in buildout's documentation.
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment