Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow the bootstrap to be verbose #265

Merged
merged 1 commit into from
Apr 10, 2012

Conversation

mathstuf
Copy link
Contributor

Addresses #264.

evmar added a commit that referenced this pull request Apr 10, 2012
@evmar evmar merged commit 39bf6c9 into ninja-build:master Apr 10, 2012
@johnbartholomew
Copy link

It might be worth noting somewhere that with this patch, configure flags must be separated from bootstrap flags:

This used to work (passing the option through to configure.py):

./bootstrap.py --with-python=/usr/bin/python2

It now fails because the OptionParser in bootstrap.py doesn't recognise the --with-python flag. To avoid this you must use -- to explicitly mark the end of the option flags, like this:

./bootstrap.py -- --with-python=/usr/bin/python2

@mathstuf mathstuf deleted the dev/verbose-bootstrap branch June 13, 2013 03:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants