Browse files

gyp: switch json for pprint when generating config

  • Loading branch information...
1 parent f999411 commit 45605c9b39ede7f0b7ccf0184949ad95ad7268ab @jbergstroem jbergstroem committed with bnoordhuis Dec 18, 2011
Showing with 2 additions and 3 deletions.
  1. +2 −3 configure
@@ -2,13 +2,12 @@
import optparse
import os
-import json
+import pprint
import subprocess
import sys
root_dir = os.path.dirname(__file__)
sys.path.insert(0, os.path.join(root_dir, 'deps', 'v8', 'tools'))
-import utils # GuessArchitecture
# parse our options
parser = optparse.OptionParser()
@@ -262,7 +261,7 @@ print "creating ", fn
f = open(fn, 'w+')
f.write("# Do not edit. Generated by the configure script.\n")
-json.dump(output, f, indent=2, skipkeys=True)
+pprint.pprint(output, stream=f, indent=2)

3 comments on commit 45605c9

ry commented on 45605c9 Dec 20, 2011

This seems to have broken make install cc @jbergstroem

ry replied Dec 20, 2011

"fixed" in b603578


jbergstroem replied Dec 20, 2011

The JSON parsers I tested with seems to be more forgiving. pprint doesn't seem to have enough options to help us with this, so it's either this or the import json debacle :-(

Please sign in to comment.