From 075847031b6cf7b9de3bb7c42648a74d168a752b Mon Sep 17 00:00:00 2001 From: Elliott Hird Date: Thu, 15 Dec 2011 14:52:22 +0000 Subject: Rewrite the bootstrap script in Python This allows trouble-free builds on platforms with non-standard Python executable names. As a bonus, it passes the arguments received on to configure.py, so you don't have to reconfigure to build in a non-standard manner. --- HACKING | 4 ++-- README | 2 +- bootstrap.py | 72 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ bootstrap.sh | 53 ------------------------------------------- src/browse.py | 2 +- 5 files changed, 76 insertions(+), 57 deletions(-) create mode 100755 bootstrap.py delete mode 100755 bootstrap.sh diff --git a/HACKING b/HACKING index dcd5888..f8545e5 100644 --- a/HACKING +++ b/HACKING @@ -1,5 +1,5 @@ Adjusting build flags: - CFLAGS=-O3 ./configure + CFLAGS=-O3 ./configure.py and rebuild. Test-driven development: @@ -56,7 +56,7 @@ Windows development on Linux (this is kind of hacky right now): Windows development on Windows: - install mingw, msys, and python -- in the mingw shell, put Python in your path, and: sh bootstrap.sh +- in the mingw shell, put Python in your path, and: python bootstrap.py - to reconfigure, run 'python configure.py' - remember to strip the resulting executable if size matters to you - you'll need to rename ninja.exe into my-ninja.exe during development, diff --git a/README b/README index 1b80cf1..a8fe582 100644 --- a/README +++ b/README @@ -5,7 +5,7 @@ See the manual -- http://martine.github.com/ninja/manual.html or doc/manual.asciidoc included in the distribution -- for background and more details. -To build, run ./bootstrap.sh. It first blindly compiles all non-test +To build, run ./bootstrap.py. It first blindly compiles all non-test source files together, then re-builds Ninja using itself. You should end up with a 'ninja' binary in the source root. Run './ninja -h' for help. diff --git a/bootstrap.py b/bootstrap.py new file mode 100755 index 0000000..ea35a5b --- /dev/null +++ b/bootstrap.py @@ -0,0 +1,72 @@ +#!/usr/bin/env python +# Copyright 2011 Google Inc. All Rights Reserved. +# +# Licensed under the Apache License, Version 2.0 (the "License"); +# you may not use this file except in compliance with the License. +# You may obtain a copy of the License at +# +# http://www.apache.org/licenses/LICENSE-2.0 +# +# Unless required by applicable law or agreed to in writing, software +# distributed under the License is distributed on an "AS IS" BASIS, +# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +# See the License for the specific language governing permissions and +# limitations under the License. + +import re +import sys +import os +import glob +import errno +import subprocess + +def run(*args, **kwargs): + try: + subprocess.check_call(*args, **kwargs) + except subprocess.CalledProcessError, e: + sys.exit(e.returncode) + +# Compute system-specific CFLAGS/LDFLAGS as used in both in the below +# g++ call as well as in the later configure.py. +cflags = os.environ.get('CFLAGS', '') +ldflags = os.environ.get('LDFLAGS', '') +if sys.platform.startswith('freebsd'): + cflags += ' -I/usr/local/include' + ldflags += ' -L/usr/local/lib' + +print 'Building ninja manually...' + +try: + os.mkdir('build') +except OSError, e: + if e.errno != errno.EEXIST: + raise + +with open('src/browse.py') as browse_py: + with open('build/browse_py.h', 'w') as browse_py_h: + run(['./src/inline.sh', 'kBrowsePy'], + stdin=browse_py, stdout=browse_py_h) + +pattern = r'test\.cc$|\.in\.cc$' + +if sys.platform.startswith('win32'): + pattern += r'|/browse\.cc$|/subprocess\.cc$' +else: + pattern += r'|-win32\.cc$' + +sources = [src for src in glob.glob('src/*.cc') if not re.search(pattern, src)] + +args = [os.environ.get('CXX', 'g++'), '-Wno-deprecated', + '-DNINJA_PYTHON="' + sys.executable + '"'] +args.extend(cflags.split()) +args.extend(ldflags.split()) +args.extend(['-o', 'ninja.bootstrap']) +args.extend(sources) +run(args) + +print 'Building ninja using itself...' +run([sys.executable, 'configure.py'] + sys.argv[1:]) +run(['./ninja.bootstrap']) +os.unlink('ninja.bootstrap') + +print 'Done!' diff --git a/bootstrap.sh b/bootstrap.sh deleted file mode 100755 index 35ffd9e..0000000 --- a/bootstrap.sh +++ /dev/null @@ -1,53 +0,0 @@ -#!/bin/sh - -# Copyright 2011 Google Inc. All Rights Reserved. -# -# Licensed under the Apache License, Version 2.0 (the "License"); -# you may not use this file except in compliance with the License. -# You may obtain a copy of the License at -# -# http://www.apache.org/licenses/LICENSE-2.0 -# -# Unless required by applicable law or agreed to in writing, software -# distributed under the License is distributed on an "AS IS" BASIS, -# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. -# See the License for the specific language governing permissions and -# limitations under the License. - -set -e - -SYSTEMNAME=`uname -s` - -# Compute system-specific CFLAGS/LDFLAGS as used in both in the below -# g++ call as well as in the later configure.py. -if [ "${SYSTEMNAME}" = "Linux" ]; then - export CFLAGS="${CFLAGS}" - export LDFLAGS="${LDFLAGS}" -elif [ "${SYSTEMNAME}" = "FreeBSD" ]; then - export CFLAGS="${CFLAGS} -I/usr/local/include" - export LDFLAGS="${LDFLAGS} -L/usr/local/lib" -fi - -echo "Building ninja manually..." -mkdir -p build -./src/inline.sh kBrowsePy < src/browse.py > build/browse_py.h - -pattern='test\.cc$\|\.in\.cc$' -case "$SYSTEMNAME" in - MINGW32*) - pattern="$pattern"'\|/browse\.cc$\|/subprocess\.cc$' - ;; - *) - pattern="$pattern"'\|-win32\.cc$' - ;; -esac -srcs=$(ls src/*.cc | grep -v "$pattern") - -${CXX:-g++} -Wno-deprecated ${CFLAGS} ${LDFLAGS} -o ninja.bootstrap $srcs - -echo "Building ninja using itself..." -python ./configure.py -./ninja.bootstrap -rm ninja.bootstrap - -echo "Done!" diff --git a/src/browse.py b/src/browse.py index 9901adc..ca95197 100755 --- a/src/browse.py +++ b/src/browse.py @@ -1,4 +1,4 @@ -#!/usr/bin/python +#!/usr/bin/env python # # Copyright 2001 Google Inc. All Rights Reserved. # -- cgit v0.12