Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Newer
Older
100755 472 lines (368 sloc) 13.417 kb
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
1 #!/usr/bin/env python
2 import optparse
3 import os
45605c9 @jbergstroem gyp: switch json for pprint when generating config
jbergstroem authored
4 import pprint
30b29d8 @bnoordhuis build: disable -fstrict-aliasing if gcc < 4.6.0
bnoordhuis authored
5 import re
b9e1bb3 @bnoordhuis build: auto-run tools/gyp_node after ./configure
bnoordhuis authored
6 import subprocess
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
7 import sys
5cf4cef @ry add configure
ry authored
8
5062741 @bnoordhuis build: make CC command in -fstrict-aliasing check configurable
bnoordhuis authored
9 CC = os.environ.get('CC', 'cc')
10
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
11 root_dir = os.path.dirname(__file__)
12 sys.path.insert(0, os.path.join(root_dir, 'deps', 'v8', 'tools'))
97c9745 @ry Back to WAF
ry authored
13
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
14 # parse our options
15 parser = optparse.OptionParser()
97c9745 @ry Back to WAF
ry authored
16
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
17 parser.add_option("--debug",
18 action="store_true",
19 dest="debug",
20 help="Also build debug build")
21
22 parser.add_option("--prefix",
23 action="store",
24 dest="prefix",
25 help="Select the install prefix (defaults to /usr/local)")
26
a9f2c4a @indutny --without-npm option
indutny authored
27 parser.add_option("--without-npm",
28 action="store_true",
29 dest="without_npm",
30 help="Don\'t install the bundled npm package manager")
31
6e76a7c @indutny installer: install node-waf
indutny authored
32 parser.add_option("--without-waf",
33 action="store_true",
34 dest="without_waf",
35 help="Don\'t install node-waf")
36
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
37 parser.add_option("--without-ssl",
38 action="store_true",
39 dest="without_ssl",
40 help="Build without SSL")
41
42 parser.add_option("--without-snapshot",
43 action="store_true",
44 dest="without_snapshot",
45 help="Build without snapshotting V8 libraries. You might want to set"
46 " this for cross-compiling. [Default: False]")
47
48 parser.add_option("--shared-v8",
49 action="store_true",
50 dest="shared_v8",
51 help="Link to a shared V8 DLL instead of static linking")
52
53 parser.add_option("--shared-v8-includes",
54 action="store",
55 dest="shared_v8_includes",
56 help="Directory containing V8 header files")
57
58 parser.add_option("--shared-v8-libpath",
59 action="store",
60 dest="shared_v8_libpath",
61 help="A directory to search for the shared V8 DLL")
62
63 parser.add_option("--shared-v8-libname",
64 action="store",
65 dest="shared_v8_libname",
66 help="Alternative lib name to link to (default: 'v8')")
67
f315029 @bnoordhuis build: rename openssl configure switches
bnoordhuis authored
68 parser.add_option("--shared-openssl",
69 action="store_true",
70 dest="shared_openssl",
71 help="Link to a shared OpenSSl DLL instead of static linking")
72
73 parser.add_option("--shared-openssl-includes",
74 action="store",
75 dest="shared_openssl_includes",
76 help="Directory containing OpenSSL header files")
77
78 parser.add_option("--shared-openssl-libpath",
79 action="store",
80 dest="shared_openssl_libpath",
81 help="A directory to search for the shared OpenSSL DLLs")
82
83 parser.add_option("--shared-openssl-libname",
84 action="store",
85 dest="shared_openssl_libname",
86 help="Alternative lib name to link to (default: 'crypto,ssl')")
87
88 # deprecated
e61de70 @ry gyp: use built-in openssl by default
ry authored
89 parser.add_option("--openssl-use-sys",
70e6889 @TooTallNate build: make --openssl-use-sys a boolean option
TooTallNate authored
90 action="store_true",
f315029 @bnoordhuis build: rename openssl configure switches
bnoordhuis authored
91 dest="shared_openssl",
92 help=optparse.SUPPRESS_HELP)
e61de70 @ry gyp: use built-in openssl by default
ry authored
93
f315029 @bnoordhuis build: rename openssl configure switches
bnoordhuis authored
94 # deprecated
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
95 parser.add_option("--openssl-includes",
96 action="store",
f315029 @bnoordhuis build: rename openssl configure switches
bnoordhuis authored
97 dest="shared_openssl_includes",
98 help=optparse.SUPPRESS_HELP)
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
99
f315029 @bnoordhuis build: rename openssl configure switches
bnoordhuis authored
100 # deprecated
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
101 parser.add_option("--openssl-libpath",
102 action="store",
f315029 @bnoordhuis build: rename openssl configure switches
bnoordhuis authored
103 dest="shared_openssl_libpath",
104 help=optparse.SUPPRESS_HELP)
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
105
106 parser.add_option("--no-ssl2",
107 action="store_true",
108 dest="no_ssl2",
109 help="Disable OpenSSL v2")
110
d03b848 @tchollingsworth build: support shared zlib
tchollingsworth authored
111 parser.add_option("--shared-zlib",
112 action="store_true",
113 dest="shared_zlib",
114 help="Link to a shared zlib DLL instead of static linking")
115
116 parser.add_option("--shared-zlib-includes",
117 action="store",
118 dest="shared_zlib_includes",
119 help="Directory containing zlib header files")
120
121 parser.add_option("--shared-zlib-libpath",
122 action="store",
123 dest="shared_zlib_libpath",
124 help="A directory to search for the shared zlib DLL")
125
126 parser.add_option("--shared-zlib-libname",
127 action="store",
128 dest="shared_zlib_libname",
129 help="Alternative lib name to link to (default: 'z')")
130
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
131 parser.add_option("--with-dtrace",
132 action="store_true",
133 dest="with_dtrace",
cc15299 @davepacheco build: add support for DTrace and postmortem
davepacheco authored
134 help="Build with DTrace (default is true on supported systems)")
135
136 parser.add_option("--without-dtrace",
137 action="store_true",
138 dest="without_dtrace",
139 help="Build without DTrace")
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
140
35a1421 Windows: Enable ETW events.
Igor Zinkovsky authored
141 parser.add_option("--with-etw",
142 action="store_true",
143 dest="with_etw",
144 help="Build with ETW (default is true on Windows)")
145
146 parser.add_option("--without-etw",
147 action="store_true",
148 dest="without_etw",
149 help="Build without ETW")
150
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
151 # CHECKME does this still work with recent releases of V8?
152 parser.add_option("--gdb",
153 action="store_true",
154 dest="gdb",
155 help="add gdb support")
156
157 parser.add_option("--dest-cpu",
158 action="store",
159 dest="dest_cpu",
160 help="CPU architecture to build for. Valid values are: arm, ia32, x64")
161
0ef6cfd @isaacs Fix #3465 Add ./configure --no-ifaddrs flag
isaacs authored
162 parser.add_option("--no-ifaddrs",
163 action="store_true",
164 dest="no_ifaddrs",
165 help="Use on deprecated SunOS systems that do not support ifaddrs.h")
166
90efdb3 @adammw build: improve armv7 / hard-float detection
adammw authored
167 parser.add_option("--with-arm-float-abi",
168 action="store",
169 dest="arm_float_abi",
170 help="Specifies which floating-point ABI to use. Valid values are: "
171 "soft, softfp, hard")
172
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
173 (options, args) = parser.parse_args()
174
175
a033261 @bnoordhuis build: DRY configure script
bnoordhuis authored
176 def b(value):
177 """Returns the string 'true' if value is truthy, 'false' otherwise."""
178 if value:
179 return 'true'
180 else:
181 return 'false'
182
183
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
184 def pkg_config(pkg):
185 cmd = os.popen('pkg-config --libs %s' % pkg, 'r')
186 libs = cmd.readline().strip()
187 ret = cmd.close()
188 if (ret): return None
189
190 cmd = os.popen('pkg-config --cflags %s' % pkg, 'r')
191 cflags = cmd.readline().strip()
192 ret = cmd.close()
193 if (ret): return None
194
195 return (libs, cflags)
196
197
90efdb3 @adammw build: improve armv7 / hard-float detection
adammw authored
198 def cc_macros():
199 """Checks predefined macros using the CC command."""
19133ca @TooTallNate build: better host_arch() definition in configure
TooTallNate authored
200
792d9a9 @javihernandez build: print error message if no compiler found
javihernandez authored
201 try:
202 p = subprocess.Popen(CC.split() + ['-dM', '-E', '-'],
203 stdin=subprocess.PIPE,
204 stdout=subprocess.PIPE,
205 stderr=subprocess.PIPE)
206 except OSError:
207 print '''Node.js configure error: No acceptable C compiler found!
208
209 Please make sure you have a C compiler installed on your system and/or
210 consider adjusting the CC environment variable if you installed
211 it in a non-standard prefix.
212 '''
213 sys.exit()
214
19133ca @TooTallNate build: better host_arch() definition in configure
TooTallNate authored
215 p.stdin.write('\n')
216 out = p.communicate()[0]
217
218 out = str(out).split('\n')
219
220 k = {}
221 for line in out:
222 import shlex
223 lst = shlex.split(line)
224 if len(lst) > 2:
225 key = lst[1]
226 val = lst[2]
227 k[key] = val
90efdb3 @adammw build: improve armv7 / hard-float detection
adammw authored
228 return k
229
230
231 def is_arch_armv7():
232 """Check for ARMv7 instructions"""
233 cc_macros_cache = cc_macros()
234 return ('__ARM_ARCH_7__' in cc_macros_cache or
235 '__ARM_ARCH_7A__' in cc_macros_cache or
236 '__ARM_ARCH_7R__' in cc_macros_cache or
237 '__ARM_ARCH_7M__' in cc_macros_cache)
238
239
240 def arm_hard_float_abi():
241 """Check for hardfloat or softfloat eabi on ARM"""
242 # GCC versions 4.6 and above define __ARM_PCS or __ARM_PCS_VFP to specify
243 # the Floating Point ABI used (PCS stands for Procedure Call Standard).
244 # We use these as well as a couple of other defines to statically determine
245 # what FP ABI used.
246 # GCC versions 4.4 and below don't support hard-fp.
247 # GCC versions 4.5 may support hard-fp without defining __ARM_PCS or
248 # __ARM_PCS_VFP.
249
250 if compiler_version() >= (4, 6, 0):
251 return '__ARM_PCS_VFP' in cc_macros()
252 elif compiler_version() < (4, 5, 0):
253 return False
254 elif '__ARM_PCS_VFP' in cc_macros():
255 return True
256 elif ('__ARM_PCS' in cc_macros() or
257 '__SOFTFP' in cc_macros() or
258 not '__VFP_FP__' in cc_macros()):
259 return False
260 else:
261 print '''Node.js configure error: Your version of GCC does not report
262 the Floating-Point ABI to compile for your hardware
263
264 Please manually specify which floating-point ABI to use with the
265 --with-arm-float-abi option.
266 '''
267 sys.exit()
268
269
270 def host_arch_cc():
271 """Host architecture check using the CC command."""
272
273 k = cc_macros()
19133ca @TooTallNate build: better host_arch() definition in configure
TooTallNate authored
274
275 matchup = {
276 '__x86_64__' : 'x64',
277 '__i386__' : 'ia32',
278 '__arm__' : 'arm',
09ccbef @Skomski build: detect host_arch better
Skomski authored
279 }
280
19133ca @TooTallNate build: better host_arch() definition in configure
TooTallNate authored
281 rtn = 'ia32' # default
282
283 for i in matchup:
284 if i in k and k[i] != '0':
285 rtn = matchup[i]
286 break
09ccbef @Skomski build: detect host_arch better
Skomski authored
287
19133ca @TooTallNate build: better host_arch() definition in configure
TooTallNate authored
288 return rtn
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
289
290
dc75232 @TooTallNate vcbuild: run the 'configure' script in vcbuild.bat
TooTallNate authored
291 def host_arch_win():
292 """Host architecture check using environ vars (better way to do this?)"""
293
294 arch = os.environ.get('PROCESSOR_ARCHITECTURE', 'x86')
295
296 matchup = {
297 'AMD64' : 'x64',
298 'x86' : 'ia32',
299 'arm' : 'arm',
300 }
301
302 return matchup.get(arch, 'ia32')
303
304
07e5877 @bnoordhuis build: disable strict aliasing in v8 with gcc 4.5.x
bnoordhuis authored
305 def compiler_version():
3e5139f @piscisaureus Fix the Windows build
piscisaureus authored
306 try:
307 proc = subprocess.Popen(CC.split() + ['--version'], stdout=subprocess.PIPE)
308 except WindowsError:
309 return (0, False)
310
a0add91 @bnoordhuis build: detect cc version with -dumpversion
bnoordhuis authored
311 is_clang = 'clang' in proc.communicate()[0].split('\n')[0]
f78ce08 @bnoordhuis build: handle output of localized gcc or clang
bnoordhuis authored
312
a0add91 @bnoordhuis build: detect cc version with -dumpversion
bnoordhuis authored
313 proc = subprocess.Popen(CC.split() + ['-dumpversion'], stdout=subprocess.PIPE)
314 version = tuple(map(int, proc.communicate()[0].split('.')))
f78ce08 @bnoordhuis build: handle output of localized gcc or clang
bnoordhuis authored
315
316 return (version, is_clang)
317
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
318
319 def configure_node(o):
98b1ce9 @bnoordhuis build: fix mksnapshot crash on sunos
bnoordhuis authored
320 # TODO add gdb
321 o['variables']['v8_no_strict_aliasing'] = 1 # work around compiler bugs
06c82c5 @bnoordhuis build: expand ~ in `./configure --prefix=~/a/b/c`
bnoordhuis authored
322 o['variables']['node_prefix'] = os.path.expanduser(options.prefix or '')
a9f2c4a @indutny --without-npm option
indutny authored
323 o['variables']['node_install_npm'] = b(not options.without_npm)
6e76a7c @indutny installer: install node-waf
indutny authored
324 o['variables']['node_install_waf'] = b(not options.without_waf)
680d75a @shigeki build: make default_configuration consistent with BUILDTYPE
shigeki authored
325 o['default_configuration'] = 'Debug' if options.debug else 'Release'
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
326
c6bb361 @bnoordhuis build: partially fix configure on ARM
bnoordhuis authored
327 host_arch = host_arch_win() if os.name == 'nt' else host_arch_cc()
328 target_arch = options.dest_cpu or host_arch
329 o['variables']['host_arch'] = host_arch
330 o['variables']['target_arch'] = target_arch
331
90efdb3 @adammw build: improve armv7 / hard-float detection
adammw authored
332 # V8 on ARM requires that armv7 is set. CPU Model detected by
333 # the presence of __ARM_ARCH_7__ and the like defines in compiler
c6bb361 @bnoordhuis build: partially fix configure on ARM
bnoordhuis authored
334 if target_arch == 'arm':
90efdb3 @adammw build: improve armv7 / hard-float detection
adammw authored
335 if options.arm_float_abi:
336 hard_float = options.arm_float_abi == 'hard'
337 else:
338 hard_float = arm_hard_float_abi()
339 o['variables']['v8_use_arm_eabi_hardfloat'] = b(hard_float)
340 o['variables']['armv7'] = 1 if is_arch_armv7() else 0
07e5877 @bnoordhuis build: disable strict aliasing in v8 with gcc 4.5.x
bnoordhuis authored
341
c6bb361 @bnoordhuis build: partially fix configure on ARM
bnoordhuis authored
342 cc_version, is_clang = compiler_version()
100e163 @bnoordhuis build: compile with -fno-tree-vrp when gcc >= 4.0
bnoordhuis authored
343 o['variables']['clang'] = 1 if is_clang else 0
344
8677860 @isaacs build: Don't set gcc_version for Windows
isaacs authored
345 if not is_clang and cc_version != 0:
100e163 @bnoordhuis build: compile with -fno-tree-vrp when gcc >= 4.0
bnoordhuis authored
346 o['variables']['gcc_version'] = 10 * cc_version[0] + cc_version[1]
347
348 # clang has always supported -fvisibility=hidden, right?
07e5877 @bnoordhuis build: disable strict aliasing in v8 with gcc 4.5.x
bnoordhuis authored
349 if not is_clang and cc_version < (4,0,0):
5ebc05f @bnoordhuis build: disable -fvisibility=hidden if gcc < 4.0.0
bnoordhuis authored
350 o['variables']['visibility'] = ''
30b29d8 @bnoordhuis build: disable -fstrict-aliasing if gcc < 4.6.0
bnoordhuis authored
351
cc15299 @davepacheco build: add support for DTrace and postmortem
davepacheco authored
352 # By default, enable DTrace on SunOS systems. Don't allow it on other
353 # systems, since it won't work. (The MacOS build process is different than
354 # SunOS, and we haven't implemented it.)
355 if sys.platform.startswith('sunos'):
b40f813 @bnoordhuis build: fix spurious mksnapshot crashes for good
bnoordhuis authored
356 o['variables']['node_use_dtrace'] = b(not options.without_dtrace)
cc15299 @davepacheco build: add support for DTrace and postmortem
davepacheco authored
357 elif b(options.with_dtrace) == 'true':
358 raise Exception('DTrace is currently only supported on SunOS systems.')
359 else:
360 o['variables']['node_use_dtrace'] = 'false'
361
f315029 @bnoordhuis build: rename openssl configure switches
bnoordhuis authored
362 if options.no_ifaddrs:
363 o['defines'] += ['SUNOS_NO_IFADDRS']
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
364
35a1421 Windows: Enable ETW events.
Igor Zinkovsky authored
365 # By default, enable ETW on Windows.
366 if sys.platform.startswith('win32'):
367 o['variables']['node_use_etw'] = b(not options.without_etw);
368 elif b(options.with_etw) == 'true':
369 raise Exception('ETW is only supported on Windows.')
370 else:
371 o['variables']['node_use_etw'] = 'false'
372
373
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
374 def configure_libz(o):
d03b848 @tchollingsworth build: support shared zlib
tchollingsworth authored
375 o['variables']['node_shared_zlib'] = b(options.shared_zlib)
376
377 # assume shared_zlib if one of these is set?
378 if options.shared_zlib_libpath:
379 o['libraries'] += ['-L%s' % options.shared_zlib_libpath]
380 if options.shared_zlib_libname:
381 o['libraries'] += ['-l%s' % options.shared_zlib_libname]
382 elif options.shared_zlib:
383 o['libraries'] += ['-lz']
384 if options.shared_zlib_includes:
385 o['include_dirs'] += [options.shared_zlib_includes]
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
386
387
388 def configure_v8(o):
a033261 @bnoordhuis build: DRY configure script
bnoordhuis authored
389 o['variables']['v8_use_snapshot'] = b(not options.without_snapshot)
390 o['variables']['node_shared_v8'] = b(options.shared_v8)
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
391
392 # assume shared_v8 if one of these is set?
393 if options.shared_v8_libpath:
394 o['libraries'] += ['-L%s' % options.shared_v8_libpath]
395 if options.shared_v8_libname:
396 o['libraries'] += ['-l%s' % options.shared_v8_libname]
59ecf2c @isaacs Revert "Revert "build: support shared V8 properly""
isaacs authored
397 elif options.shared_v8:
398 o['libraries'] += ['-lv8']
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
399 if options.shared_v8_includes:
400 o['include_dirs'] += [options.shared_v8_includes]
401
402
403 def configure_openssl(o):
a033261 @bnoordhuis build: DRY configure script
bnoordhuis authored
404 o['variables']['node_use_openssl'] = b(not options.without_ssl)
f315029 @bnoordhuis build: rename openssl configure switches
bnoordhuis authored
405 o['variables']['node_shared_openssl'] = b(options.shared_openssl)
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
406
407 if options.without_ssl:
408 return
409
410 if options.no_ssl2:
411 o['defines'] += ['OPENSSL_NO_SSL2=1']
412
f315029 @bnoordhuis build: rename openssl configure switches
bnoordhuis authored
413 if options.shared_openssl:
414 (libs, cflags) = pkg_config('openssl') or ('-lssl -lcrypto', '')
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
415
f315029 @bnoordhuis build: rename openssl configure switches
bnoordhuis authored
416 if options.shared_openssl_libpath:
417 o['libraries'] += ['-L%s' % options.shared_openssl_libpath]
418
419 if options.shared_openssl_libname:
420 libnames = options.shared_openssl_libname.split(',')
421 o['libraries'] += ['-l%s' % s for s in libnames]
e61de70 @ry gyp: use built-in openssl by default
ry authored
422 else:
423 o['libraries'] += libs.split()
424
f315029 @bnoordhuis build: rename openssl configure switches
bnoordhuis authored
425 if options.shared_openssl_includes:
426 o['include_dirs'] += [options.shared_openssl_includes]
e61de70 @ry gyp: use built-in openssl by default
ry authored
427 else:
428 o['cflags'] += cflags.split()
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
429
430
431 output = {
432 'variables': {},
433 'include_dirs': [],
434 'libraries': [],
435 'defines': [],
436 'cflags': [],
437 }
438
439 configure_node(output)
440 configure_libz(output)
441 configure_v8(output)
442 configure_openssl(output)
443
444 # variables should be a root level element,
445 # move everything else to target_defaults
446 variables = output['variables']
447 del output['variables']
448 output = {
449 'variables': variables,
450 'target_defaults': output
451 }
624f70e @ry GYP: rename options.gypi to config.gypi
ry authored
452 pprint.pprint(output, indent=2)
14b04b0 @ry Remove SCONS deprecate WAF
ry authored
453
e493b29 @bnoordhuis build: respect --debug configure switch
bnoordhuis authored
454 def write(filename, data):
455 filename = os.path.join(root_dir, filename)
456 print "creating ", filename
fdeeabb @TooTallNate configure: don't use "with" for Python 2.5 and older
TooTallNate authored
457 f = open(filename, 'w+')
458 f.write(data)
e493b29 @bnoordhuis build: respect --debug configure switch
bnoordhuis authored
459
460 write('config.gypi', "# Do not edit. Generated by the configure script.\n" +
9b7a6c5 @TooTallNate configure: output a newline at the end of config.gypi
TooTallNate authored
461 pprint.pformat(output, indent=2) + "\n")
e493b29 @bnoordhuis build: respect --debug configure switch
bnoordhuis authored
462
463 write('config.mk', "# Do not edit. Generated by the configure script.\n" +
464 ("BUILDTYPE=%s\n" % ('Debug' if options.debug else 'Release')))
97c9745 @ry Back to WAF
ry authored
465
dc75232 @TooTallNate vcbuild: run the 'configure' script in vcbuild.bat
TooTallNate authored
466 if os.name == 'nt':
d5f13f6 @bnoordhuis build: use proper python interpreter
bnoordhuis authored
467 gyp_args = ['-f', 'msvs', '-G', 'msvs_version=2010']
dc75232 @TooTallNate vcbuild: run the 'configure' script in vcbuild.bat
TooTallNate authored
468 else:
d5f13f6 @bnoordhuis build: use proper python interpreter
bnoordhuis authored
469 gyp_args = ['-f', 'make']
470
471 subprocess.call([sys.executable, 'tools/gyp_node'] + gyp_args)
Something went wrong with that request. Please try again.