Last Comment Bug 725517 - Error during configure after bug 683127: "line 25335: test: too many arguments"
: Error during configure after bug 683127: "line 25335: test: too many arguments"
Status: RESOLVED FIXED
[qa-]
:
Product: Core
Classification: Components
Component: Build Config (show other bugs)
: Trunk
: x86 Windows 7
: -- normal (vote)
: mozilla13
Assigned To: Ed Morley [:emorley]
:
:
Mentors:
Depends on:
Blocks: 683127
  Show dependency treegraph
 
Reported: 2012-02-08 16:11 PST by Ed Morley [:emorley]
Modified: 2012-02-10 19:46 PST (History)
1 user (show)
emorley: in‑testsuite-
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Patch v1 (1.51 KB, patch)
2012-02-09 08:37 PST, Ed Morley [:emorley]
mh+mozilla: review+
Details | Diff | Splinter Review

Description Ed Morley [:emorley] 2012-02-08 16:11:45 PST
Win7 x64 (32bit build)
MozillaBuild 1.6
https://hg.mozilla.org/integration/mozilla-inbound/rev/07da69ba7e52

{
c:/mozilla/inbound/configure: line 25335: test: too many arguments
...
c:/mozilla/inbound/configure: line 25441: test: too many arguments
}

Equates to:
http://mxr.mozilla.org/mozilla-central/source/configure.in#9066
> if test -n "$MOZ_LINKER" -a -z "$MOZ_OLD_LINKER" -a $ac_cv_func_dladdr = no ; then

and...
http://mxr.mozilla.org/mozilla-central/source/configure.in#9073
> if test -n "$MOZ_LINKER" -a -z "$MOZ_OLD_LINKER" -a $ac_cv_func_dladdr = no; then

Added by bug 683127.

Build carries on regardless, perhaps another argument for adding |set -o errexit| to configure (albeit it will require a number of other changes, along the lines of bug 698545 comment 1).
Comment 1 Mike Hommey [:glandium] 2012-02-08 23:23:23 PST
Does quoting "$ac_cv_func_dladdr" solve the issue?
Comment 2 Ed Morley [:emorley] 2012-02-09 08:37:52 PST
Created attachment 595764 [details] [diff] [review]
Patch v1

> Does quoting "$ac_cv_func_dladdr" solve the issue?

Yup :-)
Comment 3 Ed Morley [:emorley] 2012-02-09 14:34:44 PST
https://tbpl.mozilla.org/?tree=Try&rev=138e2820a551
Comment 5 Ed Morley [:emorley] 2012-02-10 19:46:03 PST
https://hg.mozilla.org/mozilla-central/rev/40c94c17b303

Note You need to log in before you can comment on or make changes to this bug.