From 1659d302d37103fd0f30c7b43412c945bc719c1f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C5=81ukasz=20Nowak?= <luke@nexedi.com> Date: Fri, 27 Aug 2010 05:47:39 +0000 Subject: [PATCH] - switch to zc.buildout 1.5.0 on bootstrap - as bootstrap is passing -S to generated bin/buildout there is not need to use it while invoking bin/buildout - as new buildout will only select non-development releases there is no need to pin to known release - fix Zope KGS issues with buildout 1.5.0 - use distribute by default, as it is more reliable in corner cases then setuptools - inform about another way to updating local bootstrap git-svn-id: https://svn.erp5.org/repos/public/erp5/trunk@38044 20353a03-c40f-0410-a6d1-a30d3c3de9de --- buildout/Makefile | 2 +- buildout/README-2.12.txt | 4 +- buildout/README.txt | 2 +- buildout/bootstrap/README.txt | 2 + buildout/bootstrap/bootstrap.py | 254 ++++++++++++++++++++++------ buildout/profiles/versions-2.12.cfg | 5 + 6 files changed, 209 insertions(+), 60 deletions(-) diff --git a/buildout/Makefile b/buildout/Makefile index 7c9b71a827..2dba8a3c4c 100644 --- a/buildout/Makefile +++ b/buildout/Makefile @@ -16,7 +16,7 @@ software: bin/buildout $(PYTHON) -SE bin/buildout $(BUILDOUT_OPT) bin/buildout: - $(PYTHON) -SE bootstrap/bootstrap.py + $(PYTHON) -SE bootstrap/bootstrap.py -d # run make assert to check that all is working assert: bin/python2.4 diff --git a/buildout/README-2.12.txt b/buildout/README-2.12.txt index 0c0112a640..48fdc76ddf 100644 --- a/buildout/README-2.12.txt +++ b/buildout/README-2.12.txt @@ -37,7 +37,7 @@ For example: Run the Zope 2.12 buildout: $ cd ~/erp5.buildout - $ python2.6 -S bootstrap/bootstrap.py -v 1.4.3 + $ python2.6 -S bootstrap/bootstrap.py -d -v buildout-2.12.cfg $ bin/buildout -v -c buildout-2.12.cfg This will download and install the software components needed to run ERP5 on @@ -115,7 +115,7 @@ parts = supervisor-instance ^D $ ~/erp5.buildout/bin/bootstrap2.6 # 4 -$ python2.6 -S bin/buildout -ov # 5 +$ bin/buildout -ov # 5 Notice how we managed to run buildout in "offline-mode" (-o). The software-home configuration (along with the 'extends-cache' in the 'instance-profiles' diff --git a/buildout/README.txt b/buildout/README.txt index 5b9dc20020..3477730a72 100644 --- a/buildout/README.txt +++ b/buildout/README.txt @@ -128,7 +128,7 @@ $ mkdir software $ cd software $ echo '[buildout]' >> buildout.cfg $ echo 'extends = https://svn.erp5.org/repos/public/erp5/trunk/buildout/buildout.cfg' >> buildout.cfg -$ wget -qO - http://svn.zope.org/*checkout*/zc.buildout/trunk/bootstrap/bootstrap.py | python -S - +$ wget -qO - http://svn.zope.org/*checkout*/zc.buildout/trunk/bootstrap/bootstrap.py | python -S - -d $ python -S bin/buildout Note on -S: this switch is overridden by PYTHON_PATH environment variable. In diff --git a/buildout/bootstrap/README.txt b/buildout/bootstrap/README.txt index a2d00d1327..4fd0f05216 100644 --- a/buildout/bootstrap/README.txt +++ b/buildout/bootstrap/README.txt @@ -1,5 +1,7 @@ bootstrap.py file is updated from time to time from: http://svn.zope.org/*checkout*/zc.buildout/trunk/bootstrap/bootstrap.py +or: + http://svn.zope.org/*checkout*/zc.buildout/tags/RELEASE/bootstrap/bootstrap.py There is no need to fix it locally, all changes shall go upstream. Moreover buildout has ability to autoupdate itself, so local manipulations on diff --git a/buildout/bootstrap/bootstrap.py b/buildout/bootstrap/bootstrap.py index 7f07933ca8..a3d5b927c6 100644 --- a/buildout/bootstrap/bootstrap.py +++ b/buildout/bootstrap/bootstrap.py @@ -18,99 +18,241 @@ The script accepts buildout command-line options, so you can use the -c option to specify an alternate configuration file. """ -import os, shutil, sys, tempfile, urllib2 +import os, shutil, sys, tempfile, textwrap, urllib, urllib2, subprocess from optparse import OptionParser -tmpeggs = tempfile.mkdtemp() +if sys.platform == 'win32': + def quote(c): + if ' ' in c: + return '"%s"' % c # work around spawn lamosity on windows + else: + return c +else: + quote = str + +# See zc.buildout.easy_install._has_broken_dash_S for motivation and comments. +stdout, stderr = subprocess.Popen( + [sys.executable, '-Sc', + 'try:\n' + ' import ConfigParser\n' + 'except ImportError:\n' + ' print 1\n' + 'else:\n' + ' print 0\n'], + stdout=subprocess.PIPE, stderr=subprocess.PIPE).communicate() +has_broken_dash_S = bool(int(stdout.strip())) + +# In order to be more robust in the face of system Pythons, we want to +# run without site-packages loaded. This is somewhat tricky, in +# particular because Python 2.6's distutils imports site, so starting +# with the -S flag is not sufficient. However, we'll start with that: +if not has_broken_dash_S and 'site' in sys.modules: + # We will restart with python -S. + args = sys.argv[:] + args[0:0] = [sys.executable, '-S'] + args = map(quote, args) + os.execv(sys.executable, args) +# Now we are running with -S. We'll get the clean sys.path, import site +# because distutils will do it later, and then reset the path and clean +# out any namespace packages from site-packages that might have been +# loaded by .pth files. +clean_path = sys.path[:] +import site +sys.path[:] = clean_path +for k, v in sys.modules.items(): + if (hasattr(v, '__path__') and + len(v.__path__)==1 and + not os.path.exists(os.path.join(v.__path__[0],'__init__.py'))): + # This is a namespace package. Remove it. + sys.modules.pop(k) is_jython = sys.platform.startswith('java') +setuptools_source = 'http://peak.telecommunity.com/dist/ez_setup.py' +distribute_source = 'http://python-distribute.org/distribute_setup.py' + # parsing arguments -parser = OptionParser() +def normalize_to_url(option, opt_str, value, parser): + if value: + if '://' not in value: # It doesn't smell like a URL. + value = 'file://%s' % ( + urllib.pathname2url( + os.path.abspath(os.path.expanduser(value))),) + if opt_str == '--download-base' and not value.endswith('/'): + # Download base needs a trailing slash to make the world happy. + value += '/' + else: + value = None + name = opt_str[2:].replace('-', '_') + setattr(parser.values, name, value) + +usage = '''\ +[DESIRED PYTHON FOR BUILDOUT] bootstrap.py [options] + +Bootstraps a buildout-based project. + +Simply run this script in a directory containing a buildout.cfg, using the +Python that you want bin/buildout to use. + +Note that by using --setup-source and --download-base to point to +local resources, you can keep this script from going over the network. +''' + +parser = OptionParser(usage=usage) parser.add_option("-v", "--version", dest="version", help="use a specific zc.buildout version") parser.add_option("-d", "--distribute", - action="store_true", dest="distribute", default=False, + action="store_true", dest="use_distribute", default=False, help="Use Distribute rather than Setuptools.") - +parser.add_option("--setup-source", action="callback", dest="setup_source", + callback=normalize_to_url, nargs=1, type="string", + help=("Specify a URL or file location for the setup file. " + "If you use Setuptools, this will default to " + + setuptools_source + "; if you use Distribute, this " + "will default to " + distribute_source +".")) +parser.add_option("--download-base", action="callback", dest="download_base", + callback=normalize_to_url, nargs=1, type="string", + help=("Specify a URL or directory for downloading " + "zc.buildout and either Setuptools or Distribute. " + "Defaults to PyPI.")) +parser.add_option("--eggs", + help=("Specify a directory for storing eggs. Defaults to " + "a temporary directory that is deleted when the " + "bootstrap script completes.")) +parser.add_option("-t", "--accept-buildout-test-releases", + dest='accept_buildout_test_releases', + action="store_true", default=False, + help=("Normally, if you do not specify a --version, the " + "bootstrap script and buildout gets the newest " + "*final* versions of zc.buildout and its recipes and " + "extensions for you. If you use this flag, " + "bootstrap and buildout will get the newest releases " + "even if they are alphas or betas.")) parser.add_option("-c", None, action="store", dest="config_file", help=("Specify the path to the buildout configuration " "file to be used.")) options, args = parser.parse_args() -# if -c was provided, we push it back into args for buildout' main function +# if -c was provided, we push it back into args for buildout's main function if options.config_file is not None: args += ['-c', options.config_file] -if options.version is not None: - VERSION = '==%s' % options.version +if options.eggs: + eggs_dir = os.path.abspath(os.path.expanduser(options.eggs)) else: - VERSION = '' + eggs_dir = tempfile.mkdtemp() + +if options.setup_source is None: + if options.use_distribute: + options.setup_source = distribute_source + else: + options.setup_source = setuptools_source -USE_DISTRIBUTE = options.distribute -args = args + ['bootstrap'] +if options.accept_buildout_test_releases: + args.append('buildout:accept-buildout-test-releases=true') +args.append('bootstrap') try: import pkg_resources - import setuptools + import setuptools # A flag. Sometimes pkg_resources is installed alone. if not hasattr(pkg_resources, '_distribute'): raise ImportError except ImportError: + ez_code = urllib2.urlopen( + options.setup_source).read().replace('\r\n', '\n') ez = {} - if USE_DISTRIBUTE: - exec urllib2.urlopen('http://python-distribute.org/distribute_setup.py' - ).read() in ez - ez['use_setuptools'](to_dir=tmpeggs, download_delay=0, no_fake=True) - else: - exec urllib2.urlopen('http://peak.telecommunity.com/dist/ez_setup.py' - ).read() in ez - ez['use_setuptools'](to_dir=tmpeggs, download_delay=0) - + exec ez_code in ez + setup_args = dict(to_dir=eggs_dir, download_delay=0) + if options.download_base: + setup_args['download_base'] = options.download_base + if options.use_distribute: + setup_args['no_fake'] = True + ez['use_setuptools'](**setup_args) reload(sys.modules['pkg_resources']) import pkg_resources + # This does not (always?) update the default working set. We will + # do it. + for path in sys.path: + if path not in pkg_resources.working_set.entries: + pkg_resources.working_set.add_entry(path) -if sys.platform == 'win32': - def quote(c): - if ' ' in c: - return '"%s"' % c # work around spawn lamosity on windows - else: - return c -else: - def quote (c): - return c +cmd = [quote(sys.executable), + '-c', + quote('from setuptools.command.easy_install import main; main()'), + '-mqNxd', + quote(eggs_dir)] -cmd = 'from setuptools.command.easy_install import main; main()' -ws = pkg_resources.working_set +if not has_broken_dash_S: + cmd.insert(1, '-S') -if USE_DISTRIBUTE: - requirement = 'distribute' +find_links = options.download_base +if not find_links: + find_links = os.environ.get('bootstrap-testing-find-links') +if find_links: + cmd.extend(['-f', quote(find_links)]) + +if options.use_distribute: + setup_requirement = 'distribute' else: - requirement = 'setuptools' + setup_requirement = 'setuptools' +ws = pkg_resources.working_set +setup_requirement_path = ws.find( + pkg_resources.Requirement.parse(setup_requirement)).location +env = dict( + os.environ, + PYTHONPATH=setup_requirement_path) + +requirement = 'zc.buildout' +version = options.version +if version is None and not options.accept_buildout_test_releases: + # Figure out the most recent final version of zc.buildout. + import setuptools.package_index + _final_parts = '*final-', '*final' + def _final_version(parsed_version): + for part in parsed_version: + if (part[:1] == '*') and (part not in _final_parts): + return False + return True + index = setuptools.package_index.PackageIndex( + search_path=[setup_requirement_path]) + if find_links: + index.add_find_links((find_links,)) + req = pkg_resources.Requirement.parse(requirement) + if index.obtain(req) is not None: + best = [] + bestv = None + for dist in index[req.project_name]: + distv = dist.parsed_version + if _final_version(distv): + if bestv is None or distv > bestv: + best = [dist] + bestv = distv + elif distv == bestv: + best.append(dist) + if best: + best.sort() + version = best[-1].version +if version: + requirement = '=='.join((requirement, version)) +cmd.append(requirement) if is_jython: import subprocess + exitcode = subprocess.Popen(cmd, env=env).wait() +else: # Windows prefers this, apparently; otherwise we would prefer subprocess + exitcode = os.spawnle(*([os.P_WAIT, sys.executable] + cmd + [env])) +if exitcode != 0: + sys.stdout.flush() + sys.stderr.flush() + print ("An error occurred when trying to install zc.buildout. " + "Look above this message for any errors that " + "were output by easy_install.") + sys.exit(exitcode) - assert subprocess.Popen([sys.executable] + ['-c', quote(cmd), '-mqNxd', - quote(tmpeggs), 'zc.buildout' + VERSION], - env=dict(os.environ, - PYTHONPATH= - ws.find(pkg_resources.Requirement.parse(requirement)).location - ), - ).wait() == 0 - -else: - assert os.spawnle( - os.P_WAIT, sys.executable, quote (sys.executable), - '-c', quote (cmd), '-mqNxd', quote (tmpeggs), 'zc.buildout' + VERSION, - dict(os.environ, - PYTHONPATH= - ws.find(pkg_resources.Requirement.parse(requirement)).location - ), - ) == 0 - -ws.add_entry(tmpeggs) -ws.require('zc.buildout' + VERSION) +ws.add_entry(eggs_dir) +ws.require(requirement) import zc.buildout.buildout zc.buildout.buildout.main(args) -shutil.rmtree(tmpeggs) +if not options.eggs: # clean up temporary egg directory + shutil.rmtree(eggs_dir) diff --git a/buildout/profiles/versions-2.12.cfg b/buildout/profiles/versions-2.12.cfg index e36266490f..8e0190adfa 100644 --- a/buildout/profiles/versions-2.12.cfg +++ b/buildout/profiles/versions-2.12.cfg @@ -34,3 +34,8 @@ erp5_bt5_revision = ${:erp5_products_revision} # 1.3.1 fixes environment leak hexagonit.recipe.cmmi >= 1.3.1 + +# Zope KGS is incompatible with buildout 1.5.0 in case of distribute and +# zc.buildout +distribute >= 0.6.14 +zc.buildout >= 1.5.0 -- 2.30.9