build: configure formatting, add final message
authorRoman Reiss <me@silverwind.io>
Wed, 28 Jan 2015 16:46:19 +0000 (17:46 +0100)
committerBen Noordhuis <info@bnoordhuis.nl>
Wed, 28 Jan 2015 17:03:54 +0000 (18:03 +0100)
This restores the pretty-printed options output for improved
readability and adds a final warning message to inform on possibly
scrolled-off warnings.

PR-URL: https://github.com/iojs/io.js/pull/638
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
configure

index fb271c6..e85621e 100755 (executable)
--- a/configure
+++ b/configure
@@ -8,7 +8,6 @@ import subprocess
 import sys
 import shutil
 import string
-import textwrap
 
 CC = os.environ.get('CC', 'cc')
 CXX = os.environ.get('CXX', 'c++')
@@ -297,9 +296,12 @@ auto_downloads = nodedownload.parse(options.download_list)
 
 
 def warn(msg):
-  prefix = '\033[1m\033[91mWARNING\033[0m' if os.isatty(1) else 'WARNING'
+  warn.warned = True
+  prefix = '\033[1m\033[93mWARNING\033[0m' if os.isatty(1) else 'WARNING'
   print('%s: %s' % (prefix, msg))
 
+# track if warnings occured
+warn.warned = False
 
 def b(value):
   """Returns the string 'true' if value is truthy, 'false' otherwise."""
@@ -960,7 +962,7 @@ output = {
   'variables': variables,
   'target_defaults': output
 }
-print textwrap.fill(str(output), 78)
+pprint.pprint(output, indent=2)
 
 write('config.gypi', do_not_edit +
       pprint.pformat(output, indent=2) + '\n')
@@ -990,4 +992,7 @@ else:
 
 gyp_args += args
 
+if warn.warned:
+  warn('warnings were emitted in the configure phase')
+
 sys.exit(subprocess.call(gyp_args))